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"
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
+#include "cc/context_provider.h"
 #include "cc/input_handler.h"
 #include "cc/layer.h"
 #include "cc/layer_tree_host.h"
 #include "cc/output_surface.h"
 #include "cc/thread_impl.h"
 #include "content/browser/gpu/browser_gpu_channel_host_factory.h"
 #include "content/browser/gpu/gpu_surface_tracker.h"
 #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"
@@ skipping 295 matching lines @@
 }
 
 void CompositorImpl::didCompleteSwapBuffers() {
   client_->OnSwapBuffersCompleted();
 }
 
 void CompositorImpl::scheduleComposite() {
   client_->ScheduleComposite();
 }
 
+class NullContextProvider : public cc::ContextProvider {
+  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<cc::ContextProvider>
+CompositorImpl::OffscreenContextProviderForMainThread() {
+  // There is no support for offscreen contexts, or compositor filters that
+  // would require them in this compositor instance. If they are needed,]

[piman, 2013/02/22 03:16:39]

nit: typo ']'

[danakj, 2013/02/22 03:23:09]

*^_^*

+  // then implement a context provider that provides contexts from
+  // ImageTransportSurfaceAndroid.
+  if (!null_offscreen_context_provider_)
+    null_offscreen_context_provider_ = new NullContextProvider();
+  return null_offscreen_context_provider_;
+}
+
+scoped_refptr<cc::ContextProvider>
+CompositorImpl::OffscreenContextProviderForCompositorThread() {
+  // There is no support for offscreen contexts, or compositor filters that
+  // would require them in this compositor instance. If they are needed,]

[piman, 2013/02/22 03:16:39]

nit: typo ']'

+  // then implement a context provider that provides contexts from
+  // ImageTransportSurfaceAndroid.
+  if (!null_offscreen_context_provider_)
+    null_offscreen_context_provider_ = new NullContextProvider();
+  return null_offscreen_context_provider_;
+}
+
 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