Preserve should_replace_current_entry across request transfers.

When a request transfer happens, we lose the should_replace_current_entry bit
of the navigation. This results in us getting history wrong when, say, a hosted
Google Calendar app is installed and we client-redirect to it while not logged
in, triggering a server redirect to accounts.google.com and a request transfer.

Push this state down to the renderer into NavigationState so it can be
stapled to the request, bubbled back up to the CrossSiteResourceHandler and
into the transferred navigation.

Also test and fix a case where the state does not round-trip correctly if a
fresh renderer with a browser-initiated navigation decides to fork the
navigation back up to the browser anyway.

BUG=311721
TEST=SitePerProcessBrowserTest.ReplaceEntryCrossProcessThenTransfers
     SitePerProcessBrowserTest.ReplaceEntryInProcessThenTransfers
     SitePerProcessBrowserTest.ReplaceEntryCrossProcessTwice

Review URL: https://codereview.chromium.org/71993002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@235918 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 1 insertions, 1 deletions

diff --git a/content/child/resource_dispatcher_unittest.cc b/content/child/resource_dispatcher_unittest.cc
index a5b0880..76627ad 100644
--- a/content/child/resource_dispatcher_unittest.cc
+++ b/content/child/resource_dispatcher_unittest.cc
@@ -177,7 +177,7 @@ class ResourceDispatcherTest : public testing::Test, public IPC::Sender {
                                 blink::WebString(),
                                 false, true, 0, GURL(),
                                 false, -1, true,
-                                PAGE_TRANSITION_LINK, -1, -1);
+                                PAGE_TRANSITION_LINK, false, -1, -1);
     request_info.extra_data = &extra_data;
 
     return dispatcher_->CreateBridge(request_info);