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

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

Issue 2515993002: Replace DOMWindow with iframe attrs: src, id, name (Closed)
Patch Set: don't truncate ID 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/timing/PerformanceLongTaskTiming.cpp
diff --git a/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp b/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
index c986b10bb93bd80661d11c058aabc643cd480908..f8266a5224eed9a044d86b3fb3149888d5db85ad 100644
--- a/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
+++ b/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
@@ -11,18 +11,29 @@ namespace blink {
PerformanceLongTaskTiming::PerformanceLongTaskTiming(double startTime,
double endTime,
String name,
- DOMWindow* culpritWindow)
+ String culpritFrameSrc,
+ String culpritFrameId,
+ String culpritFrameName)
: PerformanceEntry(name, "longtask", startTime, endTime),
- m_culpritWindow(*culpritWindow) {}
+ m_culpritFrameSrc(culpritFrameSrc),
+ m_culpritFrameId(culpritFrameId),
+ m_culpritFrameName(culpritFrameName) {}
PerformanceLongTaskTiming::~PerformanceLongTaskTiming() {}
-DOMWindow* PerformanceLongTaskTiming::culpritWindow() const {
- return m_culpritWindow.get();
+String PerformanceLongTaskTiming::culpritFrameSrc() const {
+ return m_culpritFrameSrc;
+}
+
+String PerformanceLongTaskTiming::culpritFrameId() const {
+ return m_culpritFrameId;
+}
+
+String PerformanceLongTaskTiming::culpritFrameName() const {
+ return m_culpritFrameName;
}
DEFINE_TRACE(PerformanceLongTaskTiming) {
- visitor->trace(m_culpritWindow);
PerformanceEntry::trace(visitor);
}

Powered by Google App Engine
This is Rietveld 408576698