WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
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
Committed <
http://trac.webkit.org/r169962
>.
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.
Top of Page
Format For Printing
XML
Clone This Bug