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

Side by Side Diff: cc/layers/surface_layer_unittest.cc

Issue 2183403002: cc: Move data to LayerTree from LayerTreeHost. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@layer_tree_change
Patch Set: remove unused test file. Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stdint.h> 5 #include <stdint.h>
6 6
7 #include <set> 7 #include <set>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 15 matching lines...) Expand all
26 namespace cc { 26 namespace cc {
27 namespace { 27 namespace {
28 28
29 static constexpr uint32_t kArbitraryClientId = 0; 29 static constexpr uint32_t kArbitraryClientId = 0;
30 30
31 class SurfaceLayerTest : public testing::Test { 31 class SurfaceLayerTest : public testing::Test {
32 protected: 32 protected:
33 void SetUp() override { 33 void SetUp() override {
34 layer_tree_host_ = 34 layer_tree_host_ =
35 FakeLayerTreeHost::Create(&fake_client_, &task_graph_runner_); 35 FakeLayerTreeHost::Create(&fake_client_, &task_graph_runner_);
36 layer_tree_host_->SetViewportSize(gfx::Size(10, 10)); 36 layer_tree_ = layer_tree_host_->GetLayerTree();
37 layer_tree_->SetViewportSize(gfx::Size(10, 10));
37 } 38 }
38 39
39 void TearDown() override { 40 void TearDown() override {
40 if (layer_tree_host_) { 41 if (layer_tree_host_) {
41 layer_tree_host_->SetRootLayer(nullptr); 42 layer_tree_->SetRootLayer(nullptr);
42 layer_tree_host_ = nullptr; 43 layer_tree_host_ = nullptr;
43 } 44 }
44 } 45 }
45 46
46 FakeLayerTreeHostClient fake_client_; 47 FakeLayerTreeHostClient fake_client_;
47 TestTaskGraphRunner task_graph_runner_; 48 TestTaskGraphRunner task_graph_runner_;
48 std::unique_ptr<FakeLayerTreeHost> layer_tree_host_; 49 std::unique_ptr<FakeLayerTreeHost> layer_tree_host_;
50 LayerTree* layer_tree_;
49 }; 51 };
50 52
51 void SatisfyCallback(SurfaceSequence* out, const SurfaceSequence& in) { 53 void SatisfyCallback(SurfaceSequence* out, const SurfaceSequence& in) {
52 *out = in; 54 *out = in;
53 } 55 }
54 56
55 void RequireCallback(SurfaceId* out_id, 57 void RequireCallback(SurfaceId* out_id,
56 std::set<SurfaceSequence>* out, 58 std::set<SurfaceSequence>* out,
57 const SurfaceId& in_id, 59 const SurfaceId& in_id,
58 const SurfaceSequence& in) { 60 const SurfaceSequence& in) {
59 *out_id = in_id; 61 *out_id = in_id;
60 out->insert(in); 62 out->insert(in);
61 } 63 }
62 64
63 // Check that one surface can be referenced by multiple LayerTreeHosts, and 65 // Check that one surface can be referenced by multiple LayerTreeHosts, and
64 // each will create its own SurfaceSequence that's satisfied on destruction. 66 // each will create its own SurfaceSequence that's satisfied on destruction.
65 TEST_F(SurfaceLayerTest, MultipleFramesOneSurface) { 67 TEST_F(SurfaceLayerTest, MultipleFramesOneSurface) {
66 SurfaceSequence blank_change; // Receives sequence if commit doesn't happen. 68 SurfaceSequence blank_change; // Receives sequence if commit doesn't happen.
67 69
68 SurfaceId required_id; 70 SurfaceId required_id;
69 std::set<SurfaceSequence> required_seq; 71 std::set<SurfaceSequence> required_seq;
70 scoped_refptr<SurfaceLayer> layer(SurfaceLayer::Create( 72 scoped_refptr<SurfaceLayer> layer(SurfaceLayer::Create(
71 base::Bind(&SatisfyCallback, &blank_change), 73 base::Bind(&SatisfyCallback, &blank_change),
72 base::Bind(&RequireCallback, &required_id, &required_seq))); 74 base::Bind(&RequireCallback, &required_id, &required_seq)));
73 layer->SetSurfaceId(SurfaceId(kArbitraryClientId, 1, 0), 1.f, 75 layer->SetSurfaceId(SurfaceId(kArbitraryClientId, 1, 0), 1.f,
74 gfx::Size(1, 1)); 76 gfx::Size(1, 1));
75 layer_tree_host_->set_surface_client_id(1); 77 layer_tree_host_->set_surface_client_id(1);
76 layer_tree_host_->SetRootLayer(layer); 78 layer_tree_->SetRootLayer(layer);
77 79
78 std::unique_ptr<FakeLayerTreeHost> layer_tree_host2 = 80 std::unique_ptr<FakeLayerTreeHost> layer_tree_host2 =
79 FakeLayerTreeHost::Create(&fake_client_, &task_graph_runner_); 81 FakeLayerTreeHost::Create(&fake_client_, &task_graph_runner_);
80 scoped_refptr<SurfaceLayer> layer2(SurfaceLayer::Create( 82 scoped_refptr<SurfaceLayer> layer2(SurfaceLayer::Create(
81 base::Bind(&SatisfyCallback, &blank_change), 83 base::Bind(&SatisfyCallback, &blank_change),
82 base::Bind(&RequireCallback, &required_id, &required_seq))); 84 base::Bind(&RequireCallback, &required_id, &required_seq)));
83 layer2->SetSurfaceId(SurfaceId(kArbitraryClientId, 1, 0), 1.f, 85 layer2->SetSurfaceId(SurfaceId(kArbitraryClientId, 1, 0), 1.f,
84 gfx::Size(1, 1)); 86 gfx::Size(1, 1));
85 layer_tree_host2->set_surface_client_id(2); 87 layer_tree_host2->set_surface_client_id(2);
86 layer_tree_host2->SetRootLayer(layer2); 88 layer_tree_host2->SetRootLayer(layer2);
(...skipping 11 matching lines...) Expand all
98 // satisfied. 100 // satisfied.
99 EXPECT_TRUE(blank_change == expected2); 101 EXPECT_TRUE(blank_change == expected2);
100 102
101 // Set of sequences that need to be satisfied should include sequences from 103 // Set of sequences that need to be satisfied should include sequences from
102 // both trees. 104 // both trees.
103 EXPECT_TRUE(required_id == SurfaceId(kArbitraryClientId, 1, 0)); 105 EXPECT_TRUE(required_id == SurfaceId(kArbitraryClientId, 1, 0));
104 EXPECT_EQ(2u, required_seq.size()); 106 EXPECT_EQ(2u, required_seq.size());
105 EXPECT_TRUE(required_seq.count(expected1)); 107 EXPECT_TRUE(required_seq.count(expected1));
106 EXPECT_TRUE(required_seq.count(expected2)); 108 EXPECT_TRUE(required_seq.count(expected2));
107 109
108 layer_tree_host_->SetRootLayer(nullptr); 110 layer_tree_->SetRootLayer(nullptr);
109 layer_tree_host_.reset(); 111 layer_tree_host_.reset();
110 112
111 // Layer was removed so sequence from first LayerTreeHost should be 113 // Layer was removed so sequence from first LayerTreeHost should be
112 // satisfied. 114 // satisfied.
113 EXPECT_TRUE(blank_change == expected1); 115 EXPECT_TRUE(blank_change == expected1);
114 116
115 // No more SurfaceSequences should have been generated that need to have be 117 // No more SurfaceSequences should have been generated that need to have be
116 // satisfied. 118 // satisfied.
117 EXPECT_EQ(2u, required_seq.size()); 119 EXPECT_EQ(2u, required_seq.size());
118 } 120 }
119 121
120 // Check that SurfaceSequence is sent through swap promise. 122 // Check that SurfaceSequence is sent through swap promise.
121 class SurfaceLayerSwapPromise : public LayerTreeTest { 123 class SurfaceLayerSwapPromise : public LayerTreeTest {
122 public: 124 public:
123 SurfaceLayerSwapPromise() 125 SurfaceLayerSwapPromise()
124 : commit_count_(0), sequence_was_satisfied_(false) {} 126 : commit_count_(0), sequence_was_satisfied_(false) {}
125 127
126 void BeginTest() override { 128 void BeginTest() override {
127 layer_tree_host()->set_surface_client_id(1); 129 layer_tree_host()->set_surface_client_id(1);
128 layer_ = SurfaceLayer::Create( 130 layer_ = SurfaceLayer::Create(
129 base::Bind(&SatisfyCallback, &satisfied_sequence_), 131 base::Bind(&SatisfyCallback, &satisfied_sequence_),
130 base::Bind(&RequireCallback, &required_id_, &required_set_)); 132 base::Bind(&RequireCallback, &required_id_, &required_set_));
131 layer_->SetSurfaceId(SurfaceId(kArbitraryClientId, 1, 0), 1.f, 133 layer_->SetSurfaceId(SurfaceId(kArbitraryClientId, 1, 0), 1.f,
132 gfx::Size(1, 1)); 134 gfx::Size(1, 1));
133 135
134 // Layer hasn't been added to tree so no SurfaceSequence generated yet. 136 // Layer hasn't been added to tree so no SurfaceSequence generated yet.
135 EXPECT_EQ(0u, required_set_.size()); 137 EXPECT_EQ(0u, required_set_.size());
136 138
137 layer_tree_host()->SetRootLayer(layer_); 139 layer_tree()->SetRootLayer(layer_);
138 140
139 // Should have SurfaceSequence from first tree. 141 // Should have SurfaceSequence from first tree.
140 SurfaceSequence expected(1u, 1u); 142 SurfaceSequence expected(1u, 1u);
141 EXPECT_TRUE(required_id_ == SurfaceId(kArbitraryClientId, 1, 0)); 143 EXPECT_TRUE(required_id_ == SurfaceId(kArbitraryClientId, 1, 0));
142 EXPECT_EQ(1u, required_set_.size()); 144 EXPECT_EQ(1u, required_set_.size());
143 EXPECT_TRUE(required_set_.count(expected)); 145 EXPECT_TRUE(required_set_.count(expected));
144 146
145 gfx::Size bounds(100, 100); 147 gfx::Size bounds(100, 100);
146 layer_tree_host()->SetViewportSize(bounds); 148 layer_tree()->SetViewportSize(bounds);
147 149
148 blank_layer_ = SolidColorLayer::Create(); 150 blank_layer_ = SolidColorLayer::Create();
149 blank_layer_->SetIsDrawable(true); 151 blank_layer_->SetIsDrawable(true);
150 blank_layer_->SetBounds(gfx::Size(10, 10)); 152 blank_layer_->SetBounds(gfx::Size(10, 10));
151 153
152 PostSetNeedsCommitToMainThread(); 154 PostSetNeedsCommitToMainThread();
153 } 155 }
154 156
155 virtual void ChangeTree() = 0; 157 virtual void ChangeTree() = 0;
156 158
(...skipping 15 matching lines...) Expand all
172 }; 174 };
173 175
174 // Check that SurfaceSequence is sent through swap promise. 176 // Check that SurfaceSequence is sent through swap promise.
175 class SurfaceLayerSwapPromiseWithDraw : public SurfaceLayerSwapPromise { 177 class SurfaceLayerSwapPromiseWithDraw : public SurfaceLayerSwapPromise {
176 public: 178 public:
177 void ChangeTree() override { 179 void ChangeTree() override {
178 ++commit_count_; 180 ++commit_count_;
179 switch (commit_count_) { 181 switch (commit_count_) {
180 case 1: 182 case 1:
181 // Remove SurfaceLayer from tree to cause SwapPromise to be created. 183 // Remove SurfaceLayer from tree to cause SwapPromise to be created.
182 layer_tree_host()->SetRootLayer(blank_layer_); 184 layer_tree()->SetRootLayer(blank_layer_);
183 break; 185 break;
184 case 2: 186 case 2:
185 break; 187 break;
186 default: 188 default:
187 NOTREACHED(); 189 NOTREACHED();
188 break; 190 break;
189 } 191 }
190 } 192 }
191 193
192 void DisplayReceivedCompositorFrameOnThread( 194 void DisplayReceivedCompositorFrameOnThread(
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
225 LayerTreeHostImpl::FrameData* frame, 227 LayerTreeHostImpl::FrameData* frame,
226 DrawResult draw_result) override { 228 DrawResult draw_result) override {
227 return DRAW_ABORTED_MISSING_HIGH_RES_CONTENT; 229 return DRAW_ABORTED_MISSING_HIGH_RES_CONTENT;
228 } 230 }
229 231
230 void ChangeTree() override { 232 void ChangeTree() override {
231 ++commit_count_; 233 ++commit_count_;
232 switch (commit_count_) { 234 switch (commit_count_) {
233 case 1: 235 case 1:
234 // Remove SurfaceLayer from tree to cause SwapPromise to be created. 236 // Remove SurfaceLayer from tree to cause SwapPromise to be created.
235 layer_tree_host()->SetRootLayer(blank_layer_); 237 layer_tree()->SetRootLayer(blank_layer_);
236 break; 238 break;
237 case 2: 239 case 2:
238 layer_tree_host()->SetNeedsCommit(); 240 layer_tree_host()->SetNeedsCommit();
239 break; 241 break;
240 default: 242 default:
241 EndTest(); 243 EndTest();
242 break; 244 break;
243 } 245 }
244 } 246 }
245 247
246 void AfterTest() override { 248 void AfterTest() override {
247 EXPECT_TRUE(required_id_ == SurfaceId(kArbitraryClientId, 1, 0)); 249 EXPECT_TRUE(required_id_ == SurfaceId(kArbitraryClientId, 1, 0));
248 EXPECT_EQ(1u, required_set_.size()); 250 EXPECT_EQ(1u, required_set_.size());
249 // Sequence should have been satisfied with the callback. 251 // Sequence should have been satisfied with the callback.
250 EXPECT_TRUE(satisfied_sequence_ == SurfaceSequence(1u, 1u)); 252 EXPECT_TRUE(satisfied_sequence_ == SurfaceSequence(1u, 1u));
251 } 253 }
252 }; 254 };
253 255
254 MULTI_THREAD_TEST_F(SurfaceLayerSwapPromiseWithoutDraw); 256 MULTI_THREAD_TEST_F(SurfaceLayerSwapPromiseWithoutDraw);
255 257
256 } // namespace 258 } // namespace
257 } // namespace cc 259 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698