Chromium Code Reviews Chromium Code Reviews
Issue 2048733002:
Selecting with shift+drag results in unexpected drag-n-drop (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: third_party/WebKit/LayoutTests/fast/events/shift-drag-selection-no-drag-n-drop.html |
| diff --git a/third_party/WebKit/LayoutTests/fast/events/shift-drag-selection-no-drag-n-drop.html b/third_party/WebKit/LayoutTests/fast/events/shift-drag-selection-no-drag-n-drop.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..a9b2cb6e09b2392e3499a60592ce6290d1924240 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/fast/events/shift-drag-selection-no-drag-n-drop.html |
| @@ -0,0 +1,42 @@ |
| +<!DOCTYPE html> |
| +<script src="../../resources/testharness.js"></script> |
| +<script src="../../resources/testharnessreport.js"></script> |
| +<p>This test ensures drag-n-drop does not start when extending an existing selecting with shift + mouse drag, starting over a #text.</p> |
| +<span style='font-size: 50px; padding: 10px;'>hello world</span> |
| +<script> |
| +test(function() { |
| + var span = document.getElementsByTagName('span')[0]; |
| + span.focus(); |
| + |
| + var dragStartCount = 0; |
| + document.addEventListener('dragstart', function (event) { dragStartCount++; }); |
| + |
| + if (!window.testRunner || !window.eventSender) |
|
yosin_UTC9
2016/06/27 04:21:50
How about:
assert_not_equals(window.testRunner, u
tonikitoo
2016/06/27 15:08:15
Done.
|
| + document.write('This test requires eventSender'); |
| + else { |
| + testRunner.dumpAsText(); |
|
yosin_UTC9
2016/06/27 04:21:50
We don't need to have |testRunner.dumpAsText()| fo
tonikitoo
2016/06/27 15:08:15
Done.
|
| + |
| + var y = span.offsetTop + span.offsetHeight / 2; |
| + eventSender.mouseMoveTo(span.offsetLeft + 5, y); |
| + eventSender.mouseDown(); |
| + eventSender.leapForward(200); |
| + eventSender.mouseUp(); |
| + |
| + eventSender.mouseMoveTo(span.offsetLeft + span.offsetWidth / 4, y); |
| + eventSender.mouseDown(0, ['shiftKey']); |
| + eventSender.leapForward(200); |
| + eventSender.mouseUp(); |
| + |
| + eventSender.mouseMoveTo(span.offsetLeft + span.offsetWidth / 2, y); |
| + eventSender.mouseDown(0, ['shiftKey']); |
| + eventSender.leapForward(200); |
| + eventSender.mouseMoveTo(span.offsetLeft + span.offsetWidth, y); |
| + eventSender.leapForward(200); |
| + eventSender.mouseUp(); |
| + |
| + assert_equals(dragStartCount, 0); |
| + |
| + span.parentNode.removeChild(span); |
|
yosin_UTC9
2016/06/27 04:21:51
|span.remove()| is shorter. BTW, you don't need to
tonikitoo
2016/06/27 15:08:15
Done.
|
| + } |
| +}); |
| +</script> |
