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

Unified Diff: third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp

Issue 2386403002: Pass DOMWinow to PerformanceLongTaskTiming for attribution (Closed)
Patch Set: Address review comments Created 4 years, 2 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: third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
diff --git a/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp b/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
index 3d7dda02e67cf30c8a68801c89cdef669175b9d1..b1519944162bc39b59256a22cade5455f029df27 100644
--- a/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
+++ b/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
@@ -4,16 +4,26 @@
#include "core/timing/PerformanceLongTaskTiming.h"
+#include "bindings/core/v8/V8ObjectBuilder.h"
caseq 2016/10/05 01:05:14 unused now?
panicker 2016/10/05 04:11:31 Done.
+#include "core/frame/DOMWindow.h"
+
namespace blink {
PerformanceLongTaskTiming::PerformanceLongTaskTiming(double startTime,
double endTime,
- String frameContextUrl)
- : PerformanceEntry(frameContextUrl, "longtask", startTime, endTime) {}
+ String name,
+ DOMWindow* culpritWindow)
+ : PerformanceEntry(name, "longtask", startTime, endTime),
+ m_culpritWindow(*culpritWindow) {}
PerformanceLongTaskTiming::~PerformanceLongTaskTiming() {}
+DOMWindow* PerformanceLongTaskTiming::culpritWindow() const {
+ return m_culpritWindow.get();
+}
+
DEFINE_TRACE(PerformanceLongTaskTiming) {
+ visitor->trace(m_culpritWindow);
PerformanceEntry::trace(visitor);
}

Powered by Google App Engine
This is Rietveld 408576698