Bug 243189 - Evict unused fonts after 4 rendering updates containing paint changes.
Summary: Evict unused fonts after 4 rendering updates containing paint changes.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Matt Woodrow
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-07-25 18:36 PDT by Matt Woodrow
Modified: 2022-07-26 14:45 PDT (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Woodrow 2022-07-25 18:36:28 PDT 
<rdar://97346409>


We previously evicted fonts from the resource cache if they hadn't been used
for 4 rendering updates. Many rendering updates don't contain any paint changes,
so always count all fonts as 'unused'.

This changes us to only count rendering updates that contain painting towards
deciding if a font has been unused long enough to be evicted.
Comment 1 Matt Woodrow 2022-07-25 18:37:12 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/2723
Comment 2 EWS 2022-07-26 14:45:56 PDT 
Committed 252844@main (2da5f7dd5dd8): <https://commits.webkit.org/252844@main>

Reviewed commits have been landed. Closing PR #2723 and removing active labels.