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

Unified Diff: blimp/client/core/compositor/blimp_compositor_unittest.cc

Issue 2317753002: cc: Abstract the LayerTreeHost. (Closed)
Patch Set: export for windows, keep the remote server test Created 4 years, 3 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: blimp/client/core/compositor/blimp_compositor_unittest.cc
diff --git a/blimp/client/core/compositor/blimp_compositor_unittest.cc b/blimp/client/core/compositor/blimp_compositor_unittest.cc
index bf10f20d1369277bb4267d79164e363b70f3c4d3..00c1ee26510daee6188e47ac54e26e75761ce102 100644
--- a/blimp/client/core/compositor/blimp_compositor_unittest.cc
+++ b/blimp/client/core/compositor/blimp_compositor_unittest.cc
@@ -58,7 +58,7 @@ class BlimpCompositorForTesting : public BlimpCompositor {
SendWebGestureEvent(gesture_event);
}
- cc::LayerTreeHost* host() const { return host_.get(); }
+ cc::LayerTreeHostInterface* host() const { return host_.get(); }
};
class BlimpCompositorTest : public testing::Test {
@@ -116,11 +116,11 @@ TEST_F(BlimpCompositorTest, ToggleVisibilityWithHost) {
// Check that the visibility is set correctly on the host.
EXPECT_NE(compositor_->host(), nullptr);
- EXPECT_TRUE(compositor_->host()->visible());
+ EXPECT_TRUE(compositor_->host()->IsVisible());
// Make the compositor invisible. This should make the |host_| invisible.
compositor_->SetVisible(false);
- EXPECT_FALSE(compositor_->host()->visible());
+ EXPECT_FALSE(compositor_->host()->IsVisible());
SendShutdownMessage();
EXPECT_EQ(compositor_->host(), nullptr);
@@ -136,7 +136,7 @@ TEST_F(BlimpCompositorTest, DestroyAndRecreateHost) {
SendShutdownMessage();
SendInitializeMessage();
EXPECT_NE(compositor_->host(), nullptr);
- EXPECT_TRUE(compositor_->host()->visible());
+ EXPECT_TRUE(compositor_->host()->IsVisible());
}
TEST_F(BlimpCompositorTest, MessagesHaveCorrectId) {

Powered by Google App Engine
This is Rietveld 408576698