Chromium Code Reviews Chromium Code Reviews
Issue 1541083002:
Fix invalid selection produced when dragging mouse outside the SVG text element (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: third_party/WebKit/LayoutTests/svg/text/selection-dragging-outside-1.html |
| diff --git a/third_party/WebKit/LayoutTests/svg/text/selection-dragging-outside-1.html b/third_party/WebKit/LayoutTests/svg/text/selection-dragging-outside-1.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..0d883ea4cf35015ed51fb6259be5ee9cc514077d |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/svg/text/selection-dragging-outside-1.html |
| @@ -0,0 +1,45 @@ |
| +<!DOCTYPE html> |
| +<svg version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="400" height="400" style="border: 1px solid black;"> |
| + <g> |
| + <text x="130" y="30" font-family="Arial" font-size="10" fill="black">Happy, Web</text> |
| + <text x="130" y="50" font-family="Arial" font-size="10" fill="black">Happy, SVG</text> |
| + <text x="130" y="70" font-family="Arial" font-size="10" fill="black">Happy, ;)link</text> |
| + </g> |
| + <text id="line" x="130" y="150" font-family="Arial" font-size="10" fill="black">Hello World. Hello, SVG.</text> |
| +</svg> |
| +<script src="../../resources/js-test.js"></script> |
| +<script src="resources/SelectionTestCase.js"></script> |
| +<script type="text/javascript"> |
| +var range; |
| +function runTest() { |
| + selectTextFromCharToPoint({ id: 'line', offset: 0 }, { x: 300, y: 300 }, { start: "0", end: "24" }); |
| + selectTextFromCharToPoint({ id: 'line', offset: 6 }, { x: 300, y: 300 }, { start: "6", end: "24" }); |
| + selectTextFromCharToPoint({ id: 'line', offset: 12 }, { x: 300, y: 300 }, { start: "12", end: "24" }); |
| +} |
| + |
| +function selectTextFromCharToPoint(selectionInfo, mouse, expectedRange) { |
| + var element = document.getElementById(selectionInfo.id); |
| + var startPos = element.getStartPositionOfChar(selectionInfo.offset); |
| + var absStartPos = toAbsoluteCoordinates(startPos, element); |
| + if (window.eventSender) { |
| + eventSender.mouseMoveTo(absStartPos.x, absStartPos.y); |
| + eventSender.mouseDown(); |
| + eventSender.mouseMoveTo(mouse.x, mouse.y); |
| + eventSender.mouseUp(); |
| + } |
| + verify(expectedRange.start, expectedRange.end); |
| +} |
| + |
| +function verify(start, end) { |
|
fs
2016/01/25 15:31:28
Just include this directly in the above. No need f
|
| + range = window.getSelection().getRangeAt(0); |
| + shouldBe("range.startOffset", start); |
| + shouldBe("range.endOffset", end); |
| + if (window.eventSender) { |
| + eventSender.mouseMoveTo(0,0); |
| + eventSender.mouseDown(); |
| + eventSender.mouseUp(); |
| + } |
| +} |
| + |
| +runTest(); |
| +</script> |
