Classify navigations without page id in parallel to the existing classifier.

content/common/navigation_params.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_COMMON_NAVIGATION_PARAMS_H_
 #define CONTENT_COMMON_NAVIGATION_PARAMS_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 122 matching lines @@
   // TODO(ppi): make it go away.
   base::Time request_time;
 };
 
 // Used by FrameMsg_Navigate.
 // PlzNavigate: sent to the renderer when the navigation is ready to commit.
 struct CONTENT_EXPORT HistoryNavigationParams {
   HistoryNavigationParams();
   HistoryNavigationParams(const PageState& page_state,
                           int32 page_id,
+                          int nav_entry_id,
                           int pending_history_list_offset,
                           int current_history_list_offset,
                           int current_history_list_length,
                           bool should_clear_history_list);
   ~HistoryNavigationParams();
 
   // Opaque history state (received by ViewHostMsg_UpdateState).
   PageState page_state;
 
   // The page_id for this navigation, or -1 if it is a new navigation.  Back,
   // Forward, and Reload navigations should have a valid page_id.  If the load
   // succeeds, then this page_id will be reflected in the resultant
   // FrameHostMsg_DidCommitProvisionalLoad message.
   int32 page_id;

[Charlie Reis, 2015/03/12 18:28:17]

Maybe put a TODO(avi) to remove this in favor of nav_entry_id?

[Avi (use Gerrit), 2015/03/12 19:21:17]

It feels weird to tag this one place that uses page id and not tag every other
place.

 
+  // This is the unique id of the NavigationEntry being navigated to. If the
+  // load succeeds, then this nav_entry_id will be reflected in the resultant
+  // FrameHostMsg_DidCommitProvisionalLoad message.
+  int nav_entry_id;
+
   // For history navigations, this is the offset in the history list of the
   // pending load. For non-history navigations, this will be ignored.
   int pending_history_list_offset;
 
   // Where its current page contents reside in session history and the total
   // size of the session history list.
   int current_history_list_offset;
   int current_history_list_length;
 
   // Whether session history should be cleared. In that case, the RenderView
@@ skipping 46 matching lines @@
   ~NavigationParams();
 
   CommonNavigationParams common_params;
   StartNavigationParams start_params;
   CommitNavigationParams commit_params;
   HistoryNavigationParams history_params;
 };
 }  // namespace content
 
 #endif  // CONTENT_COMMON_NAVIGATION_PARAMS_H_