Move Insert/SignalSyncPoint() to GpuControl

gpu/command_buffer/service/in_process_command_buffer.cc

Index: gpu/command_buffer/service/in_process_command_buffer.cc
diff --git a/gpu/command_buffer/service/in_process_command_buffer.cc b/gpu/command_buffer/service/in_process_command_buffer.cc
index 778ca7d9037feaf8ee546a0876b423489938ef07..38c923cfd1c202dda0d5dcd1f4181fcefb13e898 100644
--- a/gpu/command_buffer/service/in_process_command_buffer.cc
+++ b/gpu/command_buffer/service/in_process_command_buffer.cc
@@ -639,16 +639,6 @@ gpu::Buffer InProcessCommandBuffer::GetTransferBuffer(int32 id) {
   return gpu::Buffer();
 }
 
-uint32 InProcessCommandBuffer::InsertSyncPoint() {
-  NOTREACHED();
-  return 0;
-}
-void InProcessCommandBuffer::SignalSyncPoint(unsigned sync_point,
-                                             const base::Closure& callback) {
-  CheckSequencedThread();
-  QueueTask(WrapCallback(callback));
-}
-
 bool InProcessCommandBuffer::SupportsGpuMemoryBuffer() {
   return supports_gpu_memory_buffer_;
 }
@@ -675,6 +665,17 @@ void InProcessCommandBuffer::DestroyGpuMemoryBuffer(int32 id) {
   QueueTask(task);
 }
 
+uint32 InProcessCommandBuffer::InsertSyncPoint() {
+  NOTREACHED();

[piman, 2013/09/27 01:35:47]

If you want to merge the WGC3D implementations and have them delegate to the
GLES2Implementation, this would be reached (though it can still return 0).

It'd be nice to do with this patch, I think, just fixing
WebGraphicsContext3DInProcessCommandBufferImpl::insertSyncPoint to
DELEGATE_TO_GL instead.

+  return 0;
+}
+
+void InProcessCommandBuffer::SignalSyncPoint(unsigned sync_point,
+                                             const base::Closure& callback) {
+  CheckSequencedThread();
+  QueueTask(WrapCallback(callback));
+}
+
 gpu::error::Error InProcessCommandBuffer::GetLastError() {
   CheckSequencedThread();
   return last_state_.error;