Changeset 282794 in webkit

Timestamp:

Sep 20, 2021, 5:14:38 PM (5 years ago)

Author:

Chris Dumez

Message:

Avoid doing a second server pre-connect after a process-swap
​https://bugs.webkit.org/show_bug.cgi?id=230517

Reviewed by Geoffrey Garen.

Avoid doing a second server pre-connect after a process-swap. It is unnecessary as we've already
asked the network process to do so before process-swapping.

• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::loadRequestWithNavigationShared):

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/WebPageProxy.cpp (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2021-09-20  Chris Dumez  <cdumez@apple.com>
+
+        Avoid doing a second server pre-connect after a process-swap
+        https://bugs.webkit.org/show_bug.cgi?id=230517
+
+        Reviewed by Geoffrey Garen.
+
+        Avoid doing a second server pre-connect after a process-swap. It is unnecessary as we've already
+        asked the network process to do so before process-swapping.
+
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::loadRequestWithNavigationShared):
+
 2021-09-20  Megan Gardner  <megan_gardner@apple.com>
 

trunk/Source/WebKit/UIProcess/WebPageProxy.cpp

@@ -1410 +1410 @@
     addPlatformLoadParameters(process, loadParameters);
 
-    preconnectTo(url);
+    if (shouldTreatAsContinuingLoad == ShouldTreatAsContinuingLoad::No)
+        preconnectTo(url);
 
     navigation.setIsLoadedWithNavigationShared(true);