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

content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #if defined(ENABLE_GPU)
 
 #include "content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.h"
 
 #include "gpu/GLES2/gl2.h"
 #ifndef GL_GLEXT_PROTOTYPES
 #define GL_GLEXT_PROTOTYPES 1
 #endif
 #include "gpu/GLES2/gl2ext.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/bind.h"
 #include "base/lazy_instance.h"
 #include "base/string_tokenizer.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #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"
 #include "gpu/command_buffer/client/gles2_implementation.h"
 #include "gpu/command_buffer/common/constants.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebDocument.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebView.h"
 #include "webkit/glue/gl_bindings_skia_cmd_buffer.h"
 
@@ skipping 224 matching lines @@
   cached_width_ = width;
   cached_height_ = height;
 
   gl_->ResizeCHROMIUM(width, height);
 
 #ifdef FLIP_FRAMEBUFFER_VERTICALLY
   scanline_.reset(new uint8[width * 4]);
 #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,
     unsigned int width,
     unsigned int height) {
   uint8* scanline = scanline_.get();
   if (!scanline)
     return;
   unsigned int row_bytes = width * 4;
   unsigned int count = height / 2;
@@ skipping 802 matching lines @@
   if (context_lost_callback_) {
     context_lost_callback_->onContextLost();
   }
   RenderViewImpl* renderview =
       web_view_ ? RenderViewImpl::FromWebView(web_view_) : NULL;
   if (renderview)
     renderview->OnViewContextSwapBuffersAborted();
 }
 
 #endif  // defined(ENABLE_GPU)