Disallow was_within_same_page = true for a cross-process navigation

content/test/test_render_frame_host.cc

Index: content/test/test_render_frame_host.cc
diff --git a/content/test/test_render_frame_host.cc b/content/test/test_render_frame_host.cc
index 0b0a95396ed4f321140da5c8882ea00c78fd6a56..cb587ceaccd6b7deba1a6e1bdfaeeedd5d65e6a7 100644
--- a/content/test/test_render_frame_host.cc
+++ b/content/test/test_render_frame_host.cc
@@ -271,10 +271,12 @@ void TestRenderFrameHost::SendNavigateWithParameters(
     ui::PageTransition transition,
     int response_code,
     const ModificationCallback& callback) {
+  if (!IsBrowserSideNavigationEnabled())

[nasko, 2016/03/01 00:27:01]

Why is browser-side navigation different here?

[gzobqq, 2016/03/01 08:31:42]

With browser-side navigation RFHI::OnDidStartLoading(true) does nothing but
ReceivedBadMessage(), which interferes with the new test. Renderer also sends
FrameHostMsg_DidStartLoading only when !IsBrowserSideNavigationEnabled() ||
!to_different_document. It was a recent change by clamy in
https://codereview.chromium.org/1608283002

+    OnDidStartLoading(true);
+
   // DidStartProvisionalLoad may delete the pending entry that holds |url|,
   // so we keep a copy of it to use below.
   GURL url_copy(url);
-  OnDidStartLoading(true);
   OnDidStartProvisionalLoad(url_copy, base::TimeTicks::Now());
 
   FrameHostMsg_DidCommitProvisionalLoad_Params params;