Take the navigation type into account when checking if the navigation is in page.

content/browser/web_contents/navigation_controller_impl.cc

Index: content/browser/web_contents/navigation_controller_impl.cc
diff --git a/content/browser/web_contents/navigation_controller_impl.cc b/content/browser/web_contents/navigation_controller_impl.cc
index 6176606cbd44d6bbaeb66e9ee368e57bcd2ac80a..6a7da72fef125ef155e8b8d802716cb06df92d95 100644
--- a/content/browser/web_contents/navigation_controller_impl.cc
+++ b/content/browser/web_contents/navigation_controller_impl.cc
@@ -758,6 +758,10 @@ bool NavigationControllerImpl::RendererDidNavigate(
       RendererDidNavigateToSamePage(params);
       break;
     case NAVIGATION_TYPE_IN_PAGE:
+      // When navigating back from a ref URL to the top non ref one is_in_page
+      // is false (see AreURLsInPageNavigation()). In order to set the flag
+      // properly take the navigation type into account as well.
+      details->is_in_page = true;

[Charlie Reis, 2013/08/05 22:18:10]

Putting it here doesn't help other callers of IsURLInPageNavigation or
AreURLsInPageNavigation.  Those functions will still return the wrong thing for
the "back to non-ref" case.

Maybe you can move the IsURLInPageNavigation call after ClassifyNavigation and
pass in details->type?  That might let us resolve jcampan's TODO statement in
AreURLsInPageNavigation.  We could default to passing in NAVIGATION_TYPE_UNKNOWN
if we don't have the type.

       RendererDidNavigateInPage(params, &details->did_replace_entry);
       break;
     case NAVIGATION_TYPE_NEW_SUBFRAME: