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

Unified Diff: cc/surfaces/surface_synchronization_unittest.cc

Issue 2872883002: Rename EvictFrame, EvictSurface to EvictCurrentSurface (Closed)
Patch Set: Rebase Created 3 years, 7 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
« no previous file with comments | « cc/surfaces/surface_manager_ref_unittest.cc ('k') | cc/surfaces/surface_unittest.cc » ('j') | 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 5fa37ccf672d4f1bd26150c090318b92c6265b03..473e5876f2e33d76de3f09df814f448b58f41fe1 100644
--- a/cc/surfaces/surface_synchronization_unittest.cc
+++ b/cc/surfaces/surface_synchronization_unittest.cc
@@ -645,8 +645,8 @@ TEST_F(SurfaceSynchronizationTest, EvictSurfaceWithPendingFrame) {
UnorderedElementsAre(child_id2));
// Evict child_support1's current Surface.
- // TODO(fsamuel): EvictFrame => EvictCurrentSurface.
- child_support1().EvictFrame();
+ // TODO(fsamuel): EvictCurrentSurface => EvictCurrentSurface.
+ child_support1().EvictCurrentSurface();
// The parent Surface should immediately activate.
EXPECT_TRUE(parent_surface()->HasActiveFrame());
@@ -1046,7 +1046,7 @@ TEST_F(SurfaceSynchronizationTest, SurfaceResurrection) {
// Attempt to destroy the child surface. The surface must still exist since
// the parent needs it but it will be marked as destroyed.
- child_support1().EvictFrame();
+ child_support1().EvictCurrentSurface();
surface = surface_manager().GetSurfaceForId(child_id);
EXPECT_NE(nullptr, surface);
EXPECT_TRUE(surface->destroyed());
@@ -1084,7 +1084,7 @@ TEST_F(SurfaceSynchronizationTest, LocalSurfaceIdIsReusable) {
MakeCompositorFrame());
// Destroy the surface.
- child_support1().EvictFrame();
+ child_support1().EvictCurrentSurface();
EXPECT_EQ(nullptr, surface_manager().GetSurfaceForId(child_id));
// Submit another frame with the same local surface id. This should work fine
« no previous file with comments | « cc/surfaces/surface_manager_ref_unittest.cc ('k') | cc/surfaces/surface_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698