Add item and document sequence numbers to FrameNavigationEntry.

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 0491cfb0794e3e3b497d048ce104a8d52b42f3e6..04ec809f47020f3a48ec6da3b9465f12228fa63a 100644
--- a/content/browser/frame_host/navigation_controller_impl.cc
+++ b/content/browser/frame_host/navigation_controller_impl.cc
@@ -387,7 +387,7 @@ NavigationEntryImpl* NavigationControllerImpl::GetEntryWithPageID(
 bool NavigationControllerImpl::HasCommittedRealLoad(
     FrameTreeNode* frame_tree_node) const {
   NavigationEntryImpl* last_committed = GetLastCommittedEntry();
-  return last_committed && last_committed->HasFrameEntry(frame_tree_node);
+  return last_committed && last_committed->GetFrameEntry(frame_tree_node);
 }
 
 void NavigationControllerImpl::LoadEntry(NavigationEntryImpl* entry) {
@@ -1223,6 +1223,12 @@ void NavigationControllerImpl::RendererDidNavigateToNewPage(
   new_entry->SetOriginalRequestURL(params.original_request_url);
   new_entry->SetIsOverridingUserAgent(params.is_overriding_user_agent);
 
+  // Update the FrameNavigationEntry for new main frame commits.
+  FrameNavigationEntry* frame_entry =
+      new_entry->GetFrameEntry(rfh->frame_tree_node());

[Charlie Reis, 2015/06/18 20:30:18]

I went ahead and added GetFrameEntry, since it's cleaner not to use
NavigationEntryImpl::TreeNode out here and I'll need it anyway in the next CL.

+  frame_entry->set_item_sequence_number(params.item_sequence_number);
+  frame_entry->set_document_sequence_number(params.document_sequence_number);
+
   // history.pushState() is classified as a navigation to a new page, but
   // sets was_within_same_page to true. In this case, we already have the
   // title and favicon available, so set them immediately.
@@ -1399,8 +1405,9 @@ void NavigationControllerImpl::RendererDidNavigateNewSubframe(
           switches::kSitePerProcess)) {
     // Make sure new_entry takes ownership of frame_entry in a scoped_refptr.
     FrameNavigationEntry* frame_entry = new FrameNavigationEntry(
-        rfh->frame_tree_node()->frame_tree_node_id(), rfh->GetSiteInstance(),
-        params.url, params.referrer);
+        rfh->frame_tree_node()->frame_tree_node_id(),
+        params.item_sequence_number, params.document_sequence_number,
+        rfh->GetSiteInstance(), params.url, params.referrer);
     new_entry = GetLastCommittedEntry()->CloneAndReplace(rfh->frame_tree_node(),
                                                          frame_entry);
     CHECK(frame_entry->HasOneRef());
@@ -1456,6 +1463,8 @@ bool NavigationControllerImpl::RendererDidNavigateAutoSubframe(
     // it may be a "history auto" case where we update an existing one.
     NavigationEntryImpl* last_committed = GetLastCommittedEntry();
     last_committed->AddOrUpdateFrameEntry(rfh->frame_tree_node(),
+                                          params.item_sequence_number,
+                                          params.document_sequence_number,
                                           rfh->GetSiteInstance(), params.url,
                                           params.referrer, params.page_state);