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

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

Issue 2514033002: Introducing SurfaceReferenceFactory (Closed)
Patch Set: ref Created 4 years 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"
11 #include "base/single_thread_task_runner.h" 11 #include "base/single_thread_task_runner.h"
12 #include "base/threading/thread_task_runner_handle.h" 12 #include "base/threading/thread_task_runner_handle.h"
13 #include "cc/animation/animation_host.h" 13 #include "cc/animation/animation_host.h"
14 #include "cc/layers/solid_color_layer.h" 14 #include "cc/layers/solid_color_layer.h"
15 #include "cc/layers/surface_layer.h" 15 #include "cc/layers/surface_layer.h"
16 #include "cc/output/compositor_frame.h" 16 #include "cc/output/compositor_frame.h"
17 #include "cc/surfaces/surface_embedding.h"
17 #include "cc/test/fake_impl_task_runner_provider.h" 18 #include "cc/test/fake_impl_task_runner_provider.h"
18 #include "cc/test/fake_layer_tree_host.h" 19 #include "cc/test/fake_layer_tree_host.h"
19 #include "cc/test/fake_layer_tree_host_client.h" 20 #include "cc/test/fake_layer_tree_host_client.h"
20 #include "cc/test/fake_layer_tree_host_impl.h" 21 #include "cc/test/fake_layer_tree_host_impl.h"
21 #include "cc/test/fake_output_surface.h" 22 #include "cc/test/fake_output_surface.h"
22 #include "cc/test/layer_tree_test.h" 23 #include "cc/test/layer_tree_test.h"
23 #include "cc/test/test_task_graph_runner.h" 24 #include "cc/test/test_task_graph_runner.h"
24 #include "cc/trees/layer_tree_host.h" 25 #include "cc/trees/layer_tree_host.h"
25 #include "testing/gmock/include/gmock/gmock.h" 26 #include "testing/gmock/include/gmock/gmock.h"
26 #include "testing/gtest/include/gtest/gtest.h" 27 #include "testing/gtest/include/gtest/gtest.h"
(...skipping 20 matching lines...) Expand all
47 } 48 }
48 } 49 }
49 50
50 FakeLayerTreeHostClient fake_client_; 51 FakeLayerTreeHostClient fake_client_;
51 TestTaskGraphRunner task_graph_runner_; 52 TestTaskGraphRunner task_graph_runner_;
52 std::unique_ptr<AnimationHost> animation_host_; 53 std::unique_ptr<AnimationHost> animation_host_;
53 std::unique_ptr<FakeLayerTreeHost> layer_tree_host_; 54 std::unique_ptr<FakeLayerTreeHost> layer_tree_host_;
54 LayerTree* layer_tree_; 55 LayerTree* layer_tree_;
55 }; 56 };
56 57
57 void SatisfyCallback(SurfaceSequence* out, const SurfaceSequence& in) { 58 class TestSurfaceEmbedding : public SurfaceEmbeddingUsingSequence {
58 *out = in; 59 protected:
59 } 60 void SatisfySequence(const SurfaceSequence& seq) const override {
61 *out_seq_ = seq;
62 }
60 63
61 void RequireCallback(SurfaceId* out_id, 64 void RequireSequence(const SurfaceId& id,
62 std::set<SurfaceSequence>* out, 65 const SurfaceSequence& seq) const override {
63 const SurfaceId& in_id, 66 *out_id_ = id;
64 const SurfaceSequence& in) { 67 out_set_->insert(seq);
65 *out_id = in_id; 68 }
66 out->insert(in); 69
67 } 70 public:
71 TestSurfaceEmbedding(SurfaceSequence* out_seq,
72 SurfaceId* out_id,
73 std::set<SurfaceSequence>* out_set)
74 : out_seq_(out_seq), out_id_(out_id), out_set_(out_set) {}
75
76 protected:
77 ~TestSurfaceEmbedding() override = default;
78
79 private:
80 SurfaceSequence* out_seq_;
81 SurfaceId* out_id_;
82 std::set<SurfaceSequence>* out_set_;
83 };
68 84
69 // Check that one surface can be referenced by multiple LayerTreeHosts, and 85 // Check that one surface can be referenced by multiple LayerTreeHosts, and
70 // each will create its own SurfaceSequence that's satisfied on destruction. 86 // each will create its own SurfaceSequence that's satisfied on destruction.
71 TEST_F(SurfaceLayerTest, MultipleFramesOneSurface) { 87 TEST_F(SurfaceLayerTest, MultipleFramesOneSurface) {
72 const base::UnguessableToken kArbitraryToken = 88 const base::UnguessableToken kArbitraryToken =
73 base::UnguessableToken::Create(); 89 base::UnguessableToken::Create();
74 SurfaceSequence blank_change; // Receives sequence if commit doesn't happen. 90 SurfaceSequence blank_change; // Receives sequence if commit doesn't happen.
75 91
76 SurfaceId required_id; 92 SurfaceId required_id;
77 std::set<SurfaceSequence> required_seq; 93 std::set<SurfaceSequence> required_seq;
78 scoped_refptr<SurfaceLayer> layer(SurfaceLayer::Create( 94 SurfaceEmbeddingPtr surface_ref =
79 base::Bind(&SatisfyCallback, &blank_change), 95 new TestSurfaceEmbedding(&blank_change, &required_id, &required_seq);
80 base::Bind(&RequireCallback, &required_id, &required_seq))); 96 auto layer = SurfaceLayer::Create(surface_ref);
81 layer->SetSurfaceId( 97 SurfaceInfo info(
82 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(1, kArbitraryToken)), 1.f, 98 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(1, kArbitraryToken)), 1.f,
83 gfx::Size(1, 1)); 99 gfx::Size(1, 1));
100 layer->SetSurfaceInfo(info);
84 layer_tree_host_->GetSurfaceSequenceGenerator()->set_frame_sink_id( 101 layer_tree_host_->GetSurfaceSequenceGenerator()->set_frame_sink_id(
85 FrameSinkId(1, 1)); 102 FrameSinkId(1, 1));
86 layer_tree_->SetRootLayer(layer); 103 layer_tree_->SetRootLayer(layer);
87 104
88 auto animation_host2 = AnimationHost::CreateForTesting(ThreadInstance::MAIN); 105 auto animation_host2 = AnimationHost::CreateForTesting(ThreadInstance::MAIN);
89 std::unique_ptr<FakeLayerTreeHost> layer_tree_host2 = 106 std::unique_ptr<FakeLayerTreeHost> layer_tree_host2 =
90 FakeLayerTreeHost::Create(&fake_client_, &task_graph_runner_, 107 FakeLayerTreeHost::Create(&fake_client_, &task_graph_runner_,
91 animation_host2.get()); 108 animation_host2.get());
92 scoped_refptr<SurfaceLayer> layer2(SurfaceLayer::Create( 109 auto layer2 = SurfaceLayer::Create(std::move(surface_ref));
93 base::Bind(&SatisfyCallback, &blank_change), 110 layer2->SetSurfaceInfo(info);
94 base::Bind(&RequireCallback, &required_id, &required_seq)));
95 layer2->SetSurfaceId(
96 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(1, kArbitraryToken)), 1.f,
97 gfx::Size(1, 1));
98 layer_tree_host2->GetSurfaceSequenceGenerator()->set_frame_sink_id( 111 layer_tree_host2->GetSurfaceSequenceGenerator()->set_frame_sink_id(
99 FrameSinkId(2, 2)); 112 FrameSinkId(2, 2));
100 layer_tree_host2->SetRootLayer(layer2); 113 layer_tree_host2->SetRootLayer(layer2);
101 114
102 // Layers haven't been removed, so no sequence should be satisfied. 115 // Layers haven't been removed, so no sequence should be satisfied.
103 EXPECT_FALSE(blank_change.is_valid()); 116 EXPECT_FALSE(blank_change.is_valid());
104 117
105 SurfaceSequence expected1(FrameSinkId(1, 1), 1u); 118 SurfaceSequence expected1(FrameSinkId(1, 1), 1u);
106 SurfaceSequence expected2(FrameSinkId(2, 2), 1u); 119 SurfaceSequence expected2(FrameSinkId(2, 2), 1u);
107 120
(...skipping 27 matching lines...) Expand all
135 148
136 // Check that SurfaceSequence is sent through swap promise. 149 // Check that SurfaceSequence is sent through swap promise.
137 class SurfaceLayerSwapPromise : public LayerTreeTest { 150 class SurfaceLayerSwapPromise : public LayerTreeTest {
138 public: 151 public:
139 SurfaceLayerSwapPromise() 152 SurfaceLayerSwapPromise()
140 : commit_count_(0), sequence_was_satisfied_(false) {} 153 : commit_count_(0), sequence_was_satisfied_(false) {}
141 154
142 void BeginTest() override { 155 void BeginTest() override {
143 layer_tree_host()->GetSurfaceSequenceGenerator()->set_frame_sink_id( 156 layer_tree_host()->GetSurfaceSequenceGenerator()->set_frame_sink_id(
144 FrameSinkId(1, 1)); 157 FrameSinkId(1, 1));
145 layer_ = SurfaceLayer::Create( 158 SurfaceInfo info(
146 base::Bind(&SatisfyCallback, &satisfied_sequence_),
147 base::Bind(&RequireCallback, &required_id_, &required_set_));
148 layer_->SetSurfaceId(
149 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(1, kArbitraryToken)), 1.f, 159 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(1, kArbitraryToken)), 1.f,
150 gfx::Size(1, 1)); 160 gfx::Size(1, 1));
161 SurfaceEmbeddingPtr surface_ref = new TestSurfaceEmbedding(
162 &satisfied_sequence_, &required_id_, &required_set_);
163 layer_ = SurfaceLayer::Create(std::move(surface_ref));
164 layer_->SetSurfaceInfo(info);
151 165
152 // Layer hasn't been added to tree so no SurfaceSequence generated yet. 166 // Layer hasn't been added to tree so no SurfaceSequence generated yet.
153 EXPECT_EQ(0u, required_set_.size()); 167 EXPECT_EQ(0u, required_set_.size());
154 168
155 layer_tree()->SetRootLayer(layer_); 169 layer_tree()->SetRootLayer(layer_);
156 170
157 // Should have SurfaceSequence from first tree. 171 // Should have SurfaceSequence from first tree.
158 SurfaceSequence expected(kArbitraryFrameSinkId, 1u); 172 SurfaceSequence expected(kArbitraryFrameSinkId, 1u);
159 EXPECT_TRUE(required_id_ == SurfaceId(kArbitraryFrameSinkId, 173 EXPECT_TRUE(required_id_ == SurfaceId(kArbitraryFrameSinkId,
160 LocalFrameId(1, kArbitraryToken))); 174 LocalFrameId(1, kArbitraryToken)));
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after
272 // Sequence should have been satisfied with the callback. 286 // Sequence should have been satisfied with the callback.
273 EXPECT_TRUE(satisfied_sequence_ == 287 EXPECT_TRUE(satisfied_sequence_ ==
274 SurfaceSequence(kArbitraryFrameSinkId, 1u)); 288 SurfaceSequence(kArbitraryFrameSinkId, 1u));
275 } 289 }
276 }; 290 };
277 291
278 MULTI_THREAD_TEST_F(SurfaceLayerSwapPromiseWithoutDraw); 292 MULTI_THREAD_TEST_F(SurfaceLayerSwapPromiseWithoutDraw);
279 293
280 } // namespace 294 } // namespace
281 } // namespace cc 295 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698