• Alex Moshchuk's avatar
    Reland "Allow navigations to opener's site to go back into opener SiteInstance." · 46a28e9a
    Alex Moshchuk authored
    This is a reland of cd5b76ea.
    
    PrerenderBrowserTest.PrerenderWindowSize still failing after revert.
    
    Original change's description:
    > Allow navigations to opener's site to go back into opener SiteInstance.
    >
    > Previously, when a popup and its opener were cross-site, and neither
    > site required a dedicated process, navigating the popup back to the
    > opener's site did not swap processes, so the two ended up in different
    > processes and couldn't script each other.  Even though
    > DetermineSiteInstanceForURL contains logic to place same-site popups
    > into their opener's SiteInstance (added as part of the fix for a
    > similar issue 796912), this logic was never reached because we never
    > attempted a process transfer in this case, thanks to
    > IsRendererTransferNeededForNavigation() returning false.
    >
    > This CL tweaks IsRendererTransferNeededForNavigation so that a
    > cross-process transfer is always attempted is such cases.  This allows
    > DetermineSiteInstanceForURL to always place the popup that's going back
    > to its opener site into the opener's SiteInstance.
    >
    > Bug: 807184
    > Change-Id: Ia4d8d8b2107d3b9e3b9329dd473c06d49b7bd5ec
    > Reviewed-on: https://chromium-review.googlesource.com/894369
    > Reviewed-by: Charlie Reis <creis@chromium.org>
    > Commit-Queue: Alex Moshchuk <alexmos@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#533410}
    
    TBR=alexmos@chromium.org
    
    Bug: 807184, 807821
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Change-Id: I7808cef9292ff069bcfdd00fe71883d321d7f577
    Reviewed-on: https://chromium-review.googlesource.com/897262
    Commit-Queue: Trent Apted <tapted@chromium.org>
    Reviewed-by: default avatarTrent Apted <tapted@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#533588}
    46a28e9a
render_frame_host_manager.cc 104 KB