Reland of Getting rid of CompositorFrameMetadata::satisfies_sequences

cc/layers/surface_layer_unittest.cc

Index: cc/layers/surface_layer_unittest.cc
diff --git a/cc/layers/surface_layer_unittest.cc b/cc/layers/surface_layer_unittest.cc
index 5720f71fa8f09503dfdf273eccd0e9cd64e4a9fc..f288f7103be627dd8c44773fef9dbd6676c216c9 100644
--- a/cc/layers/surface_layer_unittest.cc
+++ b/cc/layers/surface_layer_unittest.cc
@@ -8,6 +8,7 @@
 #include <vector>
 
 #include "base/location.h"
+#include "base/run_loop.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "cc/animation/animation_host.h"
@@ -111,6 +112,7 @@
 
   // Layer was removed so sequence from second LayerTreeHost should be
   // satisfied.
+  base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(blank_change == expected2);
 
   // Set of sequences that need to be satisfied should include sequences from
@@ -126,6 +128,7 @@
 
   // Layer was removed so sequence from first LayerTreeHost should be
   // satisfied.
+  base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(blank_change == expected1);
 
   // No more SurfaceSequences should have been generated that need to have be
@@ -203,25 +206,11 @@
         layer_tree()->SetRootLayer(blank_layer_);
         break;
       case 2:
+        EndTest();
         break;
       default:
         NOTREACHED();
         break;
-    }
-  }
-
-  void DisplayReceivedCompositorFrameOnThread(
-      const CompositorFrame& frame) override {
-    const std::vector<uint32_t>& satisfied = frame.metadata.satisfies_sequences;
-    EXPECT_LE(satisfied.size(), 1u);
-    if (satisfied.size() == 1) {
-      // Eventually the one SurfaceSequence should be satisfied, but only
-      // after the layer was removed from the tree, and only once.
-      EXPECT_EQ(1u, satisfied[0]);
-      EXPECT_LE(1, commit_count_);
-      EXPECT_FALSE(sequence_was_satisfied_);
-      sequence_was_satisfied_ = true;
-      EndTest();
     }
   }
 
@@ -229,9 +218,8 @@
     EXPECT_TRUE(required_id_ == SurfaceId(kArbitraryFrameSinkId,
                                           LocalFrameId(1, kArbitraryToken)));
     EXPECT_EQ(1u, required_set_.size());
-    // Sequence should have been satisfied through Swap, not with the
-    // callback.
-    EXPECT_FALSE(satisfied_sequence_.is_valid());
+    EXPECT_TRUE(satisfied_sequence_ ==
+                SurfaceSequence(kArbitraryFrameSinkId, 1u));
   }
 };
 
@@ -269,7 +257,6 @@
     EXPECT_TRUE(required_id_ == SurfaceId(kArbitraryFrameSinkId,
                                           LocalFrameId(1, kArbitraryToken)));
     EXPECT_EQ(1u, required_set_.size());
-    // Sequence should have been satisfied with the callback.
     EXPECT_TRUE(satisfied_sequence_ ==
                 SurfaceSequence(kArbitraryFrameSinkId, 1u));
   }