Fix mis-classification of navigations when there is a pending page.

content/browser/frame_host/navigation_controller_impl.cc

Index: content/browser/frame_host/navigation_controller_impl.cc
diff --git a/content/browser/frame_host/navigation_controller_impl.cc b/content/browser/frame_host/navigation_controller_impl.cc
index f2850dc7efb03aba911884b695c0ff295c6149c4..73f7b3b5533903aa32ec0203d6bfeb5a5497cf50 100644
--- a/content/browser/frame_host/navigation_controller_impl.cc
+++ b/content/browser/frame_host/navigation_controller_impl.cc
@@ -1032,22 +1032,19 @@ NavigationType NavigationControllerImpl::ClassifyNavigation(
       !pending_entry_->is_renderer_initiated() &&
       existing_entry != pending_entry_ &&
       pending_entry_->GetPageID() == -1 &&
-      existing_entry == GetLastCommittedEntry()) {
-    const std::vector<GURL>& existing_redirect_chain =
-        existing_entry->GetRedirectChain();
-
-    if (existing_entry->GetURL() == pending_entry_->GetURL() ||
-        (existing_redirect_chain.size() &&
-         existing_redirect_chain[0] == pending_entry_->GetURL())) {
-      // In this case, we have a pending entry for a URL but WebCore didn't do a
-      // new navigation. This happens when you press enter in the URL bar to
-      // reload. We will create a pending entry, but WebKit will convert it to
-      // a reload since it's the same page and not create a new entry for it
-      // (the user doesn't want to have a new back/forward entry when they do
-      // this). If this matches the last committed entry, we want to just ignore
-      // the pending entry and go back to where we were (the "existing entry").
-      return NAVIGATION_TYPE_SAME_PAGE;
-    }
+      existing_entry == GetLastCommittedEntry() &&
+      !params.was_within_same_page &&

[Charlie Reis, 2015/05/01 21:37:09]

And just for posterity, this looks correct to me because the (confusingly named
in this case) |was_within_same_page| is true for things like replaceState (which
get classified as IN_PAGE) and false for the "hit enter to reload" case (which
gets classified as SAME_PAGE).

[Avi (use Gerrit), 2015/05/01 21:41:55]

This line is what allows NavigationTypeClassification_On1InPageTo1While1Pending
to pass.

+      (params.url == existing_entry->GetURL() ||
+       (params.redirects.size() &&
+        params.redirects[0] == existing_entry->GetURL()))) {
+    // In this case, we have a pending entry for a URL but Blink didn't do a new
+    // navigation. This happens when you press enter in the URL bar to reload.
+    // We will create a pending entry, but Blink will convert it to a reload
+    // since it's the same page and not create a new entry for it (the user
+    // doesn't want to have a new back/forward entry when they do this). If this
+    // matches the last committed entry, we want to just ignore the pending
+    // entry and go back to where we were (the "existing entry").
+    return NAVIGATION_TYPE_SAME_PAGE;
   }
 
   // Any toplevel navigations with the same base (minus the reference fragment)