Pass DOMWinow to PerformanceLongTaskTiming for attribution

third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/timing/PerformanceLongTaskTiming.h"
 
+#include "bindings/core/v8/V8ObjectBuilder.h"
+#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();
+}
+
+void PerformanceLongTaskTiming::buildJSONValue(V8ObjectBuilder& builder) const {
+  PerformanceEntry::buildJSONValue(builder);
+  builder.add("culpritWindow", culpritWindow());

[caseq, 2016/10/05 00:22:18]

This looks suspicious to me -- we can't really serialize the window as a JSON.
Let's just remove this?

[panicker, 2016/10/05 01:02:18]

Done.

+}
+
 DEFINE_TRACE(PerformanceLongTaskTiming) {
+  visitor->trace(m_culpritWindow);
   PerformanceEntry::trace(visitor);
 }
 
 }  // namespace blink