GitLab

Currently the decision to create new BrowsingInstances are made in two
functions: ShouldSwapBrowsingInstancesForNavigation and
DetermineSiteInstanceForURL, possibly causing confusion when only
looking at one of the functions, and other issues like not being able
to know whether it is a "forces swap" (requiring different processes)
or a "proactive swap" (may not require different processes). This CL
fixes that by moving all the BI swaps to
ShouldSwapBrowsingInstancesForNavigation.

Bug: 977562
Change-Id: Id5fb8785399584199ac0711811e9ec8b6d6efe03
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2081728
Commit-Queue: Rakina Zata Amni <rakina@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#750897}