[webkit-changes] [WebKit/WebKit] ae9067: Do not launch network process if not necessary in ...

[Sihui]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ae9067a992640619f66b670e3eb64f9cc0eb3ff5
      https://github.com/WebKit/WebKit/commit/ae9067a992640619f66b670e3eb64f9cc0eb3ff5
  Author: Sihui Liu <sihui_liu at apple.com>
  Date:   2022-09-21 (Wed, 21 Sep 2022)

  Changed paths:
    M Source/WebKit/WebProcess/Network/WebLoaderStrategy.cpp

  Log Message:
  -----------
  Do not launch network process if not necessary in WebLoaderStrategy
https://bugs.webkit.org/show_bug.cgi?id=245442
<rdar://100187344>

Reviewed by Geoffrey Garen.

We currently ensure network process connection in web process when sending IsResourceLoadFinished, RemoveLoadIdentifier,
PageLoadCompleted and BrowsingContextRemoved messages. This means if network process does not exist (e.g. due to crash),
we will relaunch it. But the newly launched network process won't have information about loads in previous network
process, so web process should not need to wait for network process to be launched in these cases. (Note we still
ensure network process connection when network process is actually needed, e.g. starting a load or setting load
parameters.)

* Source/WebKit/WebProcess/Network/WebLoaderStrategy.cpp:
(WebKit::WebLoaderStrategy::remove):
(WebKit::WebLoaderStrategy::pageLoadCompleted):
(WebKit::WebLoaderStrategy::browsingContextRemoved):
(WebKit::WebLoaderStrategy::isResourceLoadFinished):

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