Changeset 259209 in webkit

Timestamp:

Mar 30, 2020 10:35:11 AM (4 years ago)

Author:

Chris Dumez

Message:

Regression(r246188) WebProcess is launched too eagerly when [WKWebView _restoreSessionState] is called
​https://bugs.webkit.org/show_bug.cgi?id=207908
<rdar://problem/59619323>

Reviewed by Darin Adler.

Since r246188, the WebProcess is launched eagerly when [WKWebView _restoreSessionState] is called. This is bad
for performance because we are unable to leverage the process cache at this point (since we don't know which
domain will be loaded).

This patch thus reverts r246188 and fixes what r246188 was trying to address in a different way. If the process
was not launched yet when restoreSessionState() is called, the session state properly gets sent to the WebProcess
after launch, via the WebPageCreationParameters. What was missing at that point was that the session state was
restore by an API Request. To fix this, we now pass an extra itemStatesWereRestoredByAPIRequest flag in
WebPageCreationParameters.

• Shared/WebPageCreationParameters.cpp:

(WebKit::WebPageCreationParameters::encode const):
(WebKit::WebPageCreationParameters::decode):

• Shared/WebPageCreationParameters.h:
• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::restoreFromSessionState):

• UIProcess/WebPageProxy.h:
• WebProcess/WebPage/WebPage.cpp:

(WebKit::m_processDisplayName):

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• Shared/WebPageCreationParameters.cpp (modified) (2 diffs)
• Shared/WebPageCreationParameters.h (modified) (1 diff)
• UIProcess/WebPageProxy.cpp (modified) (2 diffs)
• UIProcess/WebPageProxy.h (modified) (1 diff)
• WebProcess/WebPage/WebPage.cpp (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2020-03-30  Chris Dumez  <cdumez@apple.com>
+
+        Regression(r246188) WebProcess is launched too eagerly when [WKWebView _restoreSessionState] is called
+        https://bugs.webkit.org/show_bug.cgi?id=207908
+        <rdar://problem/59619323>
+
+        Reviewed by Darin Adler.
+
+        Since r246188, the WebProcess is launched eagerly when [WKWebView _restoreSessionState] is called. This is bad
+        for performance because we are unable to leverage the process cache at this point (since we don't know which
+        domain will be loaded).
+
+        This patch thus reverts r246188 and fixes what r246188 was trying to address in a different way. If the process
+        was not launched yet when restoreSessionState() is called, the session state properly gets sent to the WebProcess
+        after launch, via the WebPageCreationParameters. What was missing at that point was that the session state was
+        restore by an API Request. To fix this, we now pass an extra itemStatesWereRestoredByAPIRequest flag in
+        WebPageCreationParameters.
+
+        * Shared/WebPageCreationParameters.cpp:
+        (WebKit::WebPageCreationParameters::encode const):
+        (WebKit::WebPageCreationParameters::decode):
+        * Shared/WebPageCreationParameters.h:
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::restoreFromSessionState):
+        * UIProcess/WebPageProxy.h:
+        * WebProcess/WebPage/WebPage.cpp:
+        (WebKit::m_processDisplayName):
+
 2020-03-30  Wenson Hsieh  <wenson_hsieh@apple.com>
 

trunk/Source/WebKit/Shared/WebPageCreationParameters.cpp

@@ -53 +53 @@
     encoder << paginationLineGridEnabled;
     encoder << userAgent;
+    encoder << itemStatesWereRestoredByAPIRequest;
     encoder << itemStates;
     encoder << userContentControllerID;
@@ -212 +213 @@
     parameters.userAgent = WTFMove(*userAgent);
 
+    Optional<bool> itemStatesWereRestoredByAPIRequest;
+    decoder >> itemStatesWereRestoredByAPIRequest;
+    if (!itemStatesWereRestoredByAPIRequest)
+        return WTF::nullopt;
+    parameters.itemStatesWereRestoredByAPIRequest = *itemStatesWereRestoredByAPIRequest;
+
     Optional<Vector<BackForwardListItemState>> itemStates;
     decoder >> itemStates;

trunk/Source/WebKit/Shared/WebPageCreationParameters.h

@@ -98 +98 @@
     String userAgent;
 
+    bool itemStatesWereRestoredByAPIRequest { false };
     Vector<BackForwardListItemState> itemStates;
 

trunk/Source/WebKit/UIProcess/WebPageProxy.cpp

@@ -3499 +3499 @@
 
     if (hasBackForwardList) {
-        // If there isn't a running process yet the RestoreSession message below is just ignored, and
-        // session is restored when the web process is created via creation parameters which is not
-        // considered an API request. So, we launch the initial process here before restoring the
-        // session to ensure the session is restored in the web process via RestoreSession IPC message
-        // which is considered an API request. See https://bugs.webkit.org/show_bug.cgi?id=198561.
-        launchInitialProcessIfNecessary();
+        m_sessionStateWasRestoredByAPIRequest = true;
 
         m_backForwardList->restoreFromState(WTFMove(sessionState.backForwardListState));
-        send(Messages::WebPage::RestoreSession(m_backForwardList->itemStates()));
+        // If the process is not launched yet, the session will be restored when sending the WebPageCreationParameters;
+        if (hasRunningProcess())
+            send(Messages::WebPage::RestoreSession(m_backForwardList->itemStates()));
 
         auto transaction = m_pageLoadState.transaction();
@@ -7685 +7682 @@
     parameters.paginationLineGridEnabled = m_paginationLineGridEnabled;
     parameters.userAgent = userAgent();
+    parameters.itemStatesWereRestoredByAPIRequest = m_sessionStateWasRestoredByAPIRequest;
     parameters.itemStates = m_backForwardList->itemStates();
     parameters.userContentControllerID = m_userContentController->identifier();

trunk/Source/WebKit/UIProcess/WebPageProxy.h

@@ -2752 +2752 @@
     COMPtr<ID3D11Device1> m_device;
 #endif
+    bool m_sessionStateWasRestoredByAPIRequest { false };
     bool m_isQuotaIncreaseDenied { false };
     bool m_isLayerTreeFrozenDueToSwipeAnimation { false };

trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp

@@ -649 +649 @@
     
     if (!parameters.itemStates.isEmpty())
-        restoreSessionInternal(parameters.itemStates, WasRestoredByAPIRequest::No, WebBackForwardListProxy::OverwriteExistingItem::Yes);
+        restoreSessionInternal(parameters.itemStates, parameters.itemStatesWereRestoredByAPIRequest ? WasRestoredByAPIRequest::Yes : WasRestoredByAPIRequest::No, WebBackForwardListProxy::OverwriteExistingItem::Yes);
 
     m_drawingArea->enablePainting();