Bug 239278

Summary: [GPUP] Add logging for installed font reconstitution failure
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: New BugsAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED WONTFIX    
Severity: Normal CC: simon.fraser
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch simon.fraser: review-

Myles C. Maxfield
Reported 2022-04-13 00:34:02 PDT
[GPUP] Add logging for installed font reconstitution failure
Attachments
Patch (1.64 KB, patch)
2022-04-13 00:47 PDT, Myles C. Maxfield 		simon.fraser: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2022-04-13 00:47:37 PDT
Created attachment 457511 [details] Patch
Simon Fraser (smfr)
Comment 2 2022-04-13 09:17:52 PDT
Comment on attachment 457511 [details] Patch No need to add logging; this is reproducible.
Myles C. Maxfield
Comment 3 2022-04-13 10:07:24 PDT
Really? I couldn’t reproduce it 🤔
Simon Fraser (smfr)
Comment 4 2022-04-13 10:48:44 PDT
I can reproduce it with `run-webkit-tests --ios-simulator fast/text/` against an iOS 15.4 SDK
Simon Fraser (smfr)
Comment 5 2022-04-13 11:06:16 PDT
Logging says: Cannot reconstitute installed font in GPU process: file://iNmEmOrYcGfOnT_0x600003184580#postscript-name=Ahem (Ahem) - { NSFontNameAttribute = Ahem; NSFontSizeAttribute = 16; }
Myles C. Maxfield
Comment 6 2022-04-13 13:06:03 PDT
Oh! This only happens in tests! It’s because we use InjectedBundle to install Ahem just for the web process. I wonder how to fix this 🤔 Surely we don’t want a GPU-Process-injected-bundle… Maybe I add testing-only code in the GPU process and only enable it in WKTR? 🤔🤔🤔
Note You need to log in before you can comment on or make changes to this bug.