Progress towards fixing 77536

content/renderer/render_widget_fullscreen_pepper.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.
 
 #include "content/renderer/render_widget_fullscreen_pepper.h"
 
 #include "base/message_loop.h"
 #include "content/common/view_messages.h"
 #include "content/renderer/renderer_gl_context.h"
 #include "content/renderer/gpu_channel_host.h"
@@ skipping 283 matching lines @@
     gl->Viewport(0, 0, size.width(), size.height());
   }
   RenderWidget::OnResize(size, resizer_rect);
 }
 
 WebWidget* RenderWidgetFullscreenPepper::CreateWebWidget() {
   return new PepperWidget(plugin_, this);
 }
 
 void RenderWidgetFullscreenPepper::CreateContext() {
   DCHECK(!context_);

[piman, 2011/04/15 21:53:33]

This can in theory be called before we have the compositing_surface, in which
case returning early is the right thing to do. The plugin's context creation
will fail but that's expected.

   RenderThread* render_thread = RenderThread::current();
   DCHECK(render_thread);
   GpuChannelHost* host = render_thread->EstablishGpuChannelSync(
     content::CAUSE_FOR_GPU_LAUNCH_RENDERWIDGETFULLSCREENPEPPER_CREATECONTEXT);
   if (!host)
     return;
   const int32 attribs[] = {
     RendererGLContext::ALPHA_SIZE, 8,
     RendererGLContext::DEPTH_SIZE, 0,
     RendererGLContext::STENCIL_SIZE, 0,
     RendererGLContext::SAMPLES, 0,
     RendererGLContext::SAMPLE_BUFFERS, 0,
     RendererGLContext::NONE,
   };
   context_ = RendererGLContext::CreateViewContext(
       host,
+      compositing_surface(),
       routing_id(),
       "GL_OES_packed_depth_stencil GL_OES_depth24",
       attribs,
       active_url_);
   if (!context_)
     return;
 
   if (!InitContext()) {
     DestroyContext(context_, program_, buffer_);
     context_ = NULL;
@@ skipping 113 matching lines @@
   // Destroy the context later, in case we got called from InitContext for
   // example. We still need to reset context_ now so that a new context gets
   // created when the plugin recreates its own.
   MessageLoop::current()->PostTask(
       FROM_HERE,
       NewRunnableFunction(DestroyContext, context_, program_, buffer_));
   context_ = NULL;
   program_ = 0;
   buffer_ = 0;
 }