gpu: Refactor GpuMemoryBuffer framework for multi-process support.

gpu/command_buffer/service/gpu_scheduler.h

 // Copyright (c) 2012 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_SERVICE_GPU_SCHEDULER_H_
 #define GPU_COMMAND_BUFFER_SERVICE_GPU_SCHEDULER_H_
 
 #include <queue>
 
 #include "base/atomic_ref_count.h"
@@ skipping 70 matching lines @@
   // Sets a callback that is invoked just before scheduler is rescheduled
   // or descheduled. Takes ownership of callback object.
   void SetSchedulingChangedCallback(const SchedulingChangedCallback& callback);
 
   // Implementation of CommandBufferEngine.
   virtual Buffer GetSharedMemoryBuffer(int32 shm_id) OVERRIDE;
   virtual void set_token(int32 token) OVERRIDE;
   virtual bool SetGetBuffer(int32 transfer_buffer_id) OVERRIDE;
   virtual bool SetGetOffset(int32 offset) OVERRIDE;
   virtual int32 GetGetOffset() OVERRIDE;
+  virtual gfx::GLImage* GetImage(int32 image_id) OVERRIDE;

[no sievers, 2013/08/01 00:38:04]

I find plumbing the ImageManager through the Scheduler a little odd.
Maybe the decoder should just keep its own pointer?

Would also make some changes to the unittests and other files unnecessary here.

[reveman, 2013/08/01 13:32:18]

I'd rather not add new state unless necessary. Current patch is consistent with
how transfer buffers and shared memory is handled.

[piman, 2013/08/01 21:19:30]

Transfer buffers and shared memory are an essential part of command buffers.
This is GL-specific and I think it should be kept out of the command processing
layer.

[reveman, 2013/08/08 23:19:00]

Remove this.

 
   void SetCommandProcessedCallback(const base::Closure& callback);
 
   void DeferToFence(base::Closure task);
 
   // Polls the fences, invoking callbacks that were waiting to be triggered
   // by them and returns whether all fences were complete.
   bool PollUnscheduleFences();
 
   bool HasMoreIdleWork();
@@ skipping 58 matching lines @@
   // If non-NULL and |preemption_flag_->IsSet()|, exit PutChanged early.
   scoped_refptr<PreemptionFlag> preemption_flag_;
   bool was_preempted_;
 
   DISALLOW_COPY_AND_ASSIGN(GpuScheduler);
 };
 
 }  // namespace gpu
 
 #endif  // GPU_COMMAND_BUFFER_SERVICE_GPU_SCHEDULER_H_