Bug 257210 - Page reload does not take a foreground process assertion
Summary: Page reload does not take a foreground process assertion
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ben Nham
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-05-23 09:50 PDT by Ben Nham
Modified: 2023-05-23 15:28 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 Ben Nham 2023-05-23 09:50:33 PDT 
On some app resumes where an app does a reload in an offscreen WKWebView, we have seen WebContent initialization take 15+ seconds because it's running at background priority. This is because WebPageProxy::reload doesn't take a foreground process assertion. However, WebPageProxy::loadRequest does take a foreground process assertion by default on iOS (207341). This seems like an oversight that should be fixed.
Comment 1 Ben Nham 2023-05-23 09:51:16 PDT 
rdar://109407280
Comment 2 Ben Nham 2023-05-23 09:55:00 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/14249
Comment 3 EWS 2023-05-23 15:28:20 PDT 
Committed 264443@main (bce1eb6c24fb): <https://commits.webkit.org/264443@main>

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