cc: Update Main RendererCapabilities on DeferredInitialize

cc/trees/layer_tree_host_unittest.cc

 // Copyright 2011 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 "cc/trees/layer_tree_host.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/synchronization/lock.h"
@@ skipping 3075 matching lines @@
  private:
   FakeContentLayerClient client_;
   scoped_refptr<FakePictureLayer> layer_;
   bool did_initialize_gl_;
   bool did_release_gl_;
   int last_source_frame_number_drawn_;
 };
 
 MULTI_THREAD_TEST_F(LayerTreeHostTestDeferredInitialize);
 
+class LayerTreeHostTestAbortFrameOnStaleRendererCapabilities
+    : public LayerTreeHostTest {
+ public:
+  virtual void InitializeSettings(LayerTreeSettings* settings) OVERRIDE {
+    settings->impl_side_painting = true;
+  }
+
+  virtual void SetupTree() OVERRIDE {
+    layer_ = FakePictureLayer::Create(&client_);
+    // Make sure commits are not aborted due to no update.
+    layer_->set_always_update_resources(true);
+    layer_tree_host()->SetRootLayer(layer_);
+    LayerTreeHostTest::SetupTree();
+  }
+
+  virtual scoped_ptr<FakeOutputSurface> CreateFakeOutputSurface(bool fallback)
+      OVERRIDE {
+    scoped_ptr<TestWebGraphicsContext3D> context3d(

[danakj, 2014/02/11 19:58:59]

unused?

[boliu, 2014/02/11 21:44:07]

Done.

+        TestWebGraphicsContext3D::Create());
+
+    return FakeOutputSurface::CreateDeferredGL(
+        scoped_ptr<SoftwareOutputDevice>(new SoftwareOutputDevice));
+  }
+
+  virtual void BeginTest() OVERRIDE {
+    aborted_frame_ = true;
+    main_frame_attempts_ = 0;
+    PostSetNeedsCommitToMainThread();
+  }
+
+  virtual void WillBeginMainFrame() OVERRIDE {
+    int frame_number = layer_tree_host()->source_frame_number();
+    EXPECT_EQ(0, frame_number);  // Only one frame.
+    main_frame_attempts_++;
+    // Main frame should only be aborted once.
+    EXPECT_LE(main_frame_attempts_, 2);
+
+    if (main_frame_attempts_ == 1) {
+      // Capabilities updated once on first renderer init.
+      EXPECT_EQ(static_cast<size_t>(1),

[danakj, 2014/02/11 19:58:59]

"1u" instead of static_cast<size_t>(1)

[boliu, 2014/02/11 21:44:07]

Done.

+                layer_->renderer_capabilities_changed_count());
+      layer_->reset_renderer_capabilities_changed_count();
+
+      // Make sure first main frame attemp gets aborted due to stale renderer

[danakj, 2014/02/11 19:58:59]

attempt

[boliu, 2014/02/11 21:44:07]

Done.

+      // capabilities by synchronously updating renderer capabilities on impl
+      // thread.
+      CompletionEvent completion;
+      ImplThreadTaskRunner()->PostTask(
+          FROM_HERE,
+          base::Bind(&LayerTreeHostTestAbortFrameOnStaleRendererCapabilities::
+                          UpdateRendererCapabilities,
+                     base::Unretained(this),
+                     &completion));
+      completion.Wait();
+    } else {
+      // Capabilities updated again before second attempt of first frame.
+      EXPECT_EQ(static_cast<size_t>(1),

[danakj, 2014/02/11 19:58:59]

1u

[boliu, 2014/02/11 21:44:07]

Done.

+                layer_->renderer_capabilities_changed_count());
+      layer_->reset_renderer_capabilities_changed_count();
+    }
+  }
+
+  void UpdateRendererCapabilities(CompletionEvent* completion) {
+    // Change renderer capabilities by switching renderers.
+    scoped_refptr<TestContextProvider> context_provider =
+        TestContextProvider::Create();
+    EXPECT_TRUE(
+        output_surface()->InitializeAndSetContext3d(context_provider, NULL));
+    completion->Signal();
+  }
+
+  virtual void BeginMainFrameAbortedOnThread(LayerTreeHostImpl* host_impl,
+                                             bool did_handle) OVERRIDE {
+    EXPECT_TRUE(did_handle);
+    aborted_frame_ = true;

[danakj, 2014/02/11 19:58:59]

use an int instead so you can test the number of times

[boliu, 2014/02/11 21:44:07]

Done.

+  }
+
+  virtual void DrawLayersOnThread(LayerTreeHostImpl* host_impl) OVERRIDE {
+    EndTest();
+  }
+
+  virtual void AfterTest() OVERRIDE {
+    // First main frame should have been attempted twice.
+    EXPECT_EQ(2, main_frame_attempts_);
+    // First attempted should have been aborted.

[danakj, 2014/02/11 19:58:59]

attempt

[boliu, 2014/02/11 21:44:07]

Done.

+    EXPECT_TRUE(aborted_frame_);
+  }
+
+ private:
+  FakeContentLayerClient client_;
+  scoped_refptr<FakePictureLayer> layer_;
+  int main_frame_attempts_;
+  bool aborted_frame_;
+};
+
+MULTI_THREAD_TEST_F(LayerTreeHostTestAbortFrameOnStaleRendererCapabilities);
+
 // Test for UI Resource management.
 class LayerTreeHostTestUIResource : public LayerTreeHostTest {
  public:
   LayerTreeHostTestUIResource() : num_ui_resources_(0) {}
 
   virtual void InitializeSettings(LayerTreeSettings* settings) OVERRIDE {
     settings->texture_id_allocation_chunk_size = 1;
   }
 
   virtual void BeginTest() OVERRIDE { PostSetNeedsCommitToMainThread(); }
@@ skipping 1858 matching lines @@
 
     EndTest();
   }
 
   virtual void AfterTest() OVERRIDE {}
 };
 
 MULTI_THREAD_TEST_F(LayerTreeHostTestSimpleSwapPromiseMonitor);
 
 }  // namespace cc