Reland "Android: Factor InProductHelp UI out of Tab" again

This reverts commit fbe75ef510a9d9123f6c7f8cbd54d5f5b5680277.

Reason for revert: Reverted on a false alarm. Landing again.

Original change's description:
> 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}

TBR=twellington@chromium.org,jinsukkim@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 889682, 907502, 907503, 907770, 907505, 907602
Change-Id: I1cdb497251b4a1f60f955cc6fd90c2be1eae3e96
Reviewed-on: https://chromium-review.googlesource.com/c/1368844
Reviewed-by: Jinsuk Kim <jinsukkim@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#614875}
3 files changed