commit | fbe75ef510a9d9123f6c7f8cbd54d5f5b5680277 | [log] [tgz] |
---|---|---|
author | Jinsuk Kim <jinsukkim@chromium.org> | Thu Dec 06 22:10:37 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Thu Dec 06 22:10:37 2018 |
tree | 60d6e7590db474043e5295eed8dda2289b51c4e3 | |
parent | 59f5d8a34c2c5dbc2b9178cd1b1450b0a3220bc6 [diff] |
Revert "Reland "Android: Factor InProductHelp UI out of Tab"" This reverts commit 76d39b3a8d8fbd0e990f92a992d80213724de9bd. Reason for revert: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Oreo%20Phone%20Tester/2097 https://crbug.com/907602 doesn't seem to be addressed. Original change's description: > Reland "Android: Factor InProductHelp UI out of Tab" > > > https://chromium-review.googlesource.com/c/chromium/src/+/1272779 had a couple > of bugs in |ToolbarButtonInProductHelpController|: > > 1) Ignored |profile| passed into |maybeShowDownloadContinuingIPH()| and > always used Profile.getLastUsedProfile() > 2) |OnDismissListener| was instantiated (and menu got highlight for > the text bubble) every time |setupAndMaybeShowIPHForFeature| is called, > even when the method doesn't meet the condition for IPH text bubble > and has to early out. > > The CL got reverted due to numerous regressions . This CL fixes them and > relands it. > > This reverts commit d436a7a2e33ee105a1d85370970fde3c88255302. > > Bug: 889682, 907502, 907503, 907770, 907505, 907602 > Change-Id: Id6de148a9fd4fd91ad7df986ca0ba78ed18e79e4 > Reviewed-on: https://chromium-review.googlesource.com/c/1349142 > Reviewed-by: Theresa <twellington@chromium.org> > Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> > Cr-Commit-Position: refs/heads/master@{#614161} TBR=twellington@chromium.org,jinsukkim@chromium.org Change-Id: Id92d47e6ac65a0b1a50f4ad4cb1c0b9a11189993 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 889682, 907502, 907503, 907770, 907505, 907602 Reviewed-on: https://chromium-review.googlesource.com/c/1366456 Reviewed-by: Jinsuk Kim <jinsukkim@chromium.org> Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Cr-Commit-Position: refs/heads/master@{#614501}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .