Bug 270201
| Summary: | Gathering resource load statistics resumes all WebContent processes | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ben Nham <nham> |
| Component: | WebKit Misc. | Assignee: | Ben Nham <nham> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | nham, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Local Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ben Nham
Right now, gathering all resource load statistics (via -[WKWebsiteDataStore _getResourceLoadStatisticsDataSummary:]) sends an async IPC with a completion handler to all WebContent processes, potentially resuming them if they're in a suspended state. This is only necessary to flush the last ~5 sec of resource load statistics from WebContent to NetworkProcess.
For the use cases of this API, it's okay for the data to be slightly out of date. We should just avoid the broadcast IPC (and resuming processes that might be suspended). This is important because the API is used regularly (e.g. Safari calls it once every time you open a new tab), which can cause all suspended WebContent processes to regularly spuriously wake up.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/123724907>
Ben Nham
Pull request: https://github.com/WebKit/WebKit/pull/25198
EWS
Committed 275467@main (9530e9a8df9c): <https://commits.webkit.org/275467@main>
Reviewed commits have been landed. Closing PR #25198 and removing active labels.