Summary: | REGRESSION(r171034) [GTK] Build broken. | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Javier Fernandez <jfernandez> | ||||||||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | berto, cgarcia, commit-queue, gustavo, mrobinson | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Attachments: |
|
Description
Javier Fernandez
2014-07-12 17:16:03 PDT
Created attachment 234814 [details]
Patch
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 Created attachment 234815 [details]
Patch
Comment on attachment 234815 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=234815&action=review > Source/WebKit2/UIProcess/API/gtk/PageClientImpl.cpp:395 > +void PageClientImpl::navigationGestureWillEnd(bool willNavigate, WebBackForwardListItem&) > +{ > +} > + > +void PageClientImpl::navigationGestureDidEnd(bool willNavigate, WebBackForwardListItem&) > +{ > +} Should omit argument names here to avoid unused argument warnings. Created attachment 234816 [details]
Patch
Created attachment 234818 [details]
Patch
Comment on attachment 234818 [details] Patch Clearing flags on attachment: 234818 Committed r171040: <http://trac.webkit.org/changeset/171040> All reviewed patches have been landed. Closing bug. |