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

Unified Diff: cc/trees/layer_tree_host_unittest.cc

Issue 286293002: cc: Remove CreateAndInitializeOutputSurface from the Proxy interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: createandinitos: Created 6 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
Index: cc/trees/layer_tree_host_unittest.cc
diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_unittest.cc
index d1b518b44c8b1e250437f3be03c9eea062728c81..0a0965a5ad0763b3823e6cf9a8b22e11a52a236c 100644
--- a/cc/trees/layer_tree_host_unittest.cc
+++ b/cc/trees/layer_tree_host_unittest.cc
@@ -2036,7 +2036,7 @@ TEST(LayerTreeHostTest, LimitPartialUpdates) {
settings.max_partial_texture_updates = 10;
LayerTreeHostWithProxy host(&client, settings, proxy.Pass());
- EXPECT_TRUE(host.InitializeOutputSurfaceIfNeeded());
+ host.OnCreateAndInitializeOutputSurfaceAttempted(true);
EXPECT_EQ(0u, host.MaxPartialTextureUpdates());
}
@@ -2054,7 +2054,7 @@ TEST(LayerTreeHostTest, LimitPartialUpdates) {
settings.max_partial_texture_updates = 10;
LayerTreeHostWithProxy host(&client, settings, proxy.Pass());
- EXPECT_TRUE(host.InitializeOutputSurfaceIfNeeded());
+ host.OnCreateAndInitializeOutputSurfaceAttempted(true);
EXPECT_EQ(5u, host.MaxPartialTextureUpdates());
}
@@ -2072,7 +2072,7 @@ TEST(LayerTreeHostTest, LimitPartialUpdates) {
settings.max_partial_texture_updates = 10;
LayerTreeHostWithProxy host(&client, settings, proxy.Pass());
- EXPECT_TRUE(host.InitializeOutputSurfaceIfNeeded());
+ host.OnCreateAndInitializeOutputSurfaceAttempted(true);
EXPECT_EQ(10u, host.MaxPartialTextureUpdates());
}
@@ -2088,7 +2088,8 @@ TEST(LayerTreeHostTest, PartialUpdatesWithGLRenderer) {
new TestSharedBitmapManager());
scoped_ptr<LayerTreeHost> host = LayerTreeHost::CreateSingleThreaded(
&client, &client, shared_bitmap_manager.get(), settings);
- EXPECT_TRUE(host->InitializeOutputSurfaceIfNeeded());
+ host->Composite(base::TimeTicks::Now());
+
EXPECT_EQ(4u, host->settings().max_partial_texture_updates);
}
@@ -2102,7 +2103,8 @@ TEST(LayerTreeHostTest, PartialUpdatesWithSoftwareRenderer) {
new TestSharedBitmapManager());
scoped_ptr<LayerTreeHost> host = LayerTreeHost::CreateSingleThreaded(
&client, &client, shared_bitmap_manager.get(), settings);
- EXPECT_TRUE(host->InitializeOutputSurfaceIfNeeded());
+ host->Composite(base::TimeTicks::Now());
+
EXPECT_EQ(4u, host->settings().max_partial_texture_updates);
}
@@ -2116,7 +2118,8 @@ TEST(LayerTreeHostTest, PartialUpdatesWithDelegatingRendererAndGLContent) {
new TestSharedBitmapManager());
scoped_ptr<LayerTreeHost> host = LayerTreeHost::CreateSingleThreaded(
&client, &client, shared_bitmap_manager.get(), settings);
- EXPECT_TRUE(host->InitializeOutputSurfaceIfNeeded());
+ host->Composite(base::TimeTicks::Now());
+
EXPECT_EQ(0u, host->MaxPartialTextureUpdates());
}
@@ -2131,7 +2134,8 @@ TEST(LayerTreeHostTest,
new TestSharedBitmapManager());
scoped_ptr<LayerTreeHost> host = LayerTreeHost::CreateSingleThreaded(
&client, &client, shared_bitmap_manager.get(), settings);
- EXPECT_TRUE(host->InitializeOutputSurfaceIfNeeded());
+ host->Composite(base::TimeTicks::Now());
+
EXPECT_EQ(0u, host->MaxPartialTextureUpdates());
}
« no previous file with comments | « cc/trees/layer_tree_host.cc ('k') | cc/trees/proxy.h » ('j') | cc/trees/single_thread_proxy.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698