Merge cc initialization paths

Code path between first initialization and recreate on context
lost are merged. Do this by changing the first initialization
behavior to match recreate. Now both are kicked off by the
scheduler and blocks the main thread on the impl thread.

The scheduler is started in output surface lost state and
immediately schedules recreation. This means the first
initialization in thread mode is no longer synchronous.

BUG=233664, 230197
This has been tried many many times.
NOTRY=true

Reverted twice due to breaking WebGLInfobarTest:
Committed in r196480. Reverted in r196509.
Committed in r196708. Reverted in r196790.
Fix for WebGLInfobarTest landed in r197235.

WebViewTest.Shim became flaky on chromeos but was already
flaky without this. Disabled it in r197497.

Review URL: https://chromiumcodereview.appspot.com/12544032

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@197550 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 1 deletions

diff --git a/cc/trees/layer_tree_host_client.h b/cc/trees/layer_tree_host_client.h
index 0bef011..324b93f 100644
--- a/cc/trees/layer_tree_host_client.h
+++ b/cc/trees/layer_tree_host_client.h
@@ -28,6 +28,7 @@ class LayerTreeHostClient {
   virtual void ApplyScrollAndScale(gfx::Vector2d scroll_delta,
                                    float page_scale) = 0;
   virtual scoped_ptr<OutputSurface> CreateOutputSurface() = 0;
+  // TODO(boliu): Rename to DidInitializeOutputSurface.
   virtual void DidRecreateOutputSurface(bool success) = 0;
   virtual scoped_ptr<InputHandlerClient> CreateInputHandlerClient() = 0;
   virtual void WillCommit() = 0;
@@ -46,7 +47,7 @@ class LayerTreeHostClient {
       OffscreenContextProviderForCompositorThread() = 0;
 
   // This hook is for testing.
-  virtual void WillRetryRecreateOutputSurface() {}
+  virtual void DidFailToInitializeOutputSurface() {}
 
  protected:
   virtual ~LayerTreeHostClient() {}