registrableDomainsToRemoveWebsiteDataFor() does not need to return a HashMap. It is more efficient to simply return a Vector since we never need to look up keys.
Created attachment 376892 [details] Patch
Comment on attachment 376892 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=376892&action=review I believe this has the side effect of allowing multiple entries for the same RegistrableDomain. That should be fine though, since even if we were to introduce such a bug, we would still delete the right data. > Source/WTF/wtf/CrossThreadCopier.h:156 > + Nice!
Comment on attachment 376892 [details] Patch The Vector is populated from HashMaps of RegistrableDomains, I therefore do not think we can have duplicate RegistrableDomains in the Vector right now.
Comment on attachment 376892 [details] Patch Clearing flags on attachment: 376892 Committed r248956: <https://trac.webkit.org/changeset/248956>
All reviewed patches have been landed. Closing bug.
<rdar://problem/54566740>