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

Unified Diff: content/common/navigation_params.cc

Issue 1002803002: Classify navigations without page id in parallel to the existing classifier. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: with android Created 5 years, 8 months 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: content/common/navigation_params.cc
diff --git a/content/common/navigation_params.cc b/content/common/navigation_params.cc
index 523a354017c0d0c700d8ddca641d5e744f2e5051..223c69c209ae14b2e72828243f1c791f19f32707 100644
--- a/content/common/navigation_params.cc
+++ b/content/common/navigation_params.cc
@@ -84,6 +84,7 @@ RequestNavigationParams::RequestNavigationParams()
can_load_local_resources(false),
request_time(base::Time::Now()),
page_id(-1),
+ nav_entry_id(0),
clamy 2015/04/27 11:50:04 Shouldn't we initialize intended_as_new_entry as w
Avi (use Gerrit) 2015/04/27 14:56:33 Gah. Done.
pending_history_list_offset(-1),
current_history_list_offset(-1),
current_history_list_length(0),
@@ -98,6 +99,8 @@ RequestNavigationParams::RequestNavigationParams(
base::Time request_time,
const PageState& page_state,
int32 page_id,
+ int nav_entry_id,
+ bool intended_as_new_entry,
int pending_history_list_offset,
int current_history_list_offset,
int current_history_list_length,
@@ -109,6 +112,8 @@ RequestNavigationParams::RequestNavigationParams(
request_time(request_time),
page_state(page_state),
page_id(page_id),
+ nav_entry_id(nav_entry_id),
+ intended_as_new_entry(intended_as_new_entry),
pending_history_list_offset(pending_history_list_offset),
current_history_list_offset(current_history_list_offset),
current_history_list_length(current_history_list_length),

Powered by Google App Engine
This is Rietveld 408576698