Create TaskAttributionTiming PerformanceEntry for attribution in PerformanceLongTaskTiming. Move cu…

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 "core/frame/DOMWindow.h"
+#include "core/timing/TaskAttributionTiming.h"
 
 namespace blink {
 
 namespace {
 
 double clampToMillisecond(double timeInMillis) {
   // Long task times are clamped to 1 millisecond for security.
   return floor(timeInMillis);
 }
 
 }  // namespace
 
+// static
+PerformanceLongTaskTiming* PerformanceLongTaskTiming::create(
+    double startTime,
+    double endTime,
+    String name,
+    String culpritFrameSrc,
+    String culpritFrameId,
+    String culpritFrameName) {

[tdresser, 2017/01/05 13:34:14]

I'd strip the culprit prefix throughout - I don't think it adds anything, and
currently the naming is inconsistent.

[panicker, 2017/01/05 20:56:49]

Done.

+  return new PerformanceLongTaskTiming(startTime, endTime, name,
+                                       culpritFrameSrc, culpritFrameId,
+                                       culpritFrameName);
+}
+
 PerformanceLongTaskTiming::PerformanceLongTaskTiming(double startTime,
                                                      double endTime,
                                                      String name,
                                                      String culpritFrameSrc,
                                                      String culpritFrameId,
                                                      String culpritFrameName)
     : PerformanceEntry(name,
                        "longtask",
                        clampToMillisecond(startTime),
-                       clampToMillisecond(endTime)),
-      m_culpritFrameSrc(culpritFrameSrc),
-      m_culpritFrameId(culpritFrameId),
-      m_culpritFrameName(culpritFrameName) {}
+                       clampToMillisecond(endTime)) {
+  // Only one possible name ("frame") currently.
+  TaskAttributionTiming* attributionEntry = TaskAttributionTiming::create(
+      "frame", culpritFrameSrc, culpritFrameId, culpritFrameName);
+  m_attribution.append(*attributionEntry);
+}
 
 PerformanceLongTaskTiming::~PerformanceLongTaskTiming() {}
 
-String PerformanceLongTaskTiming::culpritFrameSrc() const {
-  return m_culpritFrameSrc;
-}
-
-String PerformanceLongTaskTiming::culpritFrameId() const {
-  return m_culpritFrameId;
-}
-
-String PerformanceLongTaskTiming::culpritFrameName() const {
-  return m_culpritFrameName;
+TaskAttributionVector PerformanceLongTaskTiming::attribution() const {
+  return m_attribution;
 }
 
 DEFINE_TRACE(PerformanceLongTaskTiming) {
+  visitor->trace(m_attribution);
   PerformanceEntry::trace(visitor);
 }
 
 }  // namespace blink