Summary: | [GTK] Fix compilation problems when setting ENABLE_DRAG_SUPPORT = FALSE | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Lorenzo Tilve <ltilve> | ||||||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | cgarcia, commit-queue, gustavo, mrobinson, pnormand | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Lorenzo Tilve
2013-09-23 01:52:56 PDT
Created attachment 212338 [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 Comment on attachment 212338 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=212338&action=review r- because you're touching mac-related stuff > Source/WebCore/WebCore.exp.in:1856 > +#if ENABLE(DRAG_SUPPORT) > __ZN7WebCore12EventHandler12mouseDraggedEP7NSEvent > +#endif I am not sure this is correct, in any case, I don't think you should change Mac's symbols export file in a gtk-specific change =) Wouldn't it make sense for the client to handle this? It could do that by intercepting drag signals and not sending them to the widget. Created attachment 212438 [details]
Patch
Fix the compilation problems when hardcoding on the Profile.h +define ENABLE_DRAG_SUPPORT 0 on the GTK build, but without adding an extra configuration option. Removed the changes on Mac's symbols export file.
Attachment 212438 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/page/gtk/EventHandlerGtk.cpp', u'Source/WebCore/platform/gtk/PasteboardGtk.cpp', u'Source/WebKit/gtk/ChangeLog', u'Source/WebKit/gtk/WebCoreSupport/DragClientGtk.cpp', u'Source/WebKit/gtk/WebCoreSupport/DragClientGtk.h', u'Source/WebKit2/ChangeLog', u'Source/WebKit2/UIProcess/API/gtk/WebKitWebViewBase.cpp', u'Source/WebKit2/WebProcess/WebCoreSupport/gtk/WebDragClientGtk.cpp']" exit_code: 1
Source/WebKit2/ChangeLog:8: Line contains tab character. [whitespace/tab] [5]
Total errors found: 1 in 9 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 212440 [details]
Patch
Comment on attachment 212440 [details] Patch Clearing flags on attachment: 212440 Committed r156345: <http://trac.webkit.org/changeset/156345> All reviewed patches have been landed. Closing bug. |