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

Unified Diff: content/browser/frame_host/navigation_entry_impl.cc

Issue 1425823002: (DEPRECATED) Send navigation_start to browser process in DidStartProvisionalLoad (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Stop calling didCreateDataSource for same-page navs 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
Index: content/browser/frame_host/navigation_entry_impl.cc
diff --git a/content/browser/frame_host/navigation_entry_impl.cc b/content/browser/frame_host/navigation_entry_impl.cc
index 3c20692e03c50312f30edc0e96f0d122e6788369..367702b8e686518634b73947cf61aeab6992dd79 100644
--- a/content/browser/frame_host/navigation_entry_impl.cc
+++ b/content/browser/frame_host/navigation_entry_impl.cc
@@ -442,7 +442,8 @@ CommonNavigationParams NavigationEntryImpl::ConstructCommonNavigationParams(
const Referrer& dest_referrer,
const FrameNavigationEntry& frame_entry,
FrameMsg_Navigate_Type::Value navigation_type,
- LoFiState lofi_state) const {
+ LoFiState lofi_state,
+ const base::TimeTicks& navigation_start) const {
FrameMsg_UILoadMetricsReportType::Value report_type =
FrameMsg_UILoadMetricsReportType::NO_REPORT;
base::TimeTicks ui_timestamp = base::TimeTicks();
@@ -455,7 +456,8 @@ CommonNavigationParams NavigationEntryImpl::ConstructCommonNavigationParams(
return CommonNavigationParams(
dest_url, dest_referrer, GetTransitionType(), navigation_type,
!IsViewSourceMode(), should_replace_entry(), ui_timestamp, report_type,
- GetBaseURLForDataURL(), GetHistoryURLForDataURL(), lofi_state);
+ GetBaseURLForDataURL(), GetHistoryURLForDataURL(), lofi_state,
+ navigation_start);
}
StartNavigationParams NavigationEntryImpl::ConstructStartNavigationParams()
@@ -479,7 +481,6 @@ StartNavigationParams NavigationEntryImpl::ConstructStartNavigationParams()
RequestNavigationParams NavigationEntryImpl::ConstructRequestNavigationParams(
const FrameNavigationEntry& frame_entry,
- base::TimeTicks navigation_start,
bool is_same_document_history_load,
bool has_committed_real_load,
bool intended_as_new_entry,
@@ -505,12 +506,11 @@ RequestNavigationParams NavigationEntryImpl::ConstructRequestNavigationParams(
current_length_to_send = 0;
}
return RequestNavigationParams(
- GetIsOverridingUserAgent(), navigation_start, redirects,
- GetCanLoadLocalResources(), base::Time::Now(), frame_entry.page_state(),
- GetPageID(), GetUniqueID(), is_same_document_history_load,
- has_committed_real_load, intended_as_new_entry, pending_offset_to_send,
- current_offset_to_send, current_length_to_send,
- should_clear_history_list());
+ GetIsOverridingUserAgent(), redirects, GetCanLoadLocalResources(),
+ base::Time::Now(), frame_entry.page_state(), GetPageID(), GetUniqueID(),
+ is_same_document_history_load, has_committed_real_load,
+ intended_as_new_entry, pending_offset_to_send, current_offset_to_send,
+ current_length_to_send, should_clear_history_list());
}
void NavigationEntryImpl::ResetForCommit() {

Powered by Google App Engine
This is Rietveld 408576698