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

Unified Diff: cc/layers/layer_unittest.cc

Issue 1126253005: cc: Add LayerTreeHost::InitParams for LayerTreeHost creation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 5 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 | « android_webview/browser/hardware_renderer.cc ('k') | cc/layers/picture_layer_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/layer_unittest.cc
diff --git a/cc/layers/layer_unittest.cc b/cc/layers/layer_unittest.cc
index bae8185053fe6a9123df9667313e584a4cce0bed..6b4d36b941207b5829eb44438ebc6a5f23440b72 100644
--- a/cc/layers/layer_unittest.cc
+++ b/cc/layers/layer_unittest.cc
@@ -44,10 +44,11 @@ namespace {
class MockLayerTreeHost : public LayerTreeHost {
public:
- explicit MockLayerTreeHost(FakeLayerTreeHostClient* client)
- : LayerTreeHost(client, nullptr, nullptr, nullptr, LayerTreeSettings()) {
- InitializeSingleThreaded(client, base::ThreadTaskRunnerHandle::Get(),
- nullptr);
+ MockLayerTreeHost(LayerTreeHostSingleThreadClient* single_thread_client,
+ LayerTreeHost::InitParams* params)
+ : LayerTreeHost(params) {
+ InitializeSingleThreaded(single_thread_client,
+ base::ThreadTaskRunnerHandle::Get(), nullptr);
}
MOCK_METHOD0(SetNeedsCommit, void());
@@ -68,7 +69,12 @@ class LayerTest : public testing::Test {
protected:
void SetUp() override {
- layer_tree_host_.reset(new StrictMock<MockLayerTreeHost>(&fake_client_));
+ LayerTreeHost::InitParams params;
+ LayerTreeSettings settings;
+ params.client = &fake_client_;
+ params.settings = &settings;
+ layer_tree_host_.reset(
+ new StrictMock<MockLayerTreeHost>(&fake_client_, &params));
}
void TearDown() override {
@@ -937,18 +943,16 @@ class LayerTreeHostFactory {
shared_bitmap_manager_(new TestSharedBitmapManager),
gpu_memory_buffer_manager_(new TestGpuMemoryBufferManager) {}
- scoped_ptr<LayerTreeHost> Create() {
- return LayerTreeHost::CreateSingleThreaded(
- &client_, &client_, shared_bitmap_manager_.get(),
- gpu_memory_buffer_manager_.get(), nullptr, LayerTreeSettings(),
- base::ThreadTaskRunnerHandle::Get(), nullptr);
- }
+ scoped_ptr<LayerTreeHost> Create() { return Create(LayerTreeSettings()); }
scoped_ptr<LayerTreeHost> Create(LayerTreeSettings settings) {
- return LayerTreeHost::CreateSingleThreaded(
- &client_, &client_, shared_bitmap_manager_.get(),
- gpu_memory_buffer_manager_.get(), nullptr, settings,
- base::ThreadTaskRunnerHandle::Get(), nullptr);
+ LayerTreeHost::InitParams params;
+ params.client = &client_;
+ params.shared_bitmap_manager = shared_bitmap_manager_.get();
+ params.gpu_memory_buffer_manager = gpu_memory_buffer_manager_.get();
+ params.settings = &settings;
+ params.main_task_runner = base::ThreadTaskRunnerHandle::Get();
+ return LayerTreeHost::CreateSingleThreaded(&client_, &params);
}
private:
« no previous file with comments | « android_webview/browser/hardware_renderer.cc ('k') | cc/layers/picture_layer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698