Send vsync notification from browser to renderer

content/renderer/gpu/compositor_output_surface.cc

Index: content/renderer/gpu/compositor_output_surface.cc
diff --git a/content/renderer/gpu/compositor_output_surface.cc b/content/renderer/gpu/compositor_output_surface.cc
index cc55ffa884adf3be780559afa7a34e574381b9f3..7178af7ec3933267e3555bc4787da8388ca59577 100644
--- a/content/renderer/gpu/compositor_output_surface.cc
+++ b/content/renderer/gpu/compositor_output_surface.cc
@@ -42,6 +42,7 @@ IPC::ForwardingMessageFilter* CompositorOutputSurface::CreateFilter(
 {
   uint32 messages_to_filter[] = {
     ViewMsg_UpdateVSyncParameters::ID,
+    ViewMsg_DidVSync::ID,
     ViewMsg_SwapCompositorFrameAck::ID
   };
 
@@ -106,6 +107,7 @@ void CompositorOutputSurface::OnMessageReceived(const IPC::Message& message) {
     return;
   IPC_BEGIN_MESSAGE_MAP(CompositorOutputSurface, message)
     IPC_MESSAGE_HANDLER(ViewMsg_UpdateVSyncParameters, OnUpdateVSyncParameters);
+    IPC_MESSAGE_HANDLER(ViewMsg_DidVSync, DidVSync);
     IPC_MESSAGE_HANDLER(ViewMsg_SwapCompositorFrameAck, OnSwapAck);
   IPC_END_MESSAGE_MAP()
 }
@@ -117,6 +119,18 @@ void CompositorOutputSurface::OnUpdateVSyncParameters(
   client_->OnVSyncParametersChanged(timebase, interval);
 }
 
+void CompositorOutputSurface::EnableVSyncNotification(bool enable) {
+  DCHECK(CalledOnValidThread());
+  DCHECK(client_);

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

nit: this dcheck, and the similar one below, is redundant. in release builds, it
does nothing. in debug builds, the next line would crash which is just as strong
of a signal to the developer that there's a null dereference

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

Good point, I'll remove this and the other similar checks in this file. This
specific function didn't even use client_ so the check was doubly redundant.

+  Send(new ViewHostMsg_SetVSyncNotificationEnabled(routing_id_, enable));
+}
+
+void CompositorOutputSurface::DidVSync(base::TimeTicks frame_time) {
+  DCHECK(CalledOnValidThread());
+  DCHECK(client_);
+  client_->DidVSync(frame_time);
+}
+
 void CompositorOutputSurface::OnSwapAck(const cc::CompositorFrameAck& ack) {
   client_->OnSendFrameToParentCompositorAck(ack);
 }