cc::OutputSurfaceClient::InitializeForGL

cc/trees/layer_tree_host_impl.h

Index: cc/trees/layer_tree_host_impl.h
diff --git a/cc/trees/layer_tree_host_impl.h b/cc/trees/layer_tree_host_impl.h
index 02c0a908d8a28deb42078c9c2ae40a14a18f1eee..c75296b4df9085b2c6f3d0080e5069efbcb36eaf 100644
--- a/cc/trees/layer_tree_host_impl.h
+++ b/cc/trees/layer_tree_host_impl.h
@@ -50,6 +50,9 @@ struct RendererCapabilities;
 // LayerTreeHost->Proxy callback interface.
 class LayerTreeHostImplClient {
  public:
+  virtual void DidTryInitializeRendererOnImplThread(
+      bool success,
+      scoped_refptr<ContextProvider> offscreen_context_provider) = 0;
   virtual void DidLoseOutputSurfaceOnImplThread() = 0;
   virtual void OnSwapBuffersCompleteOnImplThread() = 0;
   virtual void OnVSyncParametersChanged(base::TimeTicks timebase,
@@ -204,6 +207,8 @@ class CC_EXPORT LayerTreeHostImpl
       OVERRIDE;
 
   // OutputSurfaceClient implementation.
+  virtual bool DeferredInitialize(
+      scoped_refptr<ContextProvider> offscreen_context_provider) OVERRIDE;

[danakj, 2013/06/05 21:53:32]

Is the embedder just going to have to always provide this provider here?

[boliu, 2013/06/05 22:17:27]

Part 3 of this patch will just pass in NULL for now.

The eventual plan (not reviewed by anyone yet), this takes a few steps:

1) First make sure LTHClient is creating the in process context for webview:
https://codereview.chromium.org/16235005/
2) Pre-create this context when output surface is created (cheap for in-process
context) and hold a ref to be passed here.
3) Move the logic to determine if an offscreen context is needed (I think if
filters are in use?) to impl side. Basically just pass the boolean over in a
commit.
4) Then finally, DeferredInitialize can decide if it's actually needed and drop
the ref if it's not.

Dreaming big about supporting repeatedly going into and out of GL mode
synchronously, dealing with lost offscreen context before the first hardware
init, and possibly clean up this code up for all platforms:

Have the compositor thread offscreen context be created/destroyed entirely on
the compositor thread. Then can avoid the set_leak_on_destroy hack as well.

   virtual void SetNeedsRedrawRect(gfx::Rect rect) OVERRIDE;
   virtual void OnVSyncParametersChanged(base::TimeTicks timebase,
                                         base::TimeDelta interval) OVERRIDE;
@@ -397,6 +402,9 @@ class CC_EXPORT LayerTreeHostImpl
   Proxy* proxy_;
 
  private:
+  bool DoInitializeRenderer(scoped_ptr<OutputSurface> output_surface,
+                            bool is_deffered_init);
+
   void AnimatePageScale(base::TimeTicks monotonic_time);
   void AnimateScrollbars(base::TimeTicks monotonic_time);
   void AnimateTopControls(base::TimeTicks monotonic_time);