Don't require a recreation of the context when fullscreening

content/renderer/pepper/pepper_platform_context_3d_impl.cc

 // Copyright (c) 2012 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.
 
 #include "content/renderer/pepper/pepper_platform_context_3d_impl.h"
 
 #include "base/bind.h"
 #include "content/common/gpu/client/gpu_channel_host.h"
 #include "content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.h"
 #include "content/renderer/pepper/pepper_parent_context_provider.h"
@@ skipping 129 matching lines @@
   parent_texture_id_ = parent_gles2->MakeTextureId();
 
   CommandBufferProxy* parent_command_buffer =
       parent_context_->GetCommandBufferProxy();
   if (!command_buffer_->SetParent(parent_command_buffer, parent_texture_id_))
     return false;
 
   return true;
 }
 
+void PlatformContext3DImpl::SetParentContext(
+    PepperParentContextProvider* parent_context_provider) {
+  if (parent_context_.get() && parent_texture_id_ != 0) {
+    // Flush any remaining commands in the parent context to make sure the
+    // texture id accounting stays consistent.
+    gpu::gles2::GLES2Implementation* parent_gles2 =
+        parent_context_->GetImplementation();
+    parent_gles2->helper()->CommandBufferHelper::Finish();

[piman, 2012/09/13 00:04:33]

BTW, this has been carried over for some time, but with the Flush semantics, I'm
pretty sure we don't need a Finish, just a Flush.

+    parent_gles2->FreeTextureId(parent_texture_id_);
+    parent_context_.reset();
+    parent_texture_id_ = 0;
+  }
+
+  WebGraphicsContext3DCommandBufferImpl* parent_context =
+      parent_context_provider->GetParentContextForPlatformContext3D();
+  if (!parent_context)
+    return;
+
+  parent_context_ = parent_context->AsWeakPtr();
+  // Flush any remaining commands in the parent context to make sure the
+  // texture id accounting stays consistent.
+  gpu::gles2::GLES2Implementation* parent_gles2 =
+      parent_context_->GetImplementation();
+  parent_gles2->helper()->CommandBufferHelper::Finish();

[piman, 2012/09/13 00:04:33]

same here.

+  parent_texture_id_ = parent_gles2->MakeTextureId();
+
+  CommandBufferProxy* parent_command_buffer =
+      parent_context_->GetCommandBufferProxy();
+  command_buffer_->SetParent(parent_command_buffer, parent_texture_id_);
+}
+
 unsigned PlatformContext3DImpl::GetBackingTextureId() {
   DCHECK(command_buffer_);
   return parent_texture_id_;
 }
 
 bool PlatformContext3DImpl::IsOpaque() {
   DCHECK(command_buffer_);
   return !has_alpha_;
 }
 
@@ skipping 29 matching lines @@
 void PlatformContext3DImpl::OnConsoleMessage(const std::string& msg, int id) {
   DCHECK(command_buffer_);
 
   if (!console_message_callback_.is_null())
     console_message_callback_.Run(msg, id);
 }
 
 }  // namespace content
 
 #endif  // ENABLE_GPU