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

content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc

Index: content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc
diff --git a/content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc b/content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc
index 4def872429965a980b8e30178450580ab1ca312d..cd9f1f5658e7db2bb732fd8e063bfd2da57f1773 100644
--- a/content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc
+++ b/content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc
@@ -24,6 +24,7 @@
 #include "base/metrics/histogram.h"
 #include "base/synchronization/lock.h"
 #include "content/common/content_switches.h"
+#include "content/renderer/gpu/command_buffer_proxy.h"
 #include "content/renderer/gpu/gpu_channel_host.h"
 #include "content/renderer/render_thread_impl.h"
 #include "content/renderer/render_view_impl.h"
@@ -268,6 +269,12 @@ void WebGraphicsContext3DCommandBufferImpl::reshape(int width, int height) {
 #endif  // FLIP_FRAMEBUFFER_VERTICALLY
 }
 
+void WebGraphicsContext3DCommandBufferImpl::setVisibility(bool visible) {
+  gl_->Flush();
+  // TODO(mmocny): Should context_ have a SetSurfaceVisible which forwards?
+  context_->GetCommandBufferProxy()->SetSurfaceVisible(visible);

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

RE TODO: I'm leaning towards having RendererGLContext have a forwarding
function, and will change this unless advised otherwise.

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

I'm OK with it as is. There's one surface per command buffer stub.

[apatrick_chromium, 2011/10/13 19:51:31]

I lean towards a forwarding function. Law of demeter. I think there is only one
caller of that function at the moment and I'm getting closer to being able to
remove it.

+}
+
 #ifdef FLIP_FRAMEBUFFER_VERTICALLY
 void WebGraphicsContext3DCommandBufferImpl::FlipVertically(
     uint8* framebuffer,