Title: [213181] trunk/Source/WebKit2
- Revision
- 213181
- Author
- beid...@apple.com
- Date
- 2017-02-28 15:08:28 -0800 (Tue, 28 Feb 2017)
Log Message
Fix API test regression(s):
REGRESSION (r213168): API test WKUserContentController.ScriptMessageHandlerMultipleHandlerRemoval crashing
https://bugs.webkit.org/show_bug.cgi?id=168991
Unreviewed.
* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::createNewWebProcessRespectingProcessCountLimit):
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (213180 => 213181)
--- trunk/Source/WebKit2/ChangeLog 2017-02-28 23:05:13 UTC (rev 213180)
+++ trunk/Source/WebKit2/ChangeLog 2017-02-28 23:08:28 UTC (rev 213181)
@@ -1,3 +1,14 @@
+2017-02-28 Brady Eidson <beid...@apple.com>
+
+ Fix API test regression(s):
+ REGRESSION (r213168): API test WKUserContentController.ScriptMessageHandlerMultipleHandlerRemoval crashing
+ https://bugs.webkit.org/show_bug.cgi?id=168991
+
+ Unreviewed.
+
+ * UIProcess/WebProcessPool.cpp:
+ (WebKit::WebProcessPool::createNewWebProcessRespectingProcessCountLimit):
+
2017-02-28 Wenson Hsieh <wenson_hs...@apple.com>
Data interaction should support attachment elements
Modified: trunk/Source/WebKit2/UIProcess/WebProcessPool.cpp (213180 => 213181)
--- trunk/Source/WebKit2/UIProcess/WebProcessPool.cpp 2017-02-28 23:05:13 UTC (rev 213180)
+++ trunk/Source/WebKit2/UIProcess/WebProcessPool.cpp 2017-02-28 23:08:28 UTC (rev 213181)
@@ -795,7 +795,8 @@
}
// Choose the process with fewest pages.
- auto* processes = mustMatchDataStore ? &m_processes : &processesMatchingDataStore;
+ auto* processes = mustMatchDataStore ? &processesMatchingDataStore : &m_processes;
+ ASSERT(!processes->isEmpty());
auto& process = *std::min_element(processes->begin(), processes->end(), [](const RefPtr<WebProcessProxy>& a, const RefPtr<WebProcessProxy>& b) {
return a->pageCount() < b->pageCount();
});
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes