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

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

Issue 2317753002: cc: Abstract the LayerTreeHost. (Closed)
Patch Set: Rebase 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
« no previous file with comments | « blimp/client/core/compositor/blimp_compositor.h ('k') | cc/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 d8a1755ed4a6bb78caeb6a59dfbf4a990e3e4823..26f1576cdc1d8cc056c1c41e355023efc861479d 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 {
@@ -112,11 +112,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);
@@ -132,7 +132,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) {
« no previous file with comments | « blimp/client/core/compositor/blimp_compositor.h ('k') | cc/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698