RESOLVED FIXED 112961
[GTK][WK2] WebKitLoaderClient: add missing initializers 
https://bugs.webkit.org/show_bug.cgi?id=112961
Summary [GTK][WK2] WebKitLoaderClient: add missing initializers
Alberto Garcia
Reported 2013-03-21 14:02:52 PDT
Else gcc complains: ../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp: In function 'void attachLoaderClientToView(WebKitWebView*)': ../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:177:5: warning: missing initializer for member 'WKPageLoaderClient::pluginLoadPolicy' [-Wmissing-field-initializers] ../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:177:5: warning: missing initializer for member 'WKPageLoaderClient::pluginDidFail' [-Wmissing-field-initializers]
Attachments
Patch (1.60 KB, patch)
2013-03-21 14:05 PDT, Alberto Garcia 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alberto Garcia
Comment 1 2013-03-21 14:05:39 PDT
Created attachment 194330 [details] Patch
WebKit Review Bot
Comment 2 2013-03-21 14:08:34 PDT
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Carlos Garcia Campos
Comment 3 2013-03-22 00:31:22 PDT
Comment on attachment 194330 [details] Patch Thanks!
WebKit Commit Bot
Comment 4 2013-04-05 01:20:36 PDT
Comment on attachment 194330 [details] Patch Clearing flags on attachment: 194330 Committed r147725: <http://trac.webkit.org/changeset/147725>
WebKit Commit Bot
Comment 5 2013-04-05 01:20:38 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.