RESOLVED FIXED 133897
Make GenericCallback variadic
https://bugs.webkit.org/show_bug.cgi?id=133897
Summary Make GenericCallback variadic
mitz
Reported 2014-06-13 22:18:07 PDT
Make GenericCallback variadic
Attachments
Make GenericCallback variadic, get rid of legacy callback functions, etc. (29.35 KB, patch)
2014-06-13 22:25 PDT, mitz
sam: review+
mitz
Comment 1 2014-06-13 22:25:10 PDT
Created attachment 233104 [details] Make GenericCallback variadic, get rid of legacy callback functions, etc.
mitz
Comment 2 2014-06-13 22:32:45 PDT
Csaba Osztrogonác
Comment 3 2014-06-14 02:22:49 PDT
(In reply to comment #2) > Committed <http://trac.webkit.org/r169962>. It broke the GTK build: ../../Source/WebKit2/UIProcess/API/gtk/WebKitPrintOperation.cpp: In function ‘void webkitPrintOperationPrintPagesForFrame(WebKitPrintOperation*, WebKit::WebFrameProxy*, GtkPrintSettings*, GtkPageSetup*)’: ../../Source/WebKit2/UIProcess/API/gtk/WebKitPrintOperation.cpp:279:6: error: no matching function for call to ‘WebKit::GenericCallback<API::Error*>::create(webkitPrintOperationPrintPagesForFrame(WebKitPrintOperation*, WebKit::WebFrameProxy*, GtkPrintSettings*, GtkPageSetup*)::__lambda3)’ })); ^ ../../Source/WebKit2/UIProcess/API/gtk/WebKitPrintOperation.cpp:279:6: note: candidate is: In file included from ../../Source/WebKit2/UIProcess/WebFrameProxy.h:31:0, from ../../Source/WebKit2/UIProcess/API/gtk/WebKitPrintOperationPrivate.h:24, from ../../Source/WebKit2/UIProcess/API/gtk/WebKitPrintOperation.cpp:23: ../../Source/WebKit2/UIProcess/GenericCallback.h:132:40: note: static WTF::PassRefPtr<WebKit::GenericCallback<T> > WebKit::GenericCallback<T>::create(WebKit::GenericCallback<T>::CallbackFunction) [with T = {API::Error*}; WebKit::GenericCallback<T>::CallbackFunction = std::function<void(API::Error*, WebKit::CallbackBase::Error)>] static PassRefPtr<GenericCallback> create(CallbackFunction callback) ^ ../../Source/WebKit2/UIProcess/GenericCallback.h:132:40: note: no known conversion for argument 1 from ‘webkitPrintOperationPrintPagesForFrame(WebKitPrintOperation*, WebKit::WebFrameProxy*, GtkPrintSettings*, GtkPageSetup*)::__lambda3’ to ‘WebKit::GenericCallback<API::Error*>::CallbackFunction {aka std::function<void(API::Error*, WebKit::CallbackBase::Error)>}’
Note You need to log in before you can comment on or make changes to this bug.