Send vsync notification from browser to renderer

content/browser/renderer_host/render_widget_host_impl.cc

Index: content/browser/renderer_host/render_widget_host_impl.cc
diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc
index 33ae4ecb55bd4abcf54654e83395cdefa9977d7c..d37e9658186bb274c359be3bfc5931260a16f47a 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -364,6 +364,8 @@ bool RenderWidgetHostImpl::OnMessageReceived(const IPC::Message &msg) {
                         OnCompositorSurfaceBuffersSwapped)
     IPC_MESSAGE_HANDLER_GENERIC(ViewHostMsg_SwapCompositorFrame,
                                 msg_is_ok = OnSwapCompositorFrame(msg))
+    IPC_MESSAGE_HANDLER(ViewHostMsg_SetVSyncNotificationEnabled,
+                        OnSetVSyncNotificationEnabled)
     IPC_MESSAGE_HANDLER(ViewHostMsg_UpdateRect, OnUpdateRect)
     IPC_MESSAGE_HANDLER(ViewHostMsg_UpdateIsDelayed, OnUpdateIsDelayed)
     IPC_MESSAGE_HANDLER(ViewHostMsg_HandleInputEvent_ACK, OnInputEventAck)
@@ -1294,6 +1296,10 @@ void RenderWidgetHostImpl::UpdateVSyncParameters(base::TimeTicks timebase,
   Send(new ViewMsg_UpdateVSyncParameters(GetRoutingID(), timebase, interval));
 }
 
+void RenderWidgetHostImpl::SendVSync(base::TimeTicks frame_time) {
+  Send(new ViewMsg_DidVSync(GetRoutingID(), frame_time));
+}
+
 void RenderWidgetHostImpl::RendererExited(base::TerminationStatus status,
                                           int exit_code) {
   // Clearing this flag causes us to re-create the renderer when recovering
@@ -1625,6 +1631,13 @@ bool RenderWidgetHostImpl::OnSwapCompositorFrame(
   return true;
 }
 
+void RenderWidgetHostImpl::OnSetVSyncNotificationEnabled(bool enabled) {
+  TRACE_EVENT1("renderer_host",
+      "RenderWidgetHostImpl::OnSetVSyncNotificationEnabled",
+      "enabled", enabled);

[jam, 2013/03/27 16:13:06]

does this need to be traced? i.e. is this something that others care to see?

I'm a bit concerned that all these internals of code in content are traced, and
there are tests that look for specific trace events. i worry that it's a
replacement of something like notification service, which we're trying to remove
from content

[Sami, 2013/03/27 17:38:06]

This particular trace was for human eyes only (i.e., debugging) and not really
that necessary in the end, so I've removed it.

+  view_->SetVSyncNotificationEnabled(enabled);
+}
+
 void RenderWidgetHostImpl::OnUpdateRect(
     const ViewHostMsg_UpdateRect_Params& params) {
   TRACE_EVENT0("renderer_host", "RenderWidgetHostImpl::OnUpdateRect");