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

Unified Diff: cc/surfaces/surface_synchronization_unittest.cc

Issue 2940183002: cc: Move ownership of surfaces to SurfaceManager (Closed)
Patch Set: c Created 3 years, 6 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
« cc/surfaces/surface_manager.cc ('K') | « cc/surfaces/surface_manager.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/surfaces/surface_synchronization_unittest.cc
diff --git a/cc/surfaces/surface_synchronization_unittest.cc b/cc/surfaces/surface_synchronization_unittest.cc
index c9983d75642d7f8627255a1983af8ccd31bd4139..ee487e59b0ff07720ac7a1faa6d53e369cbfee86 100644
--- a/cc/surfaces/surface_synchronization_unittest.cc
+++ b/cc/surfaces/surface_synchronization_unittest.cc
@@ -75,29 +75,19 @@ class SurfaceSynchronizationTest : public testing::Test {
~SurfaceSynchronizationTest() override {}
CompositorFrameSinkSupport& display_support() { return *supports_[0]; }
- Surface* display_surface() {
- return display_support().current_surface_for_testing();
- }
+ Surface* display_surface() { return display_support().GetCurrentSurface(); }
CompositorFrameSinkSupport& parent_support() { return *supports_[1]; }
- Surface* parent_surface() {
- return parent_support().current_surface_for_testing();
- }
+ Surface* parent_surface() { return parent_support().GetCurrentSurface(); }
CompositorFrameSinkSupport& child_support1() { return *supports_[2]; }
- Surface* child_surface1() {
- return child_support1().current_surface_for_testing();
- }
+ Surface* child_surface1() { return child_support1().GetCurrentSurface(); }
CompositorFrameSinkSupport& child_support2() { return *supports_[3]; }
- Surface* child_surface2() {
- return child_support2().current_surface_for_testing();
- }
+ Surface* child_surface2() { return child_support2().GetCurrentSurface(); }
CompositorFrameSinkSupport& support(int index) { return *supports_[index]; }
- Surface* surface(int index) {
- return support(index).current_surface_for_testing();
- }
+ Surface* surface(int index) { return support(index).GetCurrentSurface(); }
SurfaceManager& surface_manager() { return surface_manager_; }
@@ -1041,7 +1031,7 @@ TEST_F(SurfaceSynchronizationTest, SurfaceResurrection) {
child_support1().EvictCurrentSurface();
surface = surface_manager().GetSurfaceForId(child_id);
EXPECT_NE(nullptr, surface);
- EXPECT_TRUE(surface->destroyed());
+ EXPECT_TRUE(surface_manager().IsMarkedForDestruction(child_id));
// Child submits another frame to the same local surface id that is marked
// destroyed.
@@ -1052,7 +1042,7 @@ TEST_F(SurfaceSynchronizationTest, SurfaceResurrection) {
// used again.
Surface* surface2 = surface_manager().GetSurfaceForId(child_id);
EXPECT_EQ(surface, surface2);
- EXPECT_FALSE(surface2->destroyed());
+ EXPECT_FALSE(surface_manager().IsMarkedForDestruction(child_id));
}
// Verifies that if a LocalSurfaceId belonged to a surface that doesn't exist
« cc/surfaces/surface_manager.cc ('K') | « cc/surfaces/surface_manager.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698