2009-11-12 Daniel Bates <dbates@webkit.org>
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Nov 2009 19:21:52 +0000 (19:21 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Nov 2009 19:21:52 +0000 (19:21 +0000)
        Reviewed by Darin Adler.

        https://bugs.webkit.org/show_bug.cgi?id=30291

        Fixes an issue where the returned drop effect is incorrect when
        effectAllowed == "uninitialized".

        According to section 7.9.2 of the HTML 5 spec.
        <http://dev.w3.org/html5/spec/Overview.html#the-dragevent-and-datatransfer-interfaces>
        when effectAllowed = "uninitialized" the resulting dropEffect should be the
        user-specified dropEffect (i.e. "copy", "move", "link") and "none" for any
        other case.

        No test cases are included because we have an existing test case from
        bug #24731.

        * dom/Clipboard.cpp:
        (WebCore::dragOpFromIEOp): Added case for op == "uninitialized".
2009-11-12  Daniel Bates  <dbates@webkit.org>

        Reviewed by Darin Adler.

        https://bugs.webkit.org/show_bug.cgi?id=30291

        Rebased results for existing test case fast/events/drag-and-drop.html
        from bug #24731.

        Now, we can run the part of the test when effectAllowed == "uninitialized".

        * fast/events/drag-and-drop-expected.txt: Rebased results.
        * fast/events/drag-and-drop.html: Removed notice about this bug (#30291).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@50888 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/events/drag-and-drop-expected.txt
LayoutTests/fast/events/drag-and-drop.html
WebCore/ChangeLog
WebCore/dom/Clipboard.cpp

index 06a202c..fc72da5 100644 (file)
@@ -1,3 +1,17 @@
+2009-11-12  Daniel Bates  <dbates@webkit.org>
+
+        Reviewed by Darin Adler.
+
+        https://bugs.webkit.org/show_bug.cgi?id=30291
+
+        Rebased results for existing test case fast/events/drag-and-drop.html
+        from bug #24731.
+
+        Now, we can run the part of the test when effectAllowed == "uninitialized". 
+
+        * fast/events/drag-and-drop-expected.txt: Rebased results.
+        * fast/events/drag-and-drop.html: Removed notice about this bug (#30291).
+
 2009-11-12  Chris Fleizach  <cfleizach@apple.com>
 
         Reviewed by Darin Adler.
index f600d3a..6453365 100644 (file)
@@ -1,8 +1,6 @@
 This test checks that drag-and-drop support works and conforms to the HTML 5 specification.
 For each effectAllowed, iterates through the possible dropEffects: none, copy, move, link, dummy.
 
-Note, this test fails when effectAllowed == "uninitialized" because this effect has not been implemented yet (see bug #30291).
-
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
@@ -10,9 +8,9 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 When effectAllowed == "uninitialized"
 
 PASS event.dataTransfer.dropEffect is "none"
-FAIL event.dataTransfer.dropEffect should be copy. Was none.
-FAIL event.dataTransfer.dropEffect should be move. Was none.
-FAIL event.dataTransfer.dropEffect should be link. Was none.
+PASS event.dataTransfer.dropEffect is "copy"
+PASS event.dataTransfer.dropEffect is "move"
+PASS event.dataTransfer.dropEffect is "link"
 PASS event.dataTransfer.dropEffect is "none"
 
 When effectAllowed == "none"
index 3e43427..14fed6d 100644 (file)
     <div id="console"></div>
     <script>
         description("This test checks that drag-and-drop support works and conforms to the HTML 5 specification.<br/>" +
-                    "For each effectAllowed, iterates through the possible dropEffects: none, copy, move, link, dummy.<br/><br/>" +
-                    "Note, this test fails when effectAllowed == &quot;uninitialized&quot; because this effect has not been " +
-                    "implemented yet (see bug #30291).");
+                    "For each effectAllowed, iterates through the possible dropEffects: none, copy, move, link, dummy.");
         var successfullyParsed = true;
     </script>
 </body>
index 469b144..a69cefd 100644 (file)
@@ -1,3 +1,24 @@
+2009-11-12  Daniel Bates  <dbates@webkit.org>
+
+        Reviewed by Darin Adler.
+
+        https://bugs.webkit.org/show_bug.cgi?id=30291
+
+        Fixes an issue where the returned drop effect is incorrect when
+        effectAllowed == "uninitialized".
+
+        According to section 7.9.2 of the HTML 5 spec.
+        <http://dev.w3.org/html5/spec/Overview.html#the-dragevent-and-datatransfer-interfaces>
+        when effectAllowed = "uninitialized" the resulting dropEffect should be the
+        user-specified dropEffect (i.e. "copy", "move", "link") and "none" for any
+        other case.
+        
+        No test cases are included because we have an existing test case from
+        bug #24731.
+
+        * dom/Clipboard.cpp:
+        (WebCore::dragOpFromIEOp): Added case for op == "uninitialized".
+
 2009-11-12  Pavel Feldman  <pfeldman@chromium.org>
 
         Reviewed by Timothy Hatcher.
index 6d1bc15..2aea90a 100644 (file)
@@ -55,6 +55,8 @@ void Clipboard::setAccessPolicy(ClipboardAccessPolicy policy)
 static DragOperation dragOpFromIEOp(const String& op)
 {
     // yep, it's really just this fixed set
+    if (op == "uninitialized")
+        return DragOperationEvery;
     if (op == "none")
         return DragOperationNone;
     if (op == "copy")