• Maggie Cai's avatar
    [IntentHandling] Show intent picker icon for open in new tab. · 182b48d8
    Maggie Cai authored
    Currently if we open the link in a new tab (e.g. ctrl + click, context
    menu, from another app), the intent picker icon will not show up even
    there are apps that can handle that URL. This is because:
    1. We check the browser for the web contents in navigation throttle
    before the adding the web contents to tab strip model for new tab case.
    2. When open new tab from new-tab page, the starting URL is empty.
    This CL fixes this issue by moving these checks to only check for
    automatically pop up the bubble or automatically launch the app, but
    still show the icon if there are app found.
    
    BUG=1141049, b/171260488
    
    Change-Id: Idcf4b07c34304bf6664f874142128ceddb6d6569
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2500974Reviewed-by: default avatarDominick Ng <dominickn@chromium.org>
    Commit-Queue: Maggie Cai <mxcai@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#821656}
    182b48d8
apps_navigation_throttle.h 10.7 KB