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

Unified Diff: content/browser/frame_host/navigation_handle_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_handle_impl.cc
diff --git a/content/browser/frame_host/navigation_handle_impl.cc b/content/browser/frame_host/navigation_handle_impl.cc
index b1aace3597c5ecd773339efa3b2472beac3db8a7..204682edb9a92918292c49495d5367eb79b6d101 100644
--- a/content/browser/frame_host/navigation_handle_impl.cc
+++ b/content/browser/frame_host/navigation_handle_impl.cc
@@ -33,13 +33,16 @@ void UpdateThrottleCheckResult(
// static
scoped_ptr<NavigationHandleImpl> NavigationHandleImpl::Create(
const GURL& url,
- FrameTreeNode* frame_tree_node) {
+ FrameTreeNode* frame_tree_node,
+ const base::TimeTicks& navigation_start) {
return scoped_ptr<NavigationHandleImpl>(
- new NavigationHandleImpl(url, frame_tree_node));
+ new NavigationHandleImpl(url, frame_tree_node, navigation_start));
}
-NavigationHandleImpl::NavigationHandleImpl(const GURL& url,
- FrameTreeNode* frame_tree_node)
+NavigationHandleImpl::NavigationHandleImpl(
+ const GURL& url,
+ FrameTreeNode* frame_tree_node,
+ const base::TimeTicks& navigation_start)
: url_(url),
is_post_(false),
has_user_gesture_(false),
@@ -51,7 +54,9 @@ NavigationHandleImpl::NavigationHandleImpl(const GURL& url,
state_(INITIAL),
is_transferring_(false),
frame_tree_node_(frame_tree_node),
- next_index_(0) {
+ next_index_(0),
+ navigation_start_(navigation_start) {
+ DCHECK(!navigation_start.is_null());
// PlzNavigate
// Initialize the ServiceWorkerNavigationHandle if it can be created for this
// frame.
@@ -94,6 +99,10 @@ bool NavigationHandleImpl::IsInMainFrame() {
return frame_tree_node_->IsMainFrame();
}
+const base::TimeTicks& NavigationHandleImpl::GetNavigationStart() {
+ return navigation_start_;
+}
+
bool NavigationHandleImpl::IsPost() {
CHECK_NE(INITIAL, state_)
<< "This accessor should not be called before the request is started.";

Powered by Google App Engine
This is Rietveld 408576698