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

Side by Side Diff: cc/surfaces/surface_aggregator_perftest.cc

Issue 2449853004: Getting rid of DelegatedFrameData (Closed)
Patch Set: IsEmpty + rebase Created 4 years, 1 month 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "base/memory/ptr_util.h" 5 #include "base/memory/ptr_util.h"
6 #include "cc/debug/lap_timer.h" 6 #include "cc/debug/lap_timer.h"
7 #include "cc/output/compositor_frame.h" 7 #include "cc/output/compositor_frame.h"
8 #include "cc/output/delegated_frame_data.h"
9 #include "cc/quads/surface_draw_quad.h" 8 #include "cc/quads/surface_draw_quad.h"
10 #include "cc/quads/texture_draw_quad.h" 9 #include "cc/quads/texture_draw_quad.h"
11 #include "cc/surfaces/surface_aggregator.h" 10 #include "cc/surfaces/surface_aggregator.h"
12 #include "cc/surfaces/surface_factory.h" 11 #include "cc/surfaces/surface_factory.h"
13 #include "cc/surfaces/surface_factory_client.h" 12 #include "cc/surfaces/surface_factory_client.h"
14 #include "cc/surfaces/surface_manager.h" 13 #include "cc/surfaces/surface_manager.h"
15 #include "cc/test/fake_output_surface_client.h" 14 #include "cc/test/fake_output_surface_client.h"
16 #include "cc/test/fake_resource_provider.h" 15 #include "cc/test/fake_resource_provider.h"
17 #include "cc/test/test_context_provider.h" 16 #include "cc/test/test_context_provider.h"
18 #include "cc/test/test_shared_bitmap_manager.h" 17 #include "cc/test/test_shared_bitmap_manager.h"
(...skipping 28 matching lines...) Expand all
47 float opacity, 46 float opacity,
48 bool optimize_damage, 47 bool optimize_damage,
49 bool full_damage, 48 bool full_damage,
50 const std::string& name) { 49 const std::string& name) {
51 aggregator_.reset(new SurfaceAggregator(&manager_, resource_provider_.get(), 50 aggregator_.reset(new SurfaceAggregator(&manager_, resource_provider_.get(),
52 optimize_damage)); 51 optimize_damage));
53 for (int i = 1; i <= num_surfaces; i++) { 52 for (int i = 1; i <= num_surfaces; i++) {
54 LocalFrameId local_frame_id(i, 0); 53 LocalFrameId local_frame_id(i, 0);
55 factory_.Create(local_frame_id); 54 factory_.Create(local_frame_id);
56 std::unique_ptr<RenderPass> pass(RenderPass::Create()); 55 std::unique_ptr<RenderPass> pass(RenderPass::Create());
57 std::unique_ptr<DelegatedFrameData> frame_data(new DelegatedFrameData); 56 CompositorFrame frame;
58 57
59 SharedQuadState* sqs = pass->CreateAndAppendSharedQuadState(); 58 SharedQuadState* sqs = pass->CreateAndAppendSharedQuadState();
60 for (int j = 0; j < num_textures; j++) { 59 for (int j = 0; j < num_textures; j++) {
61 TransferableResource resource; 60 TransferableResource resource;
62 resource.id = j; 61 resource.id = j;
63 resource.is_software = true; 62 resource.is_software = true;
64 frame_data->resource_list.push_back(resource); 63 frame.resource_list.push_back(resource);
65 64
66 TextureDrawQuad* quad = 65 TextureDrawQuad* quad =
67 pass->CreateAndAppendDrawQuad<TextureDrawQuad>(); 66 pass->CreateAndAppendDrawQuad<TextureDrawQuad>();
68 const gfx::Rect rect(0, 0, 1, 1); 67 const gfx::Rect rect(0, 0, 1, 1);
69 const gfx::Rect opaque_rect; 68 const gfx::Rect opaque_rect;
70 // Half of rects should be visible with partial damage. 69 // Half of rects should be visible with partial damage.
71 gfx::Rect visible_rect = 70 gfx::Rect visible_rect =
72 j % 2 == 0 ? gfx::Rect(0, 0, 1, 1) : gfx::Rect(1, 1, 1, 1); 71 j % 2 == 0 ? gfx::Rect(0, 0, 1, 1) : gfx::Rect(1, 1, 1, 1);
73 bool needs_blending = false; 72 bool needs_blending = false;
74 bool premultiplied_alpha = false; 73 bool premultiplied_alpha = false;
(...skipping 11 matching lines...) Expand all
86 sqs = pass->CreateAndAppendSharedQuadState(); 85 sqs = pass->CreateAndAppendSharedQuadState();
87 sqs->opacity = opacity; 86 sqs->opacity = opacity;
88 if (i > 1) { 87 if (i > 1) {
89 SurfaceDrawQuad* surface_quad = 88 SurfaceDrawQuad* surface_quad =
90 pass->CreateAndAppendDrawQuad<SurfaceDrawQuad>(); 89 pass->CreateAndAppendDrawQuad<SurfaceDrawQuad>();
91 surface_quad->SetNew( 90 surface_quad->SetNew(
92 sqs, gfx::Rect(0, 0, 1, 1), gfx::Rect(0, 0, 1, 1), 91 sqs, gfx::Rect(0, 0, 1, 1), gfx::Rect(0, 0, 1, 1),
93 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(i - 1, 0))); 92 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(i - 1, 0)));
94 } 93 }
95 94
96 frame_data->render_pass_list.push_back(std::move(pass)); 95 frame.render_pass_list.push_back(std::move(pass));
97 CompositorFrame frame;
98 frame.delegated_frame_data = std::move(frame_data);
99 factory_.SubmitCompositorFrame(local_frame_id, std::move(frame), 96 factory_.SubmitCompositorFrame(local_frame_id, std::move(frame),
100 SurfaceFactory::DrawCallback()); 97 SurfaceFactory::DrawCallback());
101 } 98 }
102 99
103 factory_.Create(LocalFrameId(num_surfaces + 1, 0)); 100 factory_.Create(LocalFrameId(num_surfaces + 1, 0));
104 timer_.Reset(); 101 timer_.Reset();
105 do { 102 do {
106 std::unique_ptr<RenderPass> pass(RenderPass::Create()); 103 std::unique_ptr<RenderPass> pass(RenderPass::Create());
107 std::unique_ptr<DelegatedFrameData> frame_data(new DelegatedFrameData); 104 CompositorFrame frame;
108 105
109 SharedQuadState* sqs = pass->CreateAndAppendSharedQuadState(); 106 SharedQuadState* sqs = pass->CreateAndAppendSharedQuadState();
110 SurfaceDrawQuad* surface_quad = 107 SurfaceDrawQuad* surface_quad =
111 pass->CreateAndAppendDrawQuad<SurfaceDrawQuad>(); 108 pass->CreateAndAppendDrawQuad<SurfaceDrawQuad>();
112 surface_quad->SetNew( 109 surface_quad->SetNew(
113 sqs, gfx::Rect(0, 0, 100, 100), gfx::Rect(0, 0, 100, 100), 110 sqs, gfx::Rect(0, 0, 100, 100), gfx::Rect(0, 0, 100, 100),
114 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(num_surfaces, 0))); 111 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(num_surfaces, 0)));
115 112
116 if (full_damage) 113 if (full_damage)
117 pass->damage_rect = gfx::Rect(0, 0, 100, 100); 114 pass->damage_rect = gfx::Rect(0, 0, 100, 100);
118 else 115 else
119 pass->damage_rect = gfx::Rect(0, 0, 1, 1); 116 pass->damage_rect = gfx::Rect(0, 0, 1, 1);
120 117
121 frame_data->render_pass_list.push_back(std::move(pass)); 118 frame.render_pass_list.push_back(std::move(pass));
122 CompositorFrame frame;
123 frame.delegated_frame_data = std::move(frame_data);
124 factory_.SubmitCompositorFrame(LocalFrameId(num_surfaces + 1, 0), 119 factory_.SubmitCompositorFrame(LocalFrameId(num_surfaces + 1, 0),
125 std::move(frame), 120 std::move(frame),
126 SurfaceFactory::DrawCallback()); 121 SurfaceFactory::DrawCallback());
127 122
128 CompositorFrame aggregated = aggregator_->Aggregate( 123 CompositorFrame aggregated = aggregator_->Aggregate(
129 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(num_surfaces + 1, 0))); 124 SurfaceId(kArbitraryFrameSinkId, LocalFrameId(num_surfaces + 1, 0)));
130 timer_.NextLap(); 125 timer_.NextLap();
131 } while (!timer_.HasTimeLimitExpired()); 126 } while (!timer_.HasTimeLimitExpired());
132 127
133 perf_test::PrintResult("aggregator_speed", "", name, timer_.LapsPerSecond(), 128 perf_test::PrintResult("aggregator_speed", "", name, timer_.LapsPerSecond(),
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 TEST_F(SurfaceAggregatorPerfTest, FewSurfacesDamageCalc) { 167 TEST_F(SurfaceAggregatorPerfTest, FewSurfacesDamageCalc) {
173 RunTest(3, 1000, 1.f, true, true, "few_surfaces_damage_calc"); 168 RunTest(3, 1000, 1.f, true, true, "few_surfaces_damage_calc");
174 } 169 }
175 170
176 TEST_F(SurfaceAggregatorPerfTest, FewSurfacesAggregateDamaged) { 171 TEST_F(SurfaceAggregatorPerfTest, FewSurfacesAggregateDamaged) {
177 RunTest(3, 1000, 1.f, true, false, "few_surfaces_aggregate_damaged"); 172 RunTest(3, 1000, 1.f, true, false, "few_surfaces_aggregate_damaged");
178 } 173 }
179 174
180 } // namespace 175 } // namespace
181 } // namespace cc 176 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698