diff options
author | scottmg@chromium.org <scottmg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-04 23:42:56 +0000 |
---|---|---|
committer | scottmg@chromium.org <scottmg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-04 23:42:56 +0000 |
commit | be6dedf67bfdaf94c64e3a853e8491582905dd19 (patch) | |
tree | cccc95d45169effea10a2e8b84237200771b9da9 /cc/test/fake_content_layer_client.cc | |
parent | d3ba08d9ead26a59ae5503ca7f175c5a6046f2cc (diff) | |
download | chromium_src-be6dedf67bfdaf94c64e3a853e8491582905dd19.zip chromium_src-be6dedf67bfdaf94c64e3a853e8491582905dd19.tar.gz chromium_src-be6dedf67bfdaf94c64e3a853e8491582905dd19.tar.bz2 |
Revert 261906 "cc: Add picture recording modes for benchmarking"
Suspected of causing gcm_unit_tests failures
http://build.chromium.org/p/chromium.mac/buildstatus?builder=Mac10.7%20Tests%20%282%29&number=25118
Will file a bug that gcm_unit_tests isn't in default tryjob set if this is actually the cause.
> cc: Add picture recording modes for benchmarking
>
> This patch adds three different picture recording modes for benchmarking
> purposes:
>
> 1. RECORD_NORMALLY. This is the default and produces a normal recording
> as before.
>
> 2. RECORD_WITH_NULL_CANVAS. This mode uses SkNullCanvas to avoid (most
> of) the Skia processing that happen during recording. Can be used to
> estimate the recording workload inside Blink.
>
> 3. RECORD_WITH_PAINTING_DISABLED. Passes a NULL canvas to Blink, which
> causes the GraphicsContext object to disable painting. This mode
> gives an estimate of the recording workload of Blink excluding the
> processing inside GraphicsContext.
>
> This patch also modifies the rasterize_and_record_micro benchmark to
> report the recording time for each mode (record_time_ms,
> record_time_null_canvas_ms and record_time_painting_disabled_ms
> respectively).
>
> Anecdotally on a z600 and the key_silk_cases page set, SkNullCanvas cuts
> recording time to 50% and disabling painting down to 19%.
>
> BUG=357572
> TEST=PictureTest.RecordingModes
> TEST=tools/perf/run_benchmark rasterize_and_record_micro.key_silk_cases --browser=release
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=261543
>
> Review URL: https://codereview.chromium.org/216933002
TBR=skyostil@chromium.org
Review URL: https://codereview.chromium.org/226083007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@261914 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/test/fake_content_layer_client.cc')
-rw-r--r-- | cc/test/fake_content_layer_client.cc | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/cc/test/fake_content_layer_client.cc b/cc/test/fake_content_layer_client.cc index 3996654..2d1c304 100644 --- a/cc/test/fake_content_layer_client.cc +++ b/cc/test/fake_content_layer_client.cc @@ -10,17 +10,14 @@ namespace cc { FakeContentLayerClient::FakeContentLayerClient() - : paint_all_opaque_(false), last_canvas_(NULL) {} + : paint_all_opaque_(false) { +} FakeContentLayerClient::~FakeContentLayerClient() { } void FakeContentLayerClient::PaintContents(SkCanvas* canvas, const gfx::Rect& paint_rect, gfx::RectF* opaque_rect) { - last_canvas_ = canvas; - if (!canvas) - return; - if (paint_all_opaque_) *opaque_rect = paint_rect; |