RESOLVED FIXED Bug 116036
[Mac] Move setDragImage from ClipboardMac to PasteboardMac
https://bugs.webkit.org/show_bug.cgi?id=116036
Summary [Mac] Move setDragImage from ClipboardMac to PasteboardMac
Darin Adler
Reported 2013-05-13 08:29:24 PDT
[Mac] Move setDragImage from ClipboardMac to PasteboardMac
Attachments
Patch (14.42 KB, patch)
2013-05-13 09:35 PDT, Darin Adler
no flags
Archive of layout-test-results from webkit-ews-01 for mac-mountainlion (1.49 MB, application/zip)
2013-05-13 12:36 PDT, Build Bot
no flags
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2 (791.59 KB, application/zip)
2013-05-13 18:00 PDT, Build Bot
no flags
Patch (14.38 KB, patch)
2013-05-14 19:47 PDT, Darin Adler
andersca: review+
andersca: commit-queue-
Darin Adler
Comment 1 2013-05-13 09:35:50 PDT
Build Bot
Comment 2 2013-05-13 12:36:29 PDT
Comment on attachment 201577 [details] Patch Attachment 201577 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/395893 New failing tests: fast/table/table-rowspan-height-distribution-in-rows.html fast/events/drag-display-none-element.html editing/selection/drag-start-event-client-x-y.html fast/events/clear-drag-state.html fast/events/drag-and-drop-set-drag-data-arguments.html fast/events/mousedown-inside-dragstart-should-not-cause-crash.html editing/pasteboard/drag-and-drop-inputimage-contenteditable.html fast/events/dragging-mouse-moves.html fast/events/drag-and-drop-fire-drag-dragover.html editing/selection/user-drag-element-and-user-select-none.html fast/css-generated-content/drag-state.html fast/events/dropzone-003.html fast/events/remove-target-with-shadow-in-drag.html fast/events/drag-and-drop-dataTransfer-types-nocrash.html fast/events/drag-and-drop.html fast/events/dropzone-002.html fast/events/bogus-dropEffect-effectAllowed.html fast/events/dropzone-001.html editing/pasteboard/drag-and-drop-objectimage-contenteditable.html fast/css/user-drag-none.html
Build Bot
Comment 3 2013-05-13 12:36:30 PDT
Created attachment 201622 [details] Archive of layout-test-results from webkit-ews-01 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-01 Port: mac-mountainlion Platform: Mac OS X 10.8.2
Build Bot
Comment 4 2013-05-13 18:00:09 PDT
Comment on attachment 201577 [details] Patch Attachment 201577 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/470008 New failing tests: fast/events/clear-drag-state.html fast/events/drag-and-drop-set-drag-data-arguments.html fast/events/mousedown-inside-dragstart-should-not-cause-crash.html fast/events/dragging-mouse-moves.html editing/selection/user-drag-element-and-user-select-none.html fast/css-generated-content/drag-state.html editing/selection/drag-start-event-client-x-y.html fast/css/user-drag-none.html
Build Bot
Comment 5 2013-05-13 18:00:11 PDT
Created attachment 201657 [details] Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-16 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.2
Darin Adler
Comment 6 2013-05-13 18:02:30 PDT
Looks like I have some work to do! I had no idea all those dragging tests were failing.
Darin Adler
Comment 7 2013-05-14 19:25:59 PDT
Just one backwards boolean check. Re-uploading shortly.
Darin Adler
Comment 8 2013-05-14 19:47:56 PDT
Anders Carlsson
Comment 9 2013-05-14 21:16:29 PDT
Comment on attachment 201783 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=201783&action=review > Source/WebCore/dom/Clipboard.cpp:47 > + void stopLoading(CachedResourceHandle<CachedImage>&); Missing newline
Darin Adler
Comment 10 2013-05-14 21:27:06 PDT
Note You need to log in before you can comment on or make changes to this bug.