Title: [91458] trunk/LayoutTests
Revision
91458
Author
[email protected]
Date
2011-07-21 03:18:29 -0700 (Thu, 21 Jul 2011)

Log Message

Fix fast/speech/input-text-speechbutton.html after r91249 .
https://bugs.webkit.org/show_bug.cgi?id=64857

Reviewed by Kent Tamura.

* fast/speech/input-text-speechbutton.html: Correct mouse coordinates now that the microphone icon can be on the left in RTL..
* platform/chromium/test_expectations.txt: Test no longer times out.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (91457 => 91458)


--- trunk/LayoutTests/ChangeLog	2011-07-21 10:12:39 UTC (rev 91457)
+++ trunk/LayoutTests/ChangeLog	2011-07-21 10:18:29 UTC (rev 91458)
@@ -1,3 +1,13 @@
+2011-07-21  Jeremy Moskovich  <[email protected]>
+
+        Fix fast/speech/input-text-speechbutton.html after r91249 .
+        https://bugs.webkit.org/show_bug.cgi?id=64857
+
+        Reviewed by Kent Tamura.
+
+        * fast/speech/input-text-speechbutton.html: Correct mouse coordinates now that the microphone icon can be on the left in RTL..
+        * platform/chromium/test_expectations.txt: Test no longer times out.
+
 2011-07-21  Balazs Kelemen  <[email protected]>
 
         Unreviewed gardening for Qt.

Modified: trunk/LayoutTests/fast/speech/input-text-speechbutton.html (91457 => 91458)


--- trunk/LayoutTests/fast/speech/input-text-speechbutton.html	2011-07-21 10:12:39 UTC (rev 91457)
+++ trunk/LayoutTests/fast/speech/input-text-speechbutton.html	2011-07-21 10:18:29 UTC (rev 91458)
@@ -13,10 +13,17 @@
 function startSpeechInput(id) {
     // Clicking the speech button should fill in mock speech-recognized text.
     var input = document.getElementById(id);
-    var x = input.offsetLeft + input.offsetWidth - 4;
+    var isRTL = input.dir == 'rtl';
+    var x = 0;
+    if (isRTL)
+        x = input.offsetLeft + 4;
+    else
+        x = input.offsetLeft + input.offsetWidth - 4;
     var y = input.offsetTop + input.offsetHeight / 2;
-    if (input.style.paddingRight.length)
+    if (!isRTL && input.style.paddingRight.length)
         x -= parseInt(input.style.paddingRight.substr(0, input.style.paddingRight.length - 2));
+    else if (isRTL && input.style.paddingLeft.length)
+        x += parseInt(input.style.paddingLeft.substr(0, input.style.paddingLeft.length - 2));
     eventSender.mouseMoveTo(x, y);
     eventSender.mouseDown();
     eventSender.mouseUp();

Modified: trunk/LayoutTests/platform/chromium/test_expectations.txt (91457 => 91458)


--- trunk/LayoutTests/platform/chromium/test_expectations.txt	2011-07-21 10:12:39 UTC (rev 91457)
+++ trunk/LayoutTests/platform/chromium/test_expectations.txt	2011-07-21 10:18:29 UTC (rev 91458)
@@ -3775,8 +3775,6 @@
 
 BUGWK64788 : http/tests/websocket/tests/hybi/reserved-opcodes.html = TIMEOUT
 
-BUGCR89717 : fast/speech/input-text-speechbutton.html = TIMEOUT
-
 BUGWK64844 DEBUG : svg/custom/crash-textPath-attributes.html = CRASH
 
 BUGWK64874 MAC DEBUG : css3/images/optimize-contrast-canvas.html = CRASH
@@ -3784,3 +3782,5 @@
 BUGRNIWA DEBUG : jquery/css.html = PASS TIMEOUT
 
 BUGWK64920 WIN DEBUG : fast/canvas/webgl/renderbuffer-initialization.html = CRASH PASS
+
+BUGWK64857 LINUX : fast/speech/input-text-speechbutton.html = TEXT
\ No newline at end of file
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to