Title: [231031] trunk/Source/WebKit
- Revision
- 231031
- Author
- [email protected]
- Date
- 2018-04-25 16:17:31 -0700 (Wed, 25 Apr 2018)
Log Message
PSON: Don't create a new process when navigating to a blob URL, data URL, and about:blank
https://bugs.webkit.org/show_bug.cgi?id=184962
Reviewed by Youenn Fablet.
<rdar://problem/39715044>
Build fix. Revert the change in r231019 to remove the empty URL and about:blank check here.
These checks are for the source / originating URL, not the target URL.
* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::processForNavigation):
Modified Paths
Diff
Modified: trunk/Source/WebKit/ChangeLog (231030 => 231031)
--- trunk/Source/WebKit/ChangeLog 2018-04-25 23:13:13 UTC (rev 231030)
+++ trunk/Source/WebKit/ChangeLog 2018-04-25 23:17:31 UTC (rev 231031)
@@ -1,3 +1,17 @@
+2018-04-25 Ryosuke Niwa <[email protected]>
+
+ PSON: Don't create a new process when navigating to a blob URL, data URL, and about:blank
+ https://bugs.webkit.org/show_bug.cgi?id=184962
+
+ Reviewed by Youenn Fablet.
+ <rdar://problem/39715044>
+
+ Build fix. Revert the change in r231019 to remove the empty URL and about:blank check here.
+ These checks are for the source / originating URL, not the target URL.
+
+ * UIProcess/WebProcessPool.cpp:
+ (WebKit::WebProcessPool::processForNavigation):
+
2018-04-25 Youenn Fablet <[email protected]>
WebLoaderStrategy::networkMetricsFromResourceLoadIdentifier should use DoNotProcessIncomingMessagesWhenWaitingForSyncReply
Modified: trunk/Source/WebKit/UIProcess/WebProcessPool.cpp (231030 => 231031)
--- trunk/Source/WebKit/UIProcess/WebProcessPool.cpp 2018-04-25 23:13:13 UTC (rev 231030)
+++ trunk/Source/WebKit/UIProcess/WebProcessPool.cpp 2018-04-25 23:17:31 UTC (rev 231031)
@@ -2049,7 +2049,7 @@
auto targetURL = navigation.currentRequest().url();
auto url = "" { ParsedURLString, page.pageLoadState().url() };
- if (!url.isValid() || protocolHostAndPortAreEqual(url, targetURL))
+ if (!url.isValid() || url.isEmpty() || url.isBlankURL() ||protocolHostAndPortAreEqual(url, targetURL))
return page.process();
action = ""
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
