DescriptionRemove GraphicsContext's deferred save mechanism.
The new Skia recording backend is not as sensitive to the number of draw
ops as the old one -> we don't have to bend over backwards to suppress
unneeded saves at the GC level.
While this results in a slight increase in the number of recorded ops,
the unnecessary saves and restores are optimized out by the bounding box
hierarchy - so there is no playback/rasterization time penalty (Skia
also has an explicit save/restore optimizer which can be used for this
purpose if we ever decide to drop the BBH).
Local perf measurements show no rasterization impact and a minor (<1%)
improvement for record time.
R=schenney@chromium.org,senorblanco@chromium.org
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183750
Patch Set 1 #Patch Set 2 : saveCount() still needed for asserts #Patch Set 3 : implement saveCount() based on m_paintStateStack #Patch Set 4 : minor cleanup #
Messages
Total messages: 8 (2 generated)
|