diff options
author | 2014-04-29 15:34:03 +0000 | |
---|---|---|
committer | 2014-04-29 15:34:03 +0000 | |
commit | 705b1ff617ffe672e65d53bcaf3b690e8bb6b025 (patch) | |
tree | 7067ee31d459ec7c8857e6da48b3dbf412c603a5 /tests/PictureShaderTest.cpp | |
parent | 0a5c233e3b911232c0d6f9a88ded99ecf88b8a97 (diff) |
Don't bother doing the empty clip check in SkRecordDraw.
On Mike's suggestion, I tested out not doing any empty-clip check at all in
SkRecordDraw, given that mostly we'll do that again anyway inside SkCanvas.
Most SKPs are identical to the status quo, whether bot or silk, played back in tiles
or full. Average playback performance, both arithmetic and geometric mean, is also
unchanged.
A handful of SKPs do draw faster or slower reliably, particularly when tiled. E.g. a
cnn tile draws about 40% faster, a cuteoverload tile about 20% slower. Their profiles
look pretty much the same before and after, so I can't really explain the changes.
I'd say, given that performance is mostly identical and very identical in bulk,
we might as well remove this code. It's nice to keep SkRecordDraw as dumb as possible.
BUG=skia:2378
R=reed@google.com, fmalita@chromium.org, mtklein@google.com, borenet@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/258183002
git-svn-id: http://skia.googlecode.com/svn/trunk@14433 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'tests/PictureShaderTest.cpp')
0 files changed, 0 insertions, 0 deletions