[webkit-changes] [WebKit/WebKit] 9530e9: Gathering resource load statistics resumes suspend...

[bnham]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 9530e9a8df9ca849d688eb79159296a2300614b6
      https://github.com/WebKit/WebKit/commit/9530e9a8df9ca849d688eb79159296a2300614b6
  Author: Ben Nham <nham at apple.com>
  Date:   2024-02-28 (Wed, 28 Feb 2024)

  Changed paths:
    M Source/WebKit/UIProcess/WebProcessPool.cpp

  Log Message:
  -----------
  Gathering resource load statistics resumes suspended WebContent processes
https://bugs.webkit.org/show_bug.cgi?id=270201
rdar://problem/123724907

Reviewed by Alex Christensen.

Right now, gathering all resource load statistics (via ` _getResourceLoadStatisticsDataSummary:`)
sends an async IPC with a completion handler to all WebContent processes. This causes a background
activity to run which causes all suspended WebContent processes to resume just to handle this IPC.

This process resume isn't necessary because WebProcess already flushes all pending resource load
statistics to NetworkProcess at suspend time (see the call to `flushResourceLoadStatistics` in
`WebContent::prepareToSuspend`). So we can avoid the process resume by just not sending this IPC to
suspended WebContent processes.

* Source/WebKit/UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::sendResourceLoadStatisticsDataImmediately):

Canonical link: https://commits.webkit.org/275467@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications