RESOLVED WONTFIX 67854
[chromium] Add a paint aggregator to TiledLayerChromium 
https://bugs.webkit.org/show_bug.cgi?id=67854
Summary [chromium] Add a paint aggregator to TiledLayerChromium
Adrienne Walker
Reported 2011-09-09 10:14:10 PDT
TiledLayerChromium currently unions all invalidations together before painting the combined region. This is inefficient in certain cases (L-shaped invalidations) and it should support at least some small number of discrete invalidation regions. It should be smart about merging adjacent regions. This is similar to what paint_aggregator.cc does in Chromium.
Attachments
Add attachment proposed patch, testcase, etc.
Adrienne Walker
Comment 1 2011-10-11 09:54:16 PDT
As a part of fixing this bug, consider tracking a set of update rects rather than a single update rect per layer so that --show-paint-rects is more accurate. See: https://bugs.webkit.org/show_bug.cgi?id=69441
Vangelis Kokkevis
Comment 2 2011-10-11 11:12:42 PDT
(In reply to comment #1) > As a part of fixing this bug, consider tracking a set of update rects rather than a single update rect per layer so that --show-paint-rects is more accurate. > > See: https://bugs.webkit.org/show_bug.cgi?id=69441 I'm curious whether a good first approximation would be to register the dirty regions directly with the tiles, instead of doing a union. That way at least we wouldn't be unioning dirty regions across tile borders.
Note You need to log in before you can comment on or make changes to this bug.