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

Unified Diff: cc/surfaces/surface_factory_unittest.cc

Issue 2614423003: Adding SatisfySequence and RequireSequence to SurfaceManager (Closed)
Patch Set: up 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_factory_unittest.cc
diff --git a/cc/surfaces/surface_factory_unittest.cc b/cc/surfaces/surface_factory_unittest.cc
index 2bc51e9d638d3a820971507c2f754c5ab7abbde7..8099815d2298277b62ca6f9451a510a096f33020 100644
--- a/cc/surfaces/surface_factory_unittest.cc
+++ b/cc/surfaces/surface_factory_unittest.cc
@@ -526,8 +526,8 @@ TEST_F(SurfaceFactoryTest, DestroySequence) {
CompositorFrame frame;
DCHECK(manager_.GetSurfaceForId(id2));
- std::vector<uint32_t> sequences = {4, 6};
- manager_.DidSatisfySequences(kArbitraryFrameSinkId, &sequences);
+ manager_.SatisfySequence(SurfaceSequence(kArbitraryFrameSinkId, 4));
+ manager_.SatisfySequence(SurfaceSequence(kArbitraryFrameSinkId, 6));
DCHECK(!manager_.GetSurfaceForId(id2));
// Check that waiting after the sequence is satisfied works.
@@ -602,9 +602,7 @@ TEST_F(SurfaceFactoryTest, DestroyCycle) {
SurfaceId(factory_->frame_sink_id(), local_frame_id_)));
// Satisfy last destruction dependency for id2.
- std::vector<uint32_t> to_satisfy;
- to_satisfy.push_back(4);
- manager_.DidSatisfySequences(kAnotherArbitraryFrameSinkId, &to_satisfy);
+ manager_.SatisfySequence(SurfaceSequence(kAnotherArbitraryFrameSinkId, 4));
// id2 and local_frame_id_ are in a reference cycle that has no surface
// sequences holding on to it, so they should be destroyed.

Powered by Google App Engine
This is Rietveld 408576698