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

Unified Diff: cc/benchmarks/rasterize_and_record_benchmark.cc

Issue 2851583002: cc: Update rasterize and record to capture more work in recording. (Closed)
Patch Set: rebase Created 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/benchmarks/rasterize_and_record_benchmark.cc
diff --git a/cc/benchmarks/rasterize_and_record_benchmark.cc b/cc/benchmarks/rasterize_and_record_benchmark.cc
index 6314fdef87ea4538b62ecf1123793a0b0b592fbd..c19dac1e6d3b4103603f6efc9eba6bc928496d46 100644
--- a/cc/benchmarks/rasterize_and_record_benchmark.cc
+++ b/cc/benchmarks/rasterize_and_record_benchmark.cc
@@ -139,6 +139,7 @@ void RasterizeAndRecordBenchmark::RunOnLayer(PictureLayer* layer) {
return;
ContentLayerClient* painter = layer->client();
+ RecordingSource recording_source;
for (int mode_index = 0; mode_index < RecordingSource::RECORDING_MODE_COUNT;
mode_index++) {
@@ -158,11 +159,8 @@ void RasterizeAndRecordBenchmark::RunOnLayer(PictureLayer* layer) {
do {
display_list = painter->PaintContentsToDisplayList(painting_control);
- if (display_list->ShouldBeAnalyzedForSolidColor()) {
- gfx::Size layer_size = layer->paint_properties().bounds;
- skia::AnalysisCanvas canvas(layer_size.width(), layer_size.height());
- display_list->Raster(&canvas, nullptr, gfx::Rect(layer_size), 1.f);
- }
+ recording_source.UpdateDisplayItemList(
+ display_list, painter->GetApproximateUnsharedMemoryUsage());
if (memory_used) {
// Verify we are recording the same thing each time.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698