Command to mark surface inactive, so gpu process can release resources.

webkit/gpu/webgraphicscontext3d_in_process_impl.h

Index: webkit/gpu/webgraphicscontext3d_in_process_impl.h
diff --git a/webkit/gpu/webgraphicscontext3d_in_process_impl.h b/webkit/gpu/webgraphicscontext3d_in_process_impl.h
index 1bc447fa205648e78c1af6faa8383a85fa21e95f..86cbcc694e551d6cacacffe24bb742dd9b9b4dd2 100644
--- a/webkit/gpu/webgraphicscontext3d_in_process_impl.h
+++ b/webkit/gpu/webgraphicscontext3d_in_process_impl.h
@@ -72,6 +72,8 @@ class WebGraphicsContext3DInProcessImpl : public WebGraphicsContext3D {
 
   virtual void reshape(int width, int height);
 
+  virtual void setVisibility(bool visible);

[mmocny, 2011/10/13 18:54:17]

I was hesitant to add this here, as I am not sure if maybe GLSurface should have
limited functionality (imitating an actual gl surface)?

However, due to the multiple inheritance structure of the various
ImageTransportSurface implementations, the cleanest solution by far was to
migrate this function from ImageTransportSurface to GLSurface (here).
(Background: GpuCommandBufferStub owns a GLSurface surface_, and
ImageTransportSurface does not extend GLSurface, nor could it be made to)

[jonathan.backer, 2011/10/13 19:25:07]

I think that this comment is in the wrong place. I think that you're referring
to adding another method to GLSurface. And yes, that class is becoming more than
a thin wrapper around a GL surface. I think that the alternatives are uglier
though.

+
   virtual bool readBackFramebuffer(unsigned char* pixels, size_t bufferSize);
   virtual bool readBackFramebuffer(unsigned char* pixels, size_t buffer_size,
                                    WebGLId framebuffer, int width, int height);