Summary: | Remove CrossThreadTask | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Zan Dobersek <zan> | ||||
Component: | WebCore Misc. | Assignee: | Zan Dobersek <zan> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | ||||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Zan Dobersek
2014-10-09 13:32:13 PDT
Created attachment 239565 [details]
Patch
Comment on attachment 239565 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=239565&action=review > Source/WebCore/ChangeLog:10 > + Replace uses of CrossThreadTask with C++11 lambdas. The initializations > + of cross-thread copies are rather verbose at the moment, but should look > + better once we can switch to using C++14 init capture for lambdas. Anders is also working on a project that might lead to cross-thread-safe reference counting for String. If he succeeds, that will also make many of them considerably better. > Source/WebCore/loader/WorkerThreadableLoader.h:108 > + virtual void didSendData(unsigned long long bytesSent, unsigned long long totalBytesToBeSent) override; Looks like this is indented four spaces too far. Committed r174583: <http://trac.webkit.org/changeset/174583> |