Fix draggable elements are painted at incorrect position

third_party/WebKit/Source/core/input/EventHandlerTest.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/input/EventHandler.h"
 
 #include "core/dom/Document.h"
 #include "core/dom/Range.h"
 #include "core/editing/Editor.h"
 #include "core/editing/FrameSelection.h"
@@ skipping 153 matching lines @@
     document().frame()->eventHandler().handleGestureEvent(doubleTapEvent);
     ASSERT_TRUE(selection.isCaret());
     EXPECT_EQ(Position(line, 0), selection.start());
 
     TapEventBuilder tripleTapEvent(IntPoint(0, 0), 3);
     document().frame()->eventHandler().handleGestureEvent(tripleTapEvent);
     ASSERT_TRUE(selection.isCaret());
     EXPECT_EQ(Position(line, 0), selection.start());
 }
 
+TEST_F(EventHandlerTest, draggedInlinePositionTest)
+{
+    setHtmlInnerHTML(
+        "<style>"
+        "[draggable] {-moz-user-select: none; -khtml-user-select: none; -webkit-user-select: none;"

[fs, 2016/02/23 17:27:45]

Including the -moz-* and -khtml-* seems a bit excessive to me.

It might also make sense to set "margin: 0" on body to make the mapping to a
point more obvious?

[hyunjunekim2, 2016/02/24 13:06:36]

Done.

+        "user-select: none; -webkit-user-drag: element; }"
+        "</style>"
+        "<div style='width: 300px; height: 100px;'>"
+        "<span style='background: blue; width: 50px; height: 50px; margin-left: 250px; font-size: 50px;' draggable='true'>abcd</span>"
+        "</div>");
+    PlatformMouseEvent mouseDownEvent1(

[fs, 2016/02/23 17:27:45]

Why the '1' at the end?

[hyunjunekim2, 2016/02/24 13:06:36]

Done.

+        IntPoint(262, 29),
+        IntPoint(329, 167),
+        LeftButton,
+        PlatformEvent::MousePressed,
+        1,
+        PlatformEvent::Modifiers::LeftButtonDown,
+        WTF::monotonicallyIncreasingTime());
+    document().frame()->eventHandler().handleMousePressEvent(mouseDownEvent1);
+
+    PlatformMouseEvent mouseMoveEvent(
+        IntPoint(618, 298),
+        IntPoint(685, 436),
+        LeftButton,
+        PlatformEvent::MouseMoved,
+        1,
+        PlatformEvent::Modifiers::LeftButtonDown,
+        WTF::monotonicallyIncreasingTime());
+    document().frame()->eventHandler().handleMouseMoveEvent(mouseMoveEvent);
+
+    EXPECT_EQ(IntPoint(4, 21), document().frame()->eventHandler().dragDataTransferLocation());
+}
+
+TEST_F(EventHandlerTest, draggedSVGImagePositionTest)
+{
+    setHtmlInnerHTML(
+        "<style>"
+        "[draggable] {-moz-user-select: none; -khtml-user-select: none; -webkit-user-select: none;"

[fs, 2016/02/23 17:27:45]

Same as above.

[hyunjunekim2, 2016/02/24 13:06:36]

Done.

+        "user-select: none; -webkit-user-drag: element; }"
+        "</style>"
+        "<div style='width: 300px; height: 100px;'>"
+        "<svg width='500' height='500'>"
+        "<rect x='100' y='100' width='100px' height='100px' fill='blue' draggable='true'/>"
+        "</svg>"
+        "</div>");
+    PlatformMouseEvent mouseDownEvent1(

[fs, 2016/02/23 17:27:46]

Same as above.

[hyunjunekim2, 2016/02/24 13:06:36]

Done.

+        IntPoint(145, 144),
+        IntPoint(212, 282),
+        LeftButton,
+        PlatformEvent::MousePressed,
+        1,
+        PlatformEvent::Modifiers::LeftButtonDown,
+        WTF::monotonicallyIncreasingTime());
+    document().frame()->eventHandler().handleMousePressEvent(mouseDownEvent1);
+
+    PlatformMouseEvent mouseMoveEvent(
+        IntPoint(618, 298),
+        IntPoint(685, 436),
+        LeftButton,
+        PlatformEvent::MouseMoved,
+        1,
+        PlatformEvent::Modifiers::LeftButtonDown,
+        WTF::monotonicallyIncreasingTime());
+    document().frame()->eventHandler().handleMouseMoveEvent(mouseMoveEvent);
+
+    EXPECT_EQ(IntPoint(37, 36), document().frame()->eventHandler().dragDataTransferLocation());
+}
+
 } // namespace blink