Log Message
Don't force creation of process pool when enabling resource load statistics https://bugs.webkit.org/show_bug.cgi?id=180374 <rdar://problem/35545639>
Reviewed by Alex Christensen. Pass appropriate arguments to the 'processPool' accessor so that we only get the set of existing process pools. We don't want to pay the cost of launching new process pools just to set this flag. The flag will be properly set when the pools are created as soon as a WebView is instantiated. * UIProcess/WebsiteData/WebsiteDataStore.cpp: (WebKit::WebsiteDataStore::setResourceLoadStatisticsEnabled):
Modified Paths
Diff
Modified: trunk/Source/WebKit/ChangeLog (225507 => 225508)
--- trunk/Source/WebKit/ChangeLog 2017-12-05 00:57:33 UTC (rev 225507)
+++ trunk/Source/WebKit/ChangeLog 2017-12-05 01:11:52 UTC (rev 225508)
@@ -1,3 +1,19 @@
+2017-12-04 Brent Fulgham <[email protected]>
+
+ Don't force creation of process pool when enabling resource load statistics
+ https://bugs.webkit.org/show_bug.cgi?id=180374
+ <rdar://problem/35545639>
+
+ Reviewed by Alex Christensen.
+
+ Pass appropriate arguments to the 'processPool' accessor so that we only get the
+ set of existing process pools. We don't want to pay the cost of launching new
+ process pools just to set this flag. The flag will be properly set when the pools
+ are created as soon as a WebView is instantiated.
+
+ * UIProcess/WebsiteData/WebsiteDataStore.cpp:
+ (WebKit::WebsiteDataStore::setResourceLoadStatisticsEnabled):
+
2017-12-04 David Quesada <[email protected]>
Add a class for parsing application manifests
Modified: trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp (225507 => 225508)
--- trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp 2017-12-05 00:57:33 UTC (rev 225507)
+++ trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp 2017-12-05 01:11:52 UTC (rev 225508)
@@ -1326,7 +1326,9 @@
}
m_resourceLoadStatistics = nullptr;
- for (auto& processPool : processPools())
+
+ auto existingProcessPools = processPools(std::numeric_limits<size_t>::max(), false);
+ for (auto& processPool : existingProcessPools)
processPool->setResourceLoadStatisticsEnabled(false);
}
_______________________________________________ webkit-changes mailing list [email protected] https://lists.webkit.org/mailman/listinfo/webkit-changes
