Summary: | WinCairo buildfix after r168611 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Berta József <jberta.u-szeged> | ||||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bfulgham, commit-queue, darin, galpeter, ossy, peavo, zan | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Berta József
2014-05-12 09:00:36 PDT
Created attachment 231299 [details]
Speculative buildfix, not tested
callOnMainThread takes a function parameter: void callOnMainThread(std::function<void ()>); I think you have to do something like: callOnMainThread(MainThreadTask(CurlDownload::downloadFinishedCallback, download)); Comment on attachment 231299 [details] Speculative buildfix, not tested View in context: https://bugs.webkit.org/attachment.cgi?id=231299&action=review > Source/WebCore/platform/network/curl/CurlDownload.cpp:216 > - callOnMainThread<CurlDownload*, CurlDownload*>(CurlDownload::downloadFinishedCallback, download); > + callOnMainThread(CurlDownload::downloadFinishedCallback, download); I think we should use callOnMainThread(MainThreadTask((CurlDownload::downloadFinishedCallback, download)); here. > Source/WebCore/platform/network/curl/CurlDownload.cpp:218 > - callOnMainThread<CurlDownload*, CurlDownload*>(CurlDownload::downloadFailedCallback, download); > + callOnMainThread(CurlDownload::downloadFailedCallback, download); ditto > Source/WebCore/platform/network/curl/CurlDownload.cpp:403 > - callOnMainThread<CurlDownload*, CurlDownload*>(receivedResponseCallback, this); > + callOnMainThread(receivedResponseCallback, this); ditto > Source/WebCore/platform/network/curl/CurlDownload.cpp:416 > - callOnMainThread<CurlDownload*, CurlDownload*, int, int>(receivedDataCallback, this, size); > + callOnMainThread(receivedDataCallback, this, size); ditto Created attachment 231301 [details]
Updating patch
Comment on attachment 231301 [details] Updating patch Clearing flags on attachment: 231301 Committed r168626: <http://trac.webkit.org/changeset/168626> All reviewed patches have been landed. Closing bug. |