Bug 239992

Summary: Regression(r292351) Processes no longer get the right jetsam priority on iOS
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Tim Horton <thorton>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, emw, ggaren, kkinnunen, nham, simon.fraser, slewis, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 238760    
Attachments:
Description Flags
Patch
cdumez: review+, ews-feeder: commit-queue-
Patch none

Description Chris Dumez 2022-05-02 18:00:49 PDT 
Processes no longer get the right jetsam priority on iOS since r292351. Turns out that RunningBoard still uses that feature flag.
Comment 1 Chris Dumez 2022-05-02 18:07:15 PDT 
Looks like Tim actually dropped all support for feature flags (the plists are gone) so it is a bit trickier than simply reverting that patch.
Comment 2 Radar WebKit Bug Importer 2022-05-02 18:12:42 PDT 
<rdar://problem/92644672>
Comment 3 Tim Horton 2022-05-02 20:54:42 PDT 
Created attachment 458718 [details]
Patch
Comment 4 Chris Dumez 2022-05-02 20:56:40 PDT 
Comment on attachment 458718 [details]
Patch

r=me assuming the bots are happy
Comment 5 Tim Horton 2022-05-02 21:04:45 PDT 
Created attachment 458719 [details]
Patch
Comment 6 Tim Horton 2022-05-02 21:05:10 PDT 
(In reply to Chris Dumez from comment #4)
> Comment on attachment 458718 [details]
> Patch
> 
> r=me assuming the bots are happy

They are not, but I will get them happy (silly me only tested a production configuration).
Comment 7 EWS 2022-05-02 22:28:09 PDT 
Committed r293716 (250205@main): <https://commits.webkit.org/250205@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 458719 [details].