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