cc: Add readback and forced draw states to the Scheduler

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 485 matching lines @@
 
   virtual void AfterTest() OVERRIDE {}
 
  private:
   int num_commits_;
 };
 
 MULTI_THREAD_TEST_F(
     LayerTreeHostTestCompositeAndReadbackBeforePreviousCommitDraws);
 
+class LayerTreeHostTestCompositeAndReadbackDuringForcedDraw
+    : public LayerTreeHostTest {
+ protected:
+  LayerTreeHostTestCompositeAndReadbackDuringForcedDraw()
+      : prepare_to_draw_count_(0) {}
+
+  virtual void InitializeSettings(LayerTreeSettings* settings) OVERRIDE {
+    // This enables forced draws after a single prepare to draw failure.
+    settings->timeout_and_draw_when_animation_checkerboards = true;
+    settings->maximum_number_of_failed_draws_before_draw_is_forced_ = 1;
+  }
+
+  virtual void BeginTest() OVERRIDE { PostSetNeedsCommitToMainThread(); }
+
+  virtual bool PrepareToDrawOnThread(LayerTreeHostImpl* host_impl,
+                                     LayerTreeHostImpl::FrameData* frame_data,
+                                     bool result) OVERRIDE {
+    prepare_to_draw_count_++;
+    EXPECT_LE(prepare_to_draw_count_, 3);
+
+    // Before we react to the failed draw by initiating the forced draw
+    // sequence, start a readback on the main thread.
+    if (prepare_to_draw_count_ ==
+        host_impl->settings()
+            .maximum_number_of_failed_draws_before_draw_is_forced_) {
+      PostReadbackToMainThread();
+    }
+
+    // Returning false will eventually result in a forced draw.
+    return false;
+  }
+
+  virtual void DrawLayersOnThread(LayerTreeHostImpl* host_impl) OVERRIDE {
+    // We should only draw for the readback and the forced draw.
+    EXPECT_GE(prepare_to_draw_count_, 2);
+    EXPECT_LE(prepare_to_draw_count_, 3);
+  }
+
+  virtual void SwapBuffersOnThread(LayerTreeHostImpl* host_impl,
+                                   bool result) OVERRIDE {
+    // We should only swap for the forced draw.
+    EXPECT_EQ(prepare_to_draw_count_, 3);

[brianderson, 2013/09/04 21:34:39]

This expectation fails with the delegated renderer. Does returning false from
PrepareToDraw behave differently with the delegated renderer?

[brianderson, 2013/09/04 21:48:59]

I looks like the delegating renderer cannot do a readback and does not bother
calling prepare to draw for the readback. I will adjust the test expectation
accordingly.

+    EndTest();
+  }
+
+  virtual void AfterTest() OVERRIDE {}
+
+  int prepare_to_draw_count_;
+};
+
+MULTI_THREAD_TEST_F(LayerTreeHostTestCompositeAndReadbackDuringForcedDraw);
+
 // If the layerTreeHost says it can't draw, Then we should not try to draw.
 class LayerTreeHostTestCanDrawBlocksDrawing : public LayerTreeHostTest {
  public:
   LayerTreeHostTestCanDrawBlocksDrawing() : num_commits_(0), done_(false) {}
 
   virtual void BeginTest() OVERRIDE { PostSetNeedsCommitToMainThread(); }
 
   virtual void DrawLayersOnThread(LayerTreeHostImpl* impl) OVERRIDE {
     if (done_)
       return;
@@ skipping 3944 matching lines @@
   int num_will_begin_frames_;
   int num_impl_commits_;
 };
 
 // Commits can only be aborted when using the thread proxy.
 MULTI_THREAD_TEST_F(LayerTreeHostTestAbortEvictedTextures);
 
 }  // namespace
 
 }  // namespace cc