predictors: Make speculative_prefetch_predictor work with PlzNavigate

chrome/browser/predictors/resource_prefetch_common.cc

 // 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.
 
 #include "chrome/browser/predictors/resource_prefetch_common.h"
 
 #include <string>
 #include <tuple>
 
 #include "base/command_line.h"
@@ skipping 58 matching lines @@
           ResourcePrefetchPredictorConfig::LEARNING |
           ResourcePrefetchPredictorConfig::PREFETCHING_FOR_NAVIGATION |
           ResourcePrefetchPredictorConfig::PREFETCHING_FOR_EXTERNAL;
       return true;
     }
   }
 
   return false;
 }
 
-NavigationID::NavigationID()
-    : render_process_id(-1),
-      render_frame_id(-1) {
+NavigationID::NavigationID() : frame_tree_node_id(-1) {
+  session_id.set_id(-1);  // Invalid.
 }
 
 NavigationID::NavigationID(const NavigationID& other)
-    : render_process_id(other.render_process_id),
-      render_frame_id(other.render_frame_id),
+    : session_id(other.session_id),
+      frame_tree_node_id(other.frame_tree_node_id),
       main_frame_url(other.main_frame_url),
-      creation_time(other.creation_time) {
-}
+      creation_time(other.creation_time) {}
 
 NavigationID::NavigationID(content::WebContents* web_contents)
-    : render_process_id(web_contents->GetRenderProcessHost()->GetID()),
-      render_frame_id(web_contents->GetMainFrame()->GetRoutingID()),
-      main_frame_url(web_contents->GetURL()) {
+    : frame_tree_node_id(web_contents->GetMainFrame()->GetFrameTreeNodeId()),
+      main_frame_url(web_contents->GetURL()),
+      creation_time(base::TimeTicks::Now()) {
+  session_id.set_id(SessionTabHelper::IdForTab(web_contents));
 }
 
 NavigationID::NavigationID(content::WebContents* web_contents,
                            const GURL& main_frame_url,
                            const base::TimeTicks& creation_time)
-    : render_process_id(web_contents->GetRenderProcessHost()->GetID()),
-      render_frame_id(web_contents->GetMainFrame()->GetRoutingID()),
+    : frame_tree_node_id(web_contents->GetMainFrame()->GetFrameTreeNodeId()),
       main_frame_url(main_frame_url),
-      creation_time(creation_time) {}
+      creation_time(creation_time) {
+  session_id.set_id(SessionTabHelper::IdForTab(web_contents));
+}
 
 bool NavigationID::is_valid() const {
-  return render_process_id != -1 && render_frame_id != -1 &&
-      !main_frame_url.is_empty();
+  return session_id.id() != -1 && !main_frame_url.is_empty();

[ahemery, 2016/12/19 14:53:41]

frame_tree_node_id can be -1 and still be valid so not using it to make
assumption about the state

 }
 
 bool NavigationID::operator<(const NavigationID& rhs) const {
   DCHECK(is_valid() && rhs.is_valid());
-  return std::tie(render_process_id, render_frame_id, main_frame_url) <
-    std::tie(rhs.render_process_id, rhs.render_frame_id, rhs.main_frame_url);
+  // Quick int copy to be able to use tie.

[ahemery, 2016/12/19 14:53:41]

Tie uses lvalues

+  int id_lhs = session_id.id();
+  int id_rhs = rhs.session_id.id();
+  return std::tie(id_lhs, frame_tree_node_id, main_frame_url) <
+         std::tie(id_rhs, rhs.frame_tree_node_id, rhs.main_frame_url);
 }
 
 bool NavigationID::operator==(const NavigationID& rhs) const {
   DCHECK(is_valid() && rhs.is_valid());
-  return IsSameRenderer(rhs) && main_frame_url == rhs.main_frame_url;
+  return IsSameLocation(rhs) && main_frame_url == rhs.main_frame_url;
 }
 
-bool NavigationID::IsSameRenderer(const NavigationID& other) const {
+bool NavigationID::IsSameLocation(const NavigationID& other) const {
   DCHECK(is_valid() && other.is_valid());
-  return render_process_id == other.render_process_id &&
-      render_frame_id == other.render_frame_id;
+  return session_id.id() == other.session_id.id() &&
+         frame_tree_node_id == other.frame_tree_node_id;
 }
 
 ResourcePrefetchPredictorConfig::ResourcePrefetchPredictorConfig()
     : mode(0),
       max_navigation_lifetime_seconds(60),
       max_urls_to_track(500),
       max_hosts_to_track(200),
       min_url_visit_count(2),
       max_resources_per_entry(50),
       max_consecutive_misses(3),
@@ skipping 48 matching lines @@
 
 bool ResourcePrefetchPredictorConfig::IsMoreResourcesEnabledForTest() const {
   return max_resources_per_entry == 100;
 }
 
 bool ResourcePrefetchPredictorConfig::IsSmallDBEnabledForTest() const {
   return max_urls_to_track == 200 && max_hosts_to_track == 100;
 }
 
 }  // namespace predictors