Summary: | [chromium] Compositor: multi-second hangs invalidating large layers | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Stephen White <senorblanco> | ||||||||
Component: | WebCore Misc. | Assignee: | Stephen White <senorblanco> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | cc-bugs, danakj, enne, jamesr, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Attachments: |
|
Description
Stephen White
2012-02-03 13:47:28 PST
Created attachment 125409 [details]
Patch
Created attachment 125421 [details]
Patch
(In reply to comment #2) > Created an attachment (id=125421) [details] > Patch This version, in addition to invalidating tiles by iterating over the hash map rather than the tile indices, removes invalid tiles (those without texture backing) in TiledLayerChromium::pushPropertiesTo(), which limits the number of tiles by the size of the texture cache. Comment on attachment 125421 [details]
Patch
Unofficially LGTM. Thanks for this!
Comment on attachment 125421 [details]
Patch
R=me
Comment on attachment 125421 [details] Patch Clearing flags on attachment: 125421 Committed r106732: <http://trac.webkit.org/changeset/106732> All reviewed patches have been landed. Closing bug. |