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

Unified Diff: third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp

Issue 2472583003: Navigation Timing Level 2 (Closed)
Patch Set: First working version Created 4 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: third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp
diff --git a/third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp b/third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp
index 5ba9b469a15c86c1539ea5390ec2abd981c4a013..57fc4921e1689af6d1d900b1ec57ac7aa8ab9e02 100644
--- a/third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp
+++ b/third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp
@@ -74,6 +74,8 @@
#include "core/page/Page.h"
#include "core/page/WindowFeatures.h"
#include "core/page/scrolling/ScrollingCoordinator.h"
+#include "core/timing/DOMWindowPerformance.h"
+#include "core/timing/PerformanceBase.h"
#include "platform/EventDispatchForbiddenScope.h"
#include "platform/weborigin/SecurityOrigin.h"
#include "platform/weborigin/Suborigin.h"
@@ -1398,6 +1400,11 @@ void LocalDOMWindow::dispatchLoadEvent() {
} else {
dispatchEvent(loadEvent, document());
}
+ // This is unfortunately that we have to cast in here
panicker 2016/11/02 20:12:37 You shouldn't need a cast here.
sunjian 2016/11/04 01:12:38 Done.
+ // (TODO:sunjian) Find a better way of doing it
+ // Also check nullptr
panicker 2016/11/02 20:12:37 Add DCHECK here: Performance* performance = DOMWin
sunjian 2016/11/04 01:12:38 Done.
+ ((PerformanceBase*)(DOMWindowPerformance::performance(*this)))
+ ->addNavigationTiming(frame());
// For load events, send a separate load event to the enclosing frame only.
// This is a DOM extension and is independent of bubbling/capturing rules of

Powered by Google App Engine
This is Rietveld 408576698