RESOLVED FIXED 64518
removes #if ENABLE(DRAG_SUPPORT) check 
https://bugs.webkit.org/show_bug.cgi?id=64518
Summary removes #if ENABLE(DRAG_SUPPORT) check
arno.
Reported 2011-07-14 01:47:28 PDT
Hi, Source/WebCore/platform/gtk/ClipboardGtk.h has a #if ENABLE(DRAG_SUPPORT) check. It's the only place in gtk port where there is this check. In all other places, DRAG_SUPPORT is supposed to be enabled. So, if I understand correctly, it makes little sense to keep it.
Attachments
removes last ENABLE(DRAG_SUPPORT) gtk check (1.39 KB, patch)
2011-07-14 01:51 PDT, arno. 		mrobinson: review-
DetailsFormatted DiffDiff
patch v1.1 (1.57 KB, patch)
2011-07-25 02:00 PDT, arno. 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
arno.
Comment 1 2011-07-14 01:51:56 PDT
Created attachment 100788 [details] removes last ENABLE(DRAG_SUPPORT) gtk check
Martin Robinson
Comment 2 2011-07-24 18:12:52 PDT
Comment on attachment 100788 [details] removes last ENABLE(DRAG_SUPPORT) gtk check View in context: https://bugs.webkit.org/attachment.cgi?id=100788&action=review This seems fine. It's still used in other places throughout WebCore, but otherwise, only Mac uses it in the platform layer. Please fix the ChangeLog, then we can use the commit-queue to commit this. > Source/WebCore/ChangeLog:8 > + No new tests. (OOPS!) You should remove this line.
arno.
Comment 3 2011-07-25 02:00:22 PDT
Created attachment 101855 [details] patch v1.1
WebKit Review Bot
Comment 4 2011-07-25 11:08:49 PDT
Comment on attachment 101855 [details] patch v1.1 Clearing flags on attachment: 101855 Committed r91681: <http://trac.webkit.org/changeset/91681>
WebKit Review Bot
Comment 5 2011-07-25 11:08:53 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.