Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1261)

Unified Diff: chrome/browser/predictors/resource_prefetch_common.h

Issue 2587443002: predictors: Make speculative_prefetch_predictor work with PlzNavigate (Closed)
Patch Set: From SessionID to SessionID::id_type Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/predictors/resource_prefetch_common.h
diff --git a/chrome/browser/predictors/resource_prefetch_common.h b/chrome/browser/predictors/resource_prefetch_common.h
index afbd9240545ce0846f09f6818db313f583d4ee85..decad5b95081ccbb9415e8eb09f989cccd246889 100644
--- a/chrome/browser/predictors/resource_prefetch_common.h
+++ b/chrome/browser/predictors/resource_prefetch_common.h
@@ -8,6 +8,8 @@
#include <stddef.h>
#include "base/time/time.h"
+#include "chrome/browser/sessions/session_tab_helper.h"
alexilin 2016/12/20 14:06:47 Do we really need this here? Would you move it to
ahemery 2016/12/20 14:44:56 Done in patch 6
+#include "components/sessions/core/session_id.h"
#include "url/gurl.h"
class Profile;
@@ -43,17 +45,15 @@ struct NavigationID {
const GURL& main_frame_url,
const base::TimeTicks& creation_time);
NavigationID(const NavigationID& other);
+
bool operator<(const NavigationID& rhs) const;
bool operator==(const NavigationID& rhs) const;
- bool IsSameRenderer(const NavigationID& other) const;
-
// Returns true iff the render_process_id_, render_frame_id_ and
// frame_url_ has been set correctly.
bool is_valid() const;
- int render_process_id;
- int render_frame_id;
+ SessionID::id_type session_id;
GURL main_frame_url;
// NOTE: Even though we store the creation time here, it is not used during

Powered by Google App Engine
This is Rietveld 408576698