Bug 240497 - add a new RemoteResourceCache entry type for glyph buffers
Summary: add a new RemoteResourceCache entry type for glyph buffers
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Process Model (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Cameron McCormack (:heycam)
URL:
Keywords: InRadar
Depends on: 241336
Blocks:
  Show dependency treegraph
 
Reported: 2022-05-16 18:54 PDT by Cameron McCormack (:heycam)
Modified: 2022-06-23 12:10 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Cameron McCormack (:heycam) 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.
Comment 1 Radar WebKit Bug Importer 2022-05-16 18:55:00 PDT
<rdar://problem/93387615>
Comment 2 Cameron McCormack (:heycam) 2022-05-23 00:07:51 PDT
Pull request: https://github.com/WebKit/WebKit/pull/904
Comment 3 EWS 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.
Comment 4 WebKit Commit Bot 2022-06-06 07:14:13 PDT
Re-opened since this is blocked by bug 241336
Comment 5 Cameron McCormack (:heycam) 2022-06-07 15:10:32 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1345
Comment 6 EWS 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.
Comment 7 Cameron McCormack (:heycam) 2022-06-14 22:06:25 PDT
This was reverted in bug 241621.
Comment 8 Cameron McCormack (:heycam) 2022-06-15 16:37:02 PDT
Will be re-landing this with some changes.
Comment 9 Cameron McCormack (:heycam) 2022-06-15 17:28:39 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1555
Comment 10 EWS 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.