RESOLVED FIXED Bug 208546
Drop WebsiteDataStore::processPools()'s ensureAPoolExists optional parameter and add a ensureProcessPools() method instead 
https://bugs.webkit.org/show_bug.cgi?id=208546
Summary Drop WebsiteDataStore::processPools()'s ensureAPoolExists optional parameter ...
Chris Dumez
Reported 2020-03-03 16:35:03 PST
Make WebsiteDataStore::processPools() parameter mandatory to make it clear it may create a process pool. Otherwise, people call processPools() to iterate over existing process pools and do not realize that it actually creates a process pool.
Attachments
Patch (44.33 KB, patch)
2020-03-03 16:36 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (43.38 KB, patch)
2020-03-04 09:59 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (43.41 KB, patch)
2020-03-04 10:10 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2020-03-03 16:36:21 PST
Created attachment 392348 [details] Patch
Sam Weinig
Comment 2 2020-03-03 18:58:46 PST
Comment on attachment 392348 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=392348&action=review > Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.h:304 > + enum class EnsureHasProcessPool : bool { No, Yes }; > + HashSet<RefPtr<WebProcessPool>> processPools(EnsureHasProcessPool, size_t limit = std::numeric_limits<size_t>::max()) const; The idiom we often use for things like this is to have two functions: 1) ensureProcessPools() 2) existingProcessPools() or possibly just processPools().
Chris Dumez
Comment 3 2020-03-04 09:59:28 PST
Created attachment 392434 [details] Patch
Chris Dumez
Comment 4 2020-03-04 10:10:01 PST
Created attachment 392436 [details] Patch
Chris Dumez
Comment 5 2020-03-04 10:21:12 PST
Comment on attachment 392436 [details] Patch Clearing flags on attachment: 392436 Committed r257855: <https://trac.webkit.org/changeset/257855>
Chris Dumez
Comment 6 2020-03-04 10:21:14 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 7 2020-03-04 10:22:18 PST
<rdar://problem/60045083>
Note You need to log in before you can comment on or make changes to this bug.