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

Unified Diff: Source/core/loader/DocumentLoadTiming.cpp

Issue 379873002: Fix Performance::now() for browser navigations. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Make the comparison in the assert "<=". Created 6 years, 5 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/loader/DocumentLoadTiming.cpp
diff --git a/Source/core/loader/DocumentLoadTiming.cpp b/Source/core/loader/DocumentLoadTiming.cpp
index 6204ca613575cfa5214b40a644c225e5ac9384de..3d7edd2a1d51f8d0da6532b220d9df9f677c5750 100644
--- a/Source/core/loader/DocumentLoadTiming.cpp
+++ b/Source/core/loader/DocumentLoadTiming.cpp
@@ -74,7 +74,18 @@ void DocumentLoadTiming::markNavigationStart()
void DocumentLoadTiming::setNavigationStart(double navigationStart)
{
ASSERT(m_referenceMonotonicTime && m_referenceWallTime);
+
+ // This should be used only to indicate that the navigation started in the
+ // embedder before the renderer was created.
+ ASSERT(navigationStart <= m_navigationStart);
m_navigationStart = navigationStart;
+
+ // |m_referenceMonotonicTime| and |m_referenceWallTime| represent
+ // navigationStart. When the embedder sets navigationStart (because the
+ // navigation started earlied on the browser side), we need to adjust these
+ // as well.
+ m_referenceWallTime = monotonicTimeToPseudoWallTime(navigationStart);
+ m_referenceMonotonicTime = navigationStart;
}
void DocumentLoadTiming::addRedirect(const KURL& redirectingUrl, const KURL& redirectedUrl)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698