Pass collected frame timing events to Blink

content/renderer/gpu/render_widget_compositor.cc

 // Copyright (c) 2013 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 "content/renderer/gpu/render_widget_compositor.h"
 
 #include <limits>
 #include <string>
 
 #include "base/command_line.h"
@@ skipping 936 matching lines @@
 }
 
 void RenderWidgetCompositor::DidPostSwapBuffers() {
   widget_->OnSwapBuffersPosted();
 }
 
 void RenderWidgetCompositor::DidAbortSwapBuffers() {
   widget_->OnSwapBuffersAborted();
 }
 
+namespace {
+
+blink::WebFrameTimingEvent CompositeTimingEventToDouble(
+    const cc::FrameTimingTracker::CompositeTimingEvent& event) {
+  return blink::WebFrameTimingEvent(event.frame_id,
+      (event.timestamp - base::TimeTicks()).InSecondsF());
+}
+
+blink::WebFrameTimingEvent MainFrameTimingEventToDouble(
+    const cc::FrameTimingTracker::MainFrameTimingEvent& event) {
+  return blink::WebFrameTimingEvent(event.frame_id,
+      (event.timestamp - base::TimeTicks()).InSecondsF(),
+      (event.end_time - base::TimeTicks()).InSecondsF());
+}
+
+} // namespace
+
+void RenderWidgetCompositor::RecordFrameTimingEvents(
+    scoped_ptr<cc::FrameTimingTracker::CompositeTimingSet> composite_events,
+    scoped_ptr<cc::FrameTimingTracker::MainFrameTimingSet> main_frame_events) {
+  for (const auto& composite_event : *composite_events ) {
+    int64_t frameId = composite_event.first;
+    const std::vector<cc::FrameTimingTracker::CompositeTimingEvent>& events =
+        composite_event.second;
+    std::vector<blink::WebFrameTimingEvent> webEvents(events.size());
+    std::transform(events.begin(), events.end(), webEvents.begin(),
+        CompositeTimingEventToDouble);

[vmpstr, 2015/06/02 17:40:08]

I kind of weakly prefer an explicit loop here, but if you're going to keep this,
maybe use a lambda instead of CompositeTimingEventToDouble (and similar comment
for the second loop)

[MikeB, 2015/06/02 21:59:36]

Done.

+    widget_->webwidget()->recordFrameTimingEvent(
+        blink::WebWidget::CompositeEvent, frameId, webEvents);
+  }
+  for (const auto& main_frame_event : *main_frame_events ) {
+    int64_t frameId = main_frame_event.first;
+    const std::vector<cc::FrameTimingTracker::MainFrameTimingEvent>& events =
+        main_frame_event.second;
+    std::vector<blink::WebFrameTimingEvent> webEvents(events.size());
+    std::transform(events.begin(), events.end(), webEvents.begin(),
+                   MainFrameTimingEventToDouble);
+    widget_->webwidget()->recordFrameTimingEvent(
+        blink::WebWidget::RenderEvent, frameId, webEvents);
+  }
+}
+
 void RenderWidgetCompositor::RateLimitSharedMainThreadContext() {
   cc::ContextProvider* provider =
       compositor_deps_->GetSharedMainThreadContextProvider();
   // provider can be NULL after the GPU process crashed enough times and we
   // don't want to restart it any more (falling back to software).
   if (!provider)
     return;
   provider->ContextGL()->RateLimitOffscreenContextCHROMIUM();
 }
 
 void RenderWidgetCompositor::SetSurfaceIdNamespace(
     uint32_t surface_id_namespace) {
   layer_tree_host_->set_surface_id_namespace(surface_id_namespace);
 }
 
 }  // namespace content