Bug 211734 - [iOS] WKWebView.evaluateJavaScript should run at foreground priority when app is foreground
Summary: [iOS] WKWebView.evaluateJavaScript should run at foreground priority when app...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-05-11 11:04 PDT by Chris Dumez
Modified: 2020-05-11 12:43 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.87 KB, patch)
2020-05-11 11:05 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (2.11 KB, patch)
2020-05-11 11:10 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (2.14 KB, patch)
2020-05-11 11:31 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2020-05-11 11:04:06 PDT
WKWebView.evaluateJavaScript should run at foreground priority when app is foreground.
Comment 1 Chris Dumez 2020-05-11 11:05:15 PDT
Created attachment 399030 [details]
Patch
Comment 2 Chris Dumez 2020-05-11 11:10:29 PDT
Created attachment 399032 [details]
Patch
Comment 3 Chris Dumez 2020-05-11 11:31:05 PDT
Created attachment 399034 [details]
Patch
Comment 4 Darin Adler 2020-05-11 12:29:28 PDT
Comment on attachment 399034 [details]
Patch

Seems like a better principle is inheriting this from what triggered the need to run script rather than querying the state at the moment the WebKit API is called. This seems like a shortcut that might get it right, but might get it wrong.
Comment 5 EWS 2020-05-11 12:42:45 PDT
Committed r261489: <https://trac.webkit.org/changeset/261489>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 399034 [details].
Comment 6 Radar WebKit Bug Importer 2020-05-11 12:43:15 PDT
<rdar://problem/63101371>