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

Unified Diff: content/browser/loader/navigation_url_loader_impl_core.cc

Issue 874353003: New TimeToFirstURLJob* navigation metrics now work with PlzNavigate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Updated method names and comments. Created 5 years, 11 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/browser/loader/navigation_url_loader_impl_core.cc
diff --git a/content/browser/loader/navigation_url_loader_impl_core.cc b/content/browser/loader/navigation_url_loader_impl_core.cc
index 48845e8aa0c26976ecb6eee07461a88ef7ea4a1a..63e65928e05480e12fcb88d8f7c5e50f1b9d0251 100644
--- a/content/browser/loader/navigation_url_loader_impl_core.cc
+++ b/content/browser/loader/navigation_url_loader_impl_core.cc
@@ -101,4 +101,13 @@ void NavigationURLLoaderImplCore::NotifyRequestFailed(int net_error) {
net_error));
}
+void NavigationURLLoaderImplCore::NavigationRequested(
+ base::TimeTicks timestamp) {
+ DCHECK_CURRENTLY_ON(BrowserThread::IO);
+ BrowserThread::PostTask(
+ BrowserThread::UI, FROM_HERE,
+ base::Bind(&NavigationURLLoaderImpl::NavigationRequested, loader_,
+ timestamp));
davidben 2015/01/28 21:33:27 Rather than plumb this all the way to the RDH and
carlosk 2015/01/30 14:01:47 Done. I measured the time we'd not be accounting f
+}
+
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698