cc: Abort frame when becoming invisible and waiting for ready to draw.

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/location.h"
@@ skipping 254 matching lines @@
   }
 
  private:
   FakeContentLayerClient client_;
 };
 
 // Note: With this test setup, we only get tiles flagged as REQUIRED_FOR_DRAW in
 // single threaded mode.
 SINGLE_THREAD_TEST_F(LayerTreeHostTestReadyToDrawNonEmpty);
 
+// This tests if we get the READY_TO_DRAW signal if we become invisible and then
+// become visible again.
+class LayerTreeHostTestReadyToDrawVisibility
+    : public LayerTreeHostTestReadyToDrawNonEmpty {
+ public:
+  LayerTreeHostTestReadyToDrawVisibility()
+      : LayerTreeHostTestReadyToDrawNonEmpty(), commit_count_(0) {}
+
+  void CommitCompleteOnThread(LayerTreeHostImpl* host_impl) override {
+    commit_count_++;
+    if (commit_count_ == 1) {
+      DebugScopedSetMainThread main(proxy());
+      layer_tree_host()->SetVisible(false);

[brianderson, 2015/07/23 22:58:15]

EXPECT_FALSE(host_impl->visible()) here?

[sunnyps, 2015/07/23 23:10:50]

Done.

+    }
+  }
+
+  void DidFinishImplFrameOnThread(LayerTreeHostImpl* host_impl) override {
+    if (commit_count_ == 1) {
+      DebugScopedSetMainThread main(proxy());
+      layer_tree_host()->SetVisible(true);

[brianderson, 2015/07/23 22:58:15]

EXPECT_TRUE(host_impl->visible()) here?

[sunnyps, 2015/07/23 23:10:50]

Done.

+    }
+  }
+
+  void AfterTest() override {
+    LayerTreeHostTestReadyToDrawNonEmpty::AfterTest();
+    EXPECT_EQ(2, commit_count_);
+  }
+
+ private:
+  int commit_count_;
+};
+
+// Note: With this test setup, we only get tiles flagged as REQUIRED_FOR_DRAW in
+// single threaded mode.
+SINGLE_THREAD_TEST_F(LayerTreeHostTestReadyToDrawVisibility);
+
 class LayerTreeHostFreeWorkerContextResourcesTest : public LayerTreeHostTest {
  public:
   scoped_ptr<FakeOutputSurface> CreateFakeOutputSurface() override {
     auto output_surface = make_scoped_ptr(new testing::StrictMock<
         MockSetWorkerContextShouldAggressivelyFreeResourcesOutputSurface>(
         delegating_renderer()));
 
     // At init, we expect one call to set visibility to true.
     testing::Expectation visibility_true =
         EXPECT_CALL(*output_surface,
@@ skipping 5770 matching lines @@
   void AfterTest() override {}
 
   std::vector<int> affected_by_page_scale_;
   std::vector<int> not_affected_by_page_scale_;
 };
 
 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeTestPageScaleFlags);
 
 }  // namespace
 }  // namespace cc