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 866 matching lines @@
 void RenderWidgetCompositor::ApplyViewportDeltas(
     const gfx::Vector2d& scroll_delta,
     float page_scale,
     float top_controls_delta) {
   widget_->webwidget()->applyViewportDeltas(
       scroll_delta,
       page_scale,
       top_controls_delta);
 }
 
+static std::tuple<int, double, double> CompositeTimingEventToDouble(

[vmpstr, 2015/05/12 23:48:26]

I don't think you can use a tuple unless we have full c++11 support. You can use
a pair<int, pair<double, double>> or maybe a struct or something? Also, prefer
to put free functions like that in anonymous namespace instead of making them
static.

[MikeB, 2015/05/21 21:53:08]

Done.

+    const cc::FrameTimingTracker::CompositeTimingEvent& event) {
+  return std::make_tuple(event.frame_id,
+                        (event.timestamp - base::TimeTicks()).InSecondsF(),
+                        0);
+}
+
+void RenderWidgetCompositor::RecordCompositeTimings(
+    cc::FrameTimingTracker::CompositeTimingSet* composites) {
+  for (const auto& record : *composites ) {

[vmpstr, 2015/05/12 23:48:26]

Can you give names (variables) to record.first and record.second so it's clear
what they are?

[MikeB, 2015/05/21 21:53:08]

Done.

+    std::vector<std::tuple<int, double, double>>
+        compositeTiming(record.second.size());
+    std::transform(record.second.begin(), record.second.end(),
+                   compositeTiming.begin(), CompositeTimingEventToDouble);
+    widget_->webwidget()->recordFrameTimingEvent(
+        blink::WebWidget::CompositeEvent, record.first, compositeTiming);
+  }
+}
+
+static std::tuple<int, double, double> MainFrameTimingEventToDouble(
+    const cc::FrameTimingTracker::MainFrameTimingEvent& event) {
+  return std::make_tuple(event.frame_id,
+                         (event.timestamp - base::TimeTicks()).InSecondsF(),
+                         (event.end_time - base::TimeTicks()).InSecondsF());
+}
+
+void RenderWidgetCompositor::RecordRenderTimings(
+    cc::FrameTimingTracker::MainFrameTimingSet* renders) {
+  for (const auto& record : *renders ) {
+    std::vector<std::tuple<int, double, double>>
+        renderTiming(record.second.size());
+    std::transform(record.second.begin(), record.second.end(),
+                   renderTiming.begin(), MainFrameTimingEventToDouble);
+    widget_->webwidget()->recordFrameTimingEvent(
+        blink::WebWidget::RenderEvent, record.first, renderTiming);
+  }
+}
+
 void RenderWidgetCompositor::RequestNewOutputSurface() {
   // If the host is closing, then no more compositing is possible.  This
   // prevents shutdown races between handling the close message and
   // the CreateOutputSurface task.
   if (widget_->host_closing())
     return;
 
   // TODO(robliao): Remove ScopedTracker below once https://crbug.com/466870
   // is fixed.
   tracked_objects::ScopedTracker tracking_profile(
@@ skipping 73 matching lines @@
   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();
 }
 
 }  // namespace content