RESOLVED FIXED 240497
add a new RemoteResourceCache entry type for glyph buffers 
https://bugs.webkit.org/show_bug.cgi?id=240497
Summary add a new RemoteResourceCache entry type for glyph buffers
Cameron McCormack (:heycam)
Reported 2022-05-16 18:54:52 PDT
We have logic that detects when the same run of text is repeatedly painted and cache the result of shaping in a GlyphDisplayListCache entry. With GPU Process DOM rendering enabled, replaying that entry still results in significant IPC traffic, since we must send the vectors of glyph IDs and advances as part of the DrawGlyphs display list item. We can turn these vectors (and perhaps the other DrawGlyphs members apart from the font resource ID) into a cacheable resource that the GPU Process can store.
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-05-16 18:55:00 PDT
<rdar://problem/93387615>
Cameron McCormack (:heycam)
Comment 2 2022-05-23 00:07:51 PDT
Pull request: https://github.com/WebKit/WebKit/pull/904
EWS
Comment 3 2022-06-05 23:20:11 PDT
Committed r295278 (251324@main): <https://commits.webkit.org/251324@main> Reviewed commits have been landed. Closing PR #904 and removing active labels.
WebKit Commit Bot
Comment 4 2022-06-06 07:14:13 PDT
Re-opened since this is blocked by bug 241336
Cameron McCormack (:heycam)
Comment 5 2022-06-07 15:10:32 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1345
EWS
Comment 6 2022-06-07 21:24:20 PDT
Committed r295373 (251381@main): <https://commits.webkit.org/251381@main> Reviewed commits have been landed. Closing PR #1345 and removing active labels.
Cameron McCormack (:heycam)
Comment 7 2022-06-14 22:06:25 PDT
This was reverted in bug 241621.
Cameron McCormack (:heycam)
Comment 8 2022-06-15 16:37:02 PDT
Will be re-landing this with some changes.
Cameron McCormack (:heycam)
Comment 9 2022-06-15 17:28:39 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1555
EWS
Comment 10 2022-06-16 18:38:52 PDT
Committed r295621 (251626@main): <https://commits.webkit.org/251626@main> Reviewed commits have been landed. Closing PR #1555 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.