Splitting GpuMemoryAllocation into two parts: for renderer and browser

content/common/gpu/gpu_messages.h

Index: content/common/gpu/gpu_messages.h
diff --git a/content/common/gpu/gpu_messages.h b/content/common/gpu/gpu_messages.h
index 04278dd2d67642c6e82063166053e077dd69b150..3d84471530df18ac14854eedef50813c5abe4f2b 100644
--- a/content/common/gpu/gpu_messages.h
+++ b/content/common/gpu/gpu_messages.h
@@ -119,10 +119,9 @@ IPC_STRUCT_TRAITS_BEGIN(content::GPUInfo)
 #endif
 IPC_STRUCT_TRAITS_END()
 
-IPC_STRUCT_TRAITS_BEGIN(GpuMemoryAllocation)
+IPC_STRUCT_TRAITS_BEGIN(GpuMemoryAllocationForRenderer)
   IPC_STRUCT_TRAITS_MEMBER(gpu_resource_size_in_bytes)
-  IPC_STRUCT_TRAITS_MEMBER(has_frontbuffer)
-  IPC_STRUCT_TRAITS_MEMBER(has_backbuffer)
+  IPC_STRUCT_TRAITS_MEMBER(suggest_have_backbuffer)
 IPC_STRUCT_TRAITS_END()
 
 IPC_STRUCT_TRAITS_BEGIN(gfx::GLSurfaceHandle)
@@ -441,9 +440,13 @@ IPC_MESSAGE_ROUTED0(GpuCommandBufferMsg_EchoAck)
 // Send to stub on surface visibility change.
 IPC_MESSAGE_ROUTED1(GpuCommandBufferMsg_SetSurfaceVisible, bool /* visible */)
 
+// Request that the GPU process create/destroy the surface backbuffer.
+IPC_MESSAGE_ROUTED0(GpuCommandBufferMsg_CreateBackbuffer)

[piman, 2012/03/15 20:14:32]

Doing these as commands into the command buffer (like SetVisibility is done now)
would make it easier to track state in the GLES2CmdDecoder, as to either
generate errors in case of misuse or to implicitly recreate the back buffer on
draw/clear/readback commands.

Similarly, client-side, making it go through the GLES2Implementation would let
you add asserts.

[mmocny, 2012/03/15 20:39:27]

Okay, I was undecided here, and I think you are correct.  It may make it harder
for me to temporarily hook dropping frontbuffers onto dropping backbuffers, but
that needs decoupling anyway.

On 2012/03/15 20:14:32, piman wrote:

+IPC_MESSAGE_ROUTED0(GpuCommandBufferMsg_DestroyBackbuffer)
+
 // Sent to proxy when the gpu memory manager changes its memory allocation.
 IPC_MESSAGE_ROUTED1(GpuCommandBufferMsg_SetMemoryAllocation,
-                    GpuMemoryAllocation /* allocation */)
+                    GpuMemoryAllocationForRenderer /* allocation */)
 
 //------------------------------------------------------------------------------
 // Accelerated Video Decoder Messages