Pass more frame timing information to Blink

content/renderer/gpu/render_widget_compositor.cc

Index: content/renderer/gpu/render_widget_compositor.cc
diff --git a/content/renderer/gpu/render_widget_compositor.cc b/content/renderer/gpu/render_widget_compositor.cc
index b0931771d4afc8706a6eac1a17a5451f2cf13c6c..35dee073e582be9588a5239626f42e09bda6ec8b 100644
--- a/content/renderer/gpu/render_widget_compositor.cc
+++ b/content/renderer/gpu/render_widget_compositor.cc
@@ -759,8 +759,11 @@ void RenderWidgetCompositor::BeginMainFrame(const cc::BeginFrameArgs& args) {
   VLOG(2) << "RenderWidgetCompositor::BeginMainFrame";
   begin_main_frame_time_ = args.frame_time;
   begin_main_frame_interval_ = args.interval;
-  double frame_time = (args.frame_time - base::TimeTicks()).InSecondsF();
-  WebBeginFrameArgs web_begin_frame_args = WebBeginFrameArgs(frame_time);
+  double frame_time_sec = (args.frame_time - base::TimeTicks()).InSecondsF();
+  double deadline_sec = (args.deadline - base::TimeTicks()).InSecondsF();
+  double interval_sec = args.interval.InSecondsF();

[picksi1, 2014/09/02 14:50:40]

Is base::TimeTicks() potentially slow? Should we call it once and cache the
result locally?

[Sami, 2014/09/02 14:53:30]

It only constructs a TimeTicks instance that holds the value zero. To actually
get the current time you need to call base::TimeTicks::Now().

+  WebBeginFrameArgs web_begin_frame_args =
+      WebBeginFrameArgs(frame_time_sec, deadline_sec, interval_sec);
   widget_->webwidget()->beginFrame(web_begin_frame_args);
 }
 
@@ -801,6 +804,7 @@ void RenderWidgetCompositor::DidCommit() {
 
   widget_->DidCommitCompositorFrame();
   widget_->didBecomeReadyForAdditionalInput();
+  widget_->webwidget()->didCommitFrameToCompositor();
 }
 
 void RenderWidgetCompositor::DidCommitAndDrawFrame() {