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

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

Issue 1196553009: Revert "cc: Make tiling interest rect calc based on viewport." (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: revert extra bools. Created 5 years, 6 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
« no previous file with comments | « cc/base/switches.cc ('k') | cc/layers/picture_layer_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 <algorithm> 7 #include <algorithm>
8 #include <limits> 8 #include <limits>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 } 164 }
165 165
166 FixedInvalidationPictureLayerTilingClient client(layer, 166 FixedInvalidationPictureLayerTilingClient client(layer,
167 gfx::Rect(layer->bounds())); 167 gfx::Rect(layer->bounds()));
168 168
169 // In this benchmark, we will create a local tiling set and measure how long 169 // In this benchmark, we will create a local tiling set and measure how long
170 // it takes to rasterize content. As such, the actual settings used here don't 170 // it takes to rasterize content. As such, the actual settings used here don't
171 // really matter. 171 // really matter.
172 const LayerTreeSettings& settings = layer->layer_tree_impl()->settings(); 172 const LayerTreeSettings& settings = layer->layer_tree_impl()->settings();
173 scoped_ptr<PictureLayerTilingSet> tiling_set = PictureLayerTilingSet::Create( 173 scoped_ptr<PictureLayerTilingSet> tiling_set = PictureLayerTilingSet::Create(
174 layer->GetTree(), &client, 174 layer->GetTree(), &client, settings.max_tiles_for_interest_area,
175 settings.tiling_interest_area_viewport_multiplier,
176 settings.skewport_target_time_in_seconds, 175 settings.skewport_target_time_in_seconds,
177 settings.skewport_extrapolation_limit_in_content_pixels); 176 settings.skewport_extrapolation_limit_in_content_pixels);
178 177
179 PictureLayerTiling* tiling = 178 PictureLayerTiling* tiling =
180 tiling_set->AddTiling(1.f, layer->GetRasterSource()); 179 tiling_set->AddTiling(1.f, layer->GetRasterSource());
181 tiling->CreateAllTilesForTesting(); 180 tiling->CreateAllTilesForTesting();
182 RasterSource* raster_source = tiling->raster_source(); 181 RasterSource* raster_source = tiling->raster_source();
183 for (PictureLayerTiling::CoverageIterator it(tiling, 1.f, 182 for (PictureLayerTiling::CoverageIterator it(tiling, 1.f,
184 layer->visible_layer_rect()); 183 layer->visible_layer_rect());
185 it; ++it) { 184 it; ++it) {
(...skipping 30 matching lines...) Expand all
216 total_memory_usage(0), 215 total_memory_usage(0),
217 total_layers(0), 216 total_layers(0),
218 total_picture_layers(0), 217 total_picture_layers(0),
219 total_picture_layers_with_no_content(0), 218 total_picture_layers_with_no_content(0),
220 total_picture_layers_off_screen(0) { 219 total_picture_layers_off_screen(0) {
221 } 220 }
222 221
223 RasterizeAndRecordBenchmarkImpl::RasterizeResults::~RasterizeResults() {} 222 RasterizeAndRecordBenchmarkImpl::RasterizeResults::~RasterizeResults() {}
224 223
225 } // namespace cc 224 } // namespace cc
OLDNEW
« no previous file with comments | « cc/base/switches.cc ('k') | cc/layers/picture_layer_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698