Change glimage to accept a type.

gpu/command_buffer/service/gpu_control_service.h

Index: gpu/command_buffer/service/gpu_control_service.h
diff --git a/gpu/command_buffer/service/gpu_control_service.h b/gpu/command_buffer/service/gpu_control_service.h
index 74fd4d5a16a4081e349d264f8a6903251de22938..c153c893c54141d8e074ba6e9a2daae4485ee2bb 100644
--- a/gpu/command_buffer/service/gpu_control_service.h
+++ b/gpu/command_buffer/service/gpu_control_service.h
@@ -32,11 +32,11 @@ class GPU_EXPORT GpuControlService : public GpuControl {
 
   // GpuControl implementation.
   virtual gpu::Capabilities GetCapabilities() OVERRIDE;
-  virtual gfx::GpuMemoryBuffer* CreateGpuMemoryBuffer(
-      size_t width,
-      size_t height,
-      unsigned internalformat,
-      int32* id) OVERRIDE;
+  virtual gfx::GpuMemoryBuffer* CreateGpuMemoryBuffer(size_t width,
+                                                      size_t height,
+                                                      unsigned internalformat,
+                                                      unsigned usage,
+                                                      int32* id) OVERRIDE;
   virtual void DestroyGpuMemoryBuffer(int32 id) OVERRIDE;
   virtual uint32 InsertSyncPoint() OVERRIDE;
   virtual void SignalSyncPoint(uint32 sync_point,
@@ -55,7 +55,8 @@ class GPU_EXPORT GpuControlService : public GpuControl {
                                gfx::GpuMemoryBufferHandle buffer,
                                size_t width,
                                size_t height,
-                               unsigned internalformat);
+                               unsigned internalformat,
+                               unsigned usage);

[reveman, 2014/05/01 12:06:38]

Does the GPU service really need to know about usage? We can't rely on this to
determine the actual usage that is allowed for a buffer as a malicious renderer
can easily manufacture this.

[alexst (slow to review), 2014/05/01 13:44:51]

I suppose it's not strictly necessary,  I was going to use this to reject
buffers not meant for scanout but set as overlay early to produce a better error
message in command buffer. If the usage is malicious here, it'll get rejected
eventually anyway in the lower level ozone bits because they know what the
actual buffer is like. I can remove this if you prefer.

[reveman, 2014/05/01 13:48:46]

Let's avoid it for now and we can add it later if it turns out to be useful. 

 
  private:
   GpuMemoryBufferManagerInterface* gpu_memory_buffer_manager_;