Drop frontbuffers with ui-use-gpu-process, synchronized with browser, decoupled from backbuffer dro…

content/common/gpu/image_transport_surface.h

Index: content/common/gpu/image_transport_surface.h
diff --git a/content/common/gpu/image_transport_surface.h b/content/common/gpu/image_transport_surface.h
index 307a11ea3a71d86b451873261fc8050363998673..3cec9bc4dce183ee9bdd7274d0d6ab1a9a2812e1 100644
--- a/content/common/gpu/image_transport_surface.h
+++ b/content/common/gpu/image_transport_surface.h
@@ -30,6 +30,7 @@ struct GpuHostMsg_AcceleratedSurfaceNew_Params;
 struct GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params;
 struct GpuHostMsg_AcceleratedSurfacePostSubBuffer_Params;
 struct GpuHostMsg_AcceleratedSurfaceRelease_Params;
+struct GpuHostMsg_AcceleratedSurfaceRequestReleaseFront_Params;
 
 namespace gfx {
 class GLSurface;
@@ -67,6 +68,10 @@ class ImageTransportSurface {
   virtual void OnPostSubBufferACK() = 0;
   virtual void OnResizeViewACK() = 0;
   virtual void OnResize(gfx::Size size) = 0;
+  virtual void OnSetFrontSurfaceIsProtected(bool is_protected) = 0;
+  virtual void OnRequestReleaseFrontACK(int request_id,
+                                        int retry_count,
+                                        bool was_released) { };

[mmocny, 2012/06/01 21:06:51]

Is this default implementation alright here?  Its currently inconsistent with
the IsProtected implementation (which is also a noop on the other platforms), so
I guess I should change one or the other.

[mmocny, 2012/06/05 14:52:35]

Done.

 
   // Creates the appropriate surface depending on the GL implementation.
   static scoped_refptr<gfx::GLSurface>
@@ -109,6 +114,8 @@ class ImageTransportHelper : public IPC::Channel::Listener {
       GpuHostMsg_AcceleratedSurfacePostSubBuffer_Params params);
   void SendAcceleratedSurfaceRelease(
       GpuHostMsg_AcceleratedSurfaceRelease_Params params);
+  void SendAcceleratedSurfaceRequestReleaseFront(
+      GpuHostMsg_AcceleratedSurfaceRequestReleaseFront_Params params);
   void SendResizeView(const gfx::Size& size);
 
   // Whether or not we should execute more commands.
@@ -133,6 +140,10 @@ class ImageTransportHelper : public IPC::Channel::Listener {
   void OnBuffersSwappedACK();
   void OnPostSubBufferACK();
   void OnResizeViewACK();
+  void OnSetFrontSurfaceIsProtected(bool is_protected);
+  void OnRequestReleaseFrontACK(int request_id,
+                                int retry_count,
+                                bool was_released);
 
   // Backbuffer resize callback.
   void Resize(gfx::Size size);
@@ -173,6 +184,10 @@ class PassThroughImageTransportSurface
   virtual void OnPostSubBufferACK() OVERRIDE;
   virtual void OnResizeViewACK() OVERRIDE;
   virtual void OnResize(gfx::Size size) OVERRIDE;
+  virtual void OnSetFrontSurfaceIsProtected(bool is_protected) OVERRIDE;
+  virtual void OnRequestReleaseFrontACK(int request_id,
+                                        int retry_count,
+                                        bool was_released) OVERRIDE;
 
  protected:
   virtual ~PassThroughImageTransportSurface();