cc: Route offscreen context creation for compositor to the browser.

content/browser/renderer_host/compositor_impl_android.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/browser/renderer_host/compositor_impl_android.h"
 
 #include <android/bitmap.h>
 #include <android/native_window_jni.h>
 
 #include "base/bind.h"
@@ skipping 10 matching lines @@
 #include "content/browser/renderer_host/image_transport_factory_android.h"
 #include "content/common/gpu/client/gl_helper.h"
 #include "content/common/gpu/client/gpu_channel_host.h"
 #include "content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.h"
 #include "content/common/gpu/gpu_process_launch_causes.h"
 #include "content/public/common/content_switches.h"
 #include "third_party/khronos/GLES2/gl2.h"
 #include "third_party/khronos/GLES2/gl2ext.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebGraphicsContext3D.h"
 #include "ui/gfx/android/java_bitmap.h"
+#include "ui/gl/context_provider.h"
 #include "webkit/glue/webthread_impl.h"
 #include "webkit/gpu/webgraphicscontext3d_in_process_impl.h"
 
 namespace gfx {
 class JavaBitmap;
 }
 
 namespace {
 
 static bool g_initialized = false;
@@ skipping 278 matching lines @@
 }
 
 void CompositorImpl::didCompleteSwapBuffers() {
   client_->OnSwapBuffersCompleted();
 }
 
 void CompositorImpl::scheduleComposite() {
   client_->ScheduleComposite();
 }
 
+class NullContextProvider : public ui::ContextProvider {
+  // TODO(danakj): Get a context through ImageTransportSurfaceAndroid.
+  virtual bool InitializeOnMainThread() { return false; }
+  virtual bool BindToCurrentThread() { return false; }
+  virtual WebKit::WebGraphicsContext3D* Context3d() { return NULL; }
+  virtual class GrContext* GrContext() { return NULL; }
+  virtual void VerifyContexts() {}
+ protected:
+  virtual ~NullContextProvider() {}
+};
+
+scoped_refptr<ui::ContextProvider>
+CompositorImpl::OffscreenContextProviderForMainThread() {
+  if (!null_offscreen_context_provider_)
+    null_offscreen_context_provider_ = new NullContextProvider();
+  return null_offscreen_context_provider_;
+}
+
+scoped_refptr<ui::ContextProvider>
+CompositorImpl::OffscreenContextProviderForCompositorThread() {
+  if (!null_offscreen_context_provider_)
+    null_offscreen_context_provider_ = new NullContextProvider();
+  return null_offscreen_context_provider_;

[piman, 2013/02/21 22:49:48]

I assume that Android UI doesn't use filters right now?

[danakj, 2013/02/22 01:56:31]

Right. And don't intend to use filters that will require this context. I
confirmed this with @aelias.

[piman, 2013/02/22 02:54:07]

Worth a comment? "If you're trying to use filters and the world blows up,
implement this..."

[danakj, 2013/02/22 03:04:49]

ya, sure! the world won't blow cuz we nullcheck them, but it'll do nothing.

+}
+
 void CompositorImpl::OnViewContextSwapBuffersPosted() {
   TRACE_EVENT0("compositor", "CompositorImpl::OnViewContextSwapBuffersPosted");
 }
 
 void CompositorImpl::OnViewContextSwapBuffersComplete() {
   TRACE_EVENT0("compositor",
                "CompositorImpl::OnViewContextSwapBuffersComplete");
   client_->OnSwapBuffersCompleted();
 }
 
@@ skipping 46 matching lines @@
     case ANDROID_BITMAP_FORMAT_RGBA_8888:
       return GL_UNSIGNED_BYTE;
       break;
     case ANDROID_BITMAP_FORMAT_RGB_565:
     default:
       return GL_UNSIGNED_SHORT_5_6_5;
   }
 }
 
 } // namespace content