Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(418)

Unified Diff: cc/surfaces/surface_manager_ref_unittest.cc

Issue 2614423003: Adding SatisfySequence and RequireSequence to SurfaceManager (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/surfaces/surface_manager_ref_unittest.cc
diff --git a/cc/surfaces/surface_manager_ref_unittest.cc b/cc/surfaces/surface_manager_ref_unittest.cc
index 9776a6964f2749ac9c07ae48487b53d81488d598..b3b4995c44891cccfd566c9f3796cfef8dfdc460 100644
--- a/cc/surfaces/surface_manager_ref_unittest.cc
+++ b/cc/surfaces/surface_manager_ref_unittest.cc
@@ -247,8 +247,7 @@ TEST_F(SurfaceManagerRefTest, CheckGCWithSequences) {
EXPECT_NE(manager().GetSurfaceForId(id2), nullptr);
// Satisfy destruction dependency on |id3| and delete during GC.
- std::vector<uint32_t> satisfied({sequence.sequence});
- manager().DidSatisfySequences(kFrameSink2, &satisfied);
+ manager().SatisfySequence(sequence);
EXPECT_EQ(manager().GetSurfaceForId(id3), nullptr);
// Remove ref on |id2| and delete during GC.

Powered by Google App Engine
This is Rietveld 408576698