cc: Move scheduling logic out of OutputSurface

cc/output/output_surface_client.h

Index: cc/output/output_surface_client.h
diff --git a/cc/output/output_surface_client.h b/cc/output/output_surface_client.h
index 08d9b624884ee74d73dc021ec16dc57774057aea..485a7ba40eba708d07081619db4962dcf7798903 100644
--- a/cc/output/output_surface_client.h
+++ b/cc/output/output_surface_client.h
@@ -30,8 +30,13 @@ class CC_EXPORT OutputSurfaceClient {
   virtual bool DeferredInitialize(
       scoped_refptr<ContextProvider> offscreen_context_provider) = 0;
   virtual void ReleaseGL() = 0;
+  virtual void CommitVSyncParameters(base::TimeTicks timebase,
+                                     base::TimeDelta interval) = 0;
   virtual void SetNeedsRedrawRect(const gfx::Rect& damage_rect) = 0;
   virtual void BeginFrame(const BeginFrameArgs& args) = 0;
+  // DidSwapBuffers and OnSwapBuffersComplete should not be called
+  // when the OutputSurface is lost to avoid race conditions in the scheduler
+  // if OnSwapBuffersComplete arrives for the previous OutputSurface.

[Sami, 2014/04/08 13:46:01]

It feels like this comment belongs to OutputSurface::SwapBuffers() or elsewhere
in the implementation, because this isn't something the client has to worry
about. Also, replace "scheduler" with "client" to keep this more generic.

[brianderson, 2014/04/09 02:52:05]

Sounds good.

[brianderson, 2014/04/10 23:45:58]

Done.

   virtual void DidSwapBuffers() = 0;
   virtual void OnSwapBuffersComplete() = 0;
   virtual void ReclaimResources(const CompositorFrameAck* ack) = 0;