Remove WebGraphicsContext3D getter from cc::ContextProvider

content/browser/android/in_process/synchronous_compositor_factory_impl.cc

Index: content/browser/android/in_process/synchronous_compositor_factory_impl.cc
diff --git a/content/browser/android/in_process/synchronous_compositor_factory_impl.cc b/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
index 3c86bd5b17c35e02a2e6319c1fd0bfa125ab190c..fa06db39caf74525c72d5973f86d460c33a76e1d 100644
--- a/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
+++ b/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
@@ -32,8 +32,10 @@ class VideoContextProvider
     return gl_in_process_context_->GetSurfaceTexture(stream_id);
   }
 
-  virtual blink::WebGraphicsContext3D* Context3d() OVERRIDE {
-    return context_provider_->Context3d();
+  virtual gpu::gles2::GLES2Interface* ContextGL() OVERRIDE {
+    if (context_provider_->IsContextLost())

[danakj, 2014/01/09 21:23:34]

It's kinda weird to have a NULL pointer due to context loss, as that's usually
not what we do.

I see how you're doing this to branch where we used to branch on
makeContextCurrent() because the provider is not exposed only the ContextGL is.

If you're going to do this can you update the comment on the base class to warn
about NULL return values? Maybe it'd be better to just return the
ContextProvider instead of the ContextGL here?

[jamesr, 2014/01/10 00:12:34]

sievers@ - do you know why StreamTextureFactory uses a raw context pointer
instead of a ref to a ContextProvider?  I think using a provider would make a
bit more sense.  I just tried to preserve existing behavior here.

+      return NULL;
+    return context_provider_->ContextGL();
   }
 
  private: