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

Side by Side Diff: cc/debug/rasterize_and_record_benchmark_impl.cc

Issue 1907053004: cc: Make CallFunctionForEveryLayer use LayerListIterator (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Recover CallFunctionForEveryLayer Created 4 years, 8 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "cc/debug/rasterize_and_record_benchmark_impl.h" 5 #include "cc/debug/rasterize_and_record_benchmark_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <limits> 10 #include <limits>
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 119
120 if (settings->HasKey("rasterize_repeat_count")) 120 if (settings->HasKey("rasterize_repeat_count"))
121 settings->GetInteger("rasterize_repeat_count", &rasterize_repeat_count_); 121 settings->GetInteger("rasterize_repeat_count", &rasterize_repeat_count_);
122 } 122 }
123 123
124 RasterizeAndRecordBenchmarkImpl::~RasterizeAndRecordBenchmarkImpl() {} 124 RasterizeAndRecordBenchmarkImpl::~RasterizeAndRecordBenchmarkImpl() {}
125 125
126 void RasterizeAndRecordBenchmarkImpl::DidCompleteCommit( 126 void RasterizeAndRecordBenchmarkImpl::DidCompleteCommit(
127 LayerTreeHostImpl* host) { 127 LayerTreeHostImpl* host) {
128 LayerTreeHostCommon::CallFunctionForEveryLayer( 128 LayerTreeHostCommon::CallFunctionForEveryLayer(
129 host->active_tree(), 129 host->active_tree(), [this](LayerImpl* layer) {
130 [this](LayerImpl* layer) {
131 rasterize_results_.total_layers++; 130 rasterize_results_.total_layers++;
132 layer->RunMicroBenchmark(this); 131 layer->RunMicroBenchmark(this);
133 }, 132 });
134 CallFunctionLayerType::ALL_LAYERS);
135 133
136 std::unique_ptr<base::DictionaryValue> result(new base::DictionaryValue()); 134 std::unique_ptr<base::DictionaryValue> result(new base::DictionaryValue());
137 result->SetDouble("rasterize_time_ms", 135 result->SetDouble("rasterize_time_ms",
138 rasterize_results_.total_best_time.InMillisecondsF()); 136 rasterize_results_.total_best_time.InMillisecondsF());
139 result->SetDouble("total_pictures_in_pile_size", 137 result->SetDouble("total_pictures_in_pile_size",
140 static_cast<int>(rasterize_results_.total_memory_usage)); 138 static_cast<int>(rasterize_results_.total_memory_usage));
141 result->SetInteger("pixels_rasterized", rasterize_results_.pixels_rasterized); 139 result->SetInteger("pixels_rasterized", rasterize_results_.pixels_rasterized);
142 result->SetInteger("pixels_rasterized_with_non_solid_color", 140 result->SetInteger("pixels_rasterized_with_non_solid_color",
143 rasterize_results_.pixels_rasterized_with_non_solid_color); 141 rasterize_results_.pixels_rasterized_with_non_solid_color);
144 result->SetInteger("pixels_rasterized_as_opaque", 142 result->SetInteger("pixels_rasterized_as_opaque",
(...skipping 74 matching lines...) Expand 10 before | Expand all | Expand 10 after
219 total_memory_usage(0), 217 total_memory_usage(0),
220 total_layers(0), 218 total_layers(0),
221 total_picture_layers(0), 219 total_picture_layers(0),
222 total_picture_layers_with_no_content(0), 220 total_picture_layers_with_no_content(0),
223 total_picture_layers_off_screen(0) { 221 total_picture_layers_off_screen(0) {
224 } 222 }
225 223
226 RasterizeAndRecordBenchmarkImpl::RasterizeResults::~RasterizeResults() {} 224 RasterizeAndRecordBenchmarkImpl::RasterizeResults::~RasterizeResults() {}
227 225
228 } // namespace cc 226 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698