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

Unified Diff: content/common/navigation_params.cc

Issue 1432583002: Move browser_navigation_start to CommonNavigationParams (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nasko review Created 5 years, 1 month 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
« no previous file with comments | « content/common/navigation_params.h ('k') | content/public/test/render_view_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/navigation_params.cc
diff --git a/content/common/navigation_params.cc b/content/common/navigation_params.cc
index 63e6a118b7b7dfeedb86cec24bdc40c7a5344e89..790f2d2ec9f467fbeaf0eb441368903d09914e3c 100644
--- a/content/common/navigation_params.cc
+++ b/content/common/navigation_params.cc
@@ -30,7 +30,8 @@ CommonNavigationParams::CommonNavigationParams()
allow_download(true),
should_replace_current_entry(false),
report_type(FrameMsg_UILoadMetricsReportType::NO_REPORT),
- lofi_state(LOFI_UNSPECIFIED) {
+ lofi_state(LOFI_UNSPECIFIED),
+ navigation_start(base::TimeTicks::Now()) {
}
CommonNavigationParams::CommonNavigationParams(
@@ -44,7 +45,8 @@ CommonNavigationParams::CommonNavigationParams(
FrameMsg_UILoadMetricsReportType::Value report_type,
const GURL& base_url_for_data_url,
const GURL& history_url_for_data_url,
- LoFiState lofi_state)
+ LoFiState lofi_state,
+ const base::TimeTicks& navigation_start)
: url(url),
referrer(referrer),
transition(transition),
@@ -55,7 +57,8 @@ CommonNavigationParams::CommonNavigationParams(
report_type(report_type),
base_url_for_data_url(base_url_for_data_url),
history_url_for_data_url(history_url_for_data_url),
- lofi_state(lofi_state) {
+ lofi_state(lofi_state),
+ navigation_start(navigation_start) {
}
CommonNavigationParams::~CommonNavigationParams() {
@@ -114,7 +117,6 @@ StartNavigationParams::~StartNavigationParams() {
RequestNavigationParams::RequestNavigationParams()
: is_overriding_user_agent(false),
- browser_navigation_start(base::TimeTicks::Now()),
can_load_local_resources(false),
request_time(base::Time::Now()),
page_id(-1),
@@ -131,7 +133,6 @@ RequestNavigationParams::RequestNavigationParams()
RequestNavigationParams::RequestNavigationParams(
bool is_overriding_user_agent,
- base::TimeTicks navigation_start,
const std::vector<GURL>& redirects,
bool can_load_local_resources,
base::Time request_time,
@@ -146,7 +147,6 @@ RequestNavigationParams::RequestNavigationParams(
int current_history_list_length,
bool should_clear_history_list)
: is_overriding_user_agent(is_overriding_user_agent),
- browser_navigation_start(navigation_start),
redirects(redirects),
can_load_local_resources(can_load_local_resources),
request_time(request_time),
« no previous file with comments | « content/common/navigation_params.h ('k') | content/public/test/render_view_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698