Bug 264654
| Summary: | Don't make the decision to cache a DisplayList for a TextRun depend on the rendering update frame rate | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Said Abou-Hallawa <sabouhallawa> |
| Component: | Layout and Rendering | Assignee: | Said Abou-Hallawa <sabouhallawa> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | bfulgham, simon.fraser, webkit-bug-importer, zalan |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Said Abou-Hallawa
Currently, caching a DisplayList for the TextRun happens only if the last 30 frames happened in about a second. If the frame rate is lower than 30 FPS, we prevent the caching. The assumption is any animation should be faster than 30 FPS.
So when the frame rate is low, we make it even worse by preventing the caching. The caching should happen if the same TextRun is displayed very often (say >= 20) regardless of the current frame rate.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Said Abou-Hallawa
rdar://118259180
Said Abou-Hallawa
Pull request: https://github.com/WebKit/WebKit/pull/20357
EWS
Committed 270781@main (42af8a3d2bb4): <https://commits.webkit.org/270781@main>
Reviewed commits have been landed. Closing PR #20357 and removing active labels.