Move Insert/SignalSyncPoint() to GpuControl

gpu/command_buffer/common/gpu_control.h

 // Copyright 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.
 
 #ifndef GPU_COMMAND_BUFFER_COMMON_GPU_CONTROL_H_
 #define GPU_COMMAND_BUFFER_COMMON_GPU_CONTROL_H_
 
+#include "base/callback.h"
 #include "gpu/command_buffer/common/types.h"
 #include "gpu/gpu_export.h"
 
 namespace gfx {
 class GpuMemoryBuffer;
 }
 
 namespace gpu {
 
 // Common interface for GpuControl implementations.
 class GPU_EXPORT GpuControl {
  public:
   GpuControl() {}
   virtual ~GpuControl() {}
 
   virtual bool SupportsGpuMemoryBuffer() = 0;
 
   // Create a gpu memory buffer of the given dimensions and format. Returns
   // its ID or -1 on error.
   virtual gfx::GpuMemoryBuffer* CreateGpuMemoryBuffer(
       size_t width,
       size_t height,
       unsigned internalformat,
       int32* id) = 0;
 
   // Destroy a gpu memory buffer. The ID must be positive.
   virtual void DestroyGpuMemoryBuffer(int32 id) = 0;
 
+  // Inserts a sync point, returning its ID. Sync point IDs are global and can
+  // be used for cross-context synchronization.
+  virtual uint32 InsertSyncPoint() = 0;
+
+  // Runs |callback| when a sync point is reached or the context is destroyed.

[no sievers, 2013/09/27 18:13:04]

Does it really currently really run it when the context is destroyed?

For context lost, there is actually a test that defines the current behavior:
GLRendererTestSyncPoint.SignalSyncPointOnLostContext, see '// The sync point
shouldn't have happened since the context was lost.'

Although that check if before the context is destroyed.

+  virtual void SignalSyncPoint(uint32 sync_point,
+                               const base::Closure& callback) = 0;
+
  private:
   DISALLOW_COPY_AND_ASSIGN(GpuControl);
 };
 
 }  // namespace gpu
 
 #endif  // GPU_COMMAND_BUFFER_COMMON_GPU_CONTROL_H_