Bug 141297 – [WK2][Cocoa] Populate m_contentsFilter bloom filter from the main dispatch queue NetworkCacheStorage::initialize()

• Component: Page Loading
• Status: RESOLVED FIXED

[Chris Dumez]

Populate m_contentsFilter bloom filter from the main dispatch queue NetworkCacheStorage::initialize() to avoid thread-safety issues.

[Chris Dumez]

Created attachment 246116 [details]
Patch

[Antti Koivisto]

Comment on attachment 246116 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=246116&action=review

> Source/WebKit2/NetworkProcess/cache/NetworkCacheStorageCocoa.mm:142
> +            dispatch_async(dispatch_get_main_queue(), [this, hash] {
> +                m_contentsFilter.add(hash);
> +            });

This gets called a lot so dispatching every item individually is not very optimal. But we can do better later if needed.

[WebKit Commit Bot]

Comment on attachment 246116 [details]
Patch

Clearing flags on attachment: 246116

Committed r179704: <http://trac.webkit.org/changeset/179704>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.