Switch non-MPD nanobench path to use a separate canvas per tile
authorrobertphillips <robertphillips@google.com>
Tue, 9 Dec 2014 18:27:54 +0000 (10:27 -0800)
committerCommit bot <commit-bot@chromium.org>
Tue, 9 Dec 2014 18:27:54 +0000 (10:27 -0800)
commita3e52724ac8b9fa7b48507bff4fa8e558a213e49
tree2b12e623f2312602bf737609bb61534e6f7f3a4d
parent43be354644dfc875baea137027f32d118305817f
Switch non-MPD nanobench path to use a separate canvas per tile

It is desirable that, when layer hoisting is disabled, the MPD and non-MPD timings be
roughly the same. Unfortunately, using a separate canvas for each tile (a requirement
for MPD) introduces its own discrepancy into the timing. Using a separate canvas for
each tile doesn't seem to make a difference for 8888 (see the non-MPD 8888 column below)
but slows down GPU rendering (see the non-MPD GPU column below). Since this is how
Chromium renders I propose switching to this regimen (even though it is "slowing down"
GPU rendering).

nanobench mean times (ms) with layer hoisting disabled (for desk_amazon.skp)

8888
                          MPD     non-MPD
1 canvas (old-style)     0.628    1.71
separate (new-style)     0.795    1.63

GPU
                          MPD     non-MPD
1 canvas (old-style)     2.34     1.69
separate (new-style)     2.32     2.66

Review URL: https://codereview.chromium.org/779643002
bench/SKPBench.cpp