Bug 173904 - Avoid double thread dispatch in WebsiteDataStore::fetchDataForTopPrivatelyControlledDomains()
Summary: Avoid double thread dispatch in WebsiteDataStore::fetchDataForTopPrivatelyCon...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-06-27 21:14 PDT by Chris Dumez
Modified: 2017-06-28 00:34 PDT (History)
6 users (show)

See Also:


Attachments
Patch (8.60 KB, patch)
2017-06-27 21:17 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2017-06-27 21:14:46 PDT
Avoid double thread dispatch in WebsiteDataStore::fetchDataForTopPrivatelyControlledDomains(). It calls fetchData() which dispatches to the main queue, then fetchDataForTopPrivatelyControlledDomains() dispatches to the background queue. It should be possible to get fetchData() to dispatch directly on the right queue.
Comment 1 Chris Dumez 2017-06-27 21:17:12 PDT
Created attachment 313984 [details]
Patch
Comment 2 Brent Fulgham 2017-06-28 00:06:22 PDT
Comment on attachment 313984 [details]
Patch

R=me
Comment 3 WebKit Commit Bot 2017-06-28 00:34:45 PDT
Comment on attachment 313984 [details]
Patch

Clearing flags on attachment: 313984

Committed r218871: <http://trac.webkit.org/changeset/218871>
Comment 4 WebKit Commit Bot 2017-06-28 00:34:46 PDT
All reviewed patches have been landed.  Closing bug.