Fix "mouse" position selection in svg/text/resources/SelectionTestCase.js

LayoutTests/svg/text/resources/SelectionTestCase.js

 if (!window.eventSender || !window.testRunner) {
     alert('This test needs to be run in DRT, to get results!');
 }
 
 var svgRoot = 0;
 
 // Map 'point' into absolute coordinates, usable for eventSender
 function transformPoint(point, matrix) {
     return point.matrixTransform(matrix);
 }
@@ skipping 51 matching lines @@
     }
 
     if (window.eventSender) {
         // Trigger 'partial glyph selection' code, by adjusting the end x position by half glyph width
         var xOld = endPos.x;
         endPos.x -= endExtent.width / 2 - 1;
 
         var absStartPos = toAbsoluteCoordinates(startPos, element);
         var absEndPos = toAbsoluteCoordinates(endPos, element);
 
+        // Round the points "inwards" to avoid being affected by the truncation
+        // taking place in eventSender.mouseMoveTo(...).
+        absStartPos.x = Math.ceil(absStartPos.x);

[hyunjunekim2, 2015/04/28 02:22:07]

Could you change like this?
absStartPos.x = Math.ceil(absStartPos.x) + 4;

[pdr., 2015/04/28 03:08:39]

Why?

+        absEndPos.x = Math.floor(absEndPos.x);
+
         // Move to selection origin and hold down mouse
         eventSender.mouseMoveTo(absStartPos.x, absStartPos.y);
         eventSender.mouseDown();
 
         // Move again to start selection
         eventSender.mouseMoveTo(absStartPos.x, absStartPos.y);
 
         // Move to end location and release mouse
         eventSender.mouseMoveTo(absEndPos.x, absEndPos.y);
         eventSender.mouseUp();
@@ skipping 35 matching lines @@
         var textElement = document.createElementNS("http://www.w3.org/2000/svg", "svg:text");
         textElement.setAttribute("x", "0");
         textElement.setAttribute("y", "35");
         textElement.setAttribute("fill", "red");
         textElement.setAttribute("transform", "scale(0.5)");
         textElement.setAttribute("font-size", "8");
         textElement.textContent = "Expected '" + expectedText + "' to be selected, got: '" + actualText + "'";
         document.getElementById("container").appendChild(textElement);
     }
 }