Followup to: Make sure history navigations reuse the existing process when necessary.
authorbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 May 2018 22:30:45 +0000 (22:30 +0000)
committerbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 May 2018 22:30:45 +0000 (22:30 +0000)
https://bugs.webkit.org/show_bug.cgi?id=185532

Reviewed by Andy Estes.

* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::processForNavigationInternal): When re-using the same process,
  don't change the policy action.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@231723 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/WebProcessPool.cpp

index 5b571af..fb9b636 100644 (file)
@@ -1,3 +1,14 @@
+2018-05-11  Brady Eidson  <beidson@apple.com>
+
+        Followup to: Make sure history navigations reuse the existing process when necessary.
+        https://bugs.webkit.org/show_bug.cgi?id=185532
+
+        Reviewed by Andy Estes.
+
+        * UIProcess/WebProcessPool.cpp:
+        (WebKit::WebProcessPool::processForNavigationInternal): When re-using the same process,
+          don't change the policy action.
+
 2018-05-11  Charles Vazac  <cvazac@gmail.com>
 
         Runtime feature flag for Server-Timing
index ebdd045..a73cecd 100644 (file)
@@ -2112,10 +2112,8 @@ Ref<WebProcessProxy> WebProcessPool::processForNavigationInternal(WebPageProxy&
         // If the target back/forward item and the current back/forward item originated
         // in the same WebProcess then we should reuse the current WebProcess.
         if (auto* currentItem = page.backForwardList().currentItem()) {
-            if (currentItem->itemID().processIdentifier == backForwardListItem->itemID().processIdentifier) {
-                action = PolicyAction::Suspend;
+            if (currentItem->itemID().processIdentifier == backForwardListItem->itemID().processIdentifier)
                 return page.process();
-            }
         }
     }