Trust the renderer's same-document navigation flag if it is a same-origin nav.

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 9eb1eaae6d9a61d72d9b6f26a5a6e00f010d1142..98408fb444023036eba2ea7654b62e59010fdea0 100644
--- a/content/test/test_render_frame_host.cc
+++ b/content/test/test_render_frame_host.cc
@@ -46,7 +46,7 @@ void TestRenderFrameHost::SendNavigateWithTransition(
 
 void TestRenderFrameHost::SendFailedNavigate(int page_id, const GURL& url) {
   SendNavigateWithTransitionAndResponseCode(
-      page_id, url, PAGE_TRANSITION_LINK, 500);
+      page_id, url, PAGE_TRANSITION_RELOAD, 500);
 }
 
 void TestRenderFrameHost::SendNavigateWithTransitionAndResponseCode(
@@ -113,13 +113,19 @@ void TestRenderFrameHost::SendNavigateWithParameters(
   params.gesture = NavigationGestureUser;
   params.contents_mime_type = contents_mime_type_;
   params.is_post = false;
-  params.was_within_same_page = false;
   params.http_status_code = response_code;
   params.socket_address.set_host("2001:db8::1");
   params.socket_address.set_port(80);
   params.history_list_was_cleared = simulate_history_list_was_cleared_;
   params.original_request_url = original_request_url;
 
+  url::Replacements<char> replacements;
+  replacements.ClearRef();
+  params.was_within_same_page = transition != PAGE_TRANSITION_RELOAD &&

[nasko, 2014/05/29 22:59:30]

Why have the transition type being checked? Wouldn't PAGE_TRANSITION_LINK to a
#ref on the page still be considered "within_same_page"?

[Nate Chapin, 2014/05/29 23:03:46]

A link transition can be based on this code.

On the other hand, reloads are never within_same_page, nor are typed urls (i.e.,
the user focusing the address bar and pressing enter).

+      transition != PAGE_TRANSITION_TYPED &&
+      url.ReplaceComponents(replacements) ==
+          GetLastCommittedURL().ReplaceComponents(replacements);
+
   params.page_state = PageState::CreateForTesting(
       url,
       false,