From: ap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Tue, 14 Apr 2015 17:34:03 +0000 (+0000)
Subject: fast/sub-pixel/input-caret-on-subpixel-bound.html is flaky
X-Git-Url: http://git.webkit.org/?p=WebKit-https.git;a=commitdiff_plain;h=e52a4cd98678a5123bc05bdcae1bc31cb1706f34

fast/sub-pixel/input-caret-on-subpixel-bound.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=143697

Reviewed by Anders Carlsson.

* fast/sub-pixel/input-caret-on-subpixel-bound.html: Don't wait for a timer to fire
after removing the last character.


git-svn-id: https://svn.webkit.org/repository/webkit/trunk@182801 268f45cc-cd09-0410-ab3c-d52691b4dbfc
---

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 1102216..406802c 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,5 +1,15 @@
 2015-04-14  Alexey Proskuryakov  <ap@apple.com>
 
+        fast/sub-pixel/input-caret-on-subpixel-bound.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=143697
+
+        Reviewed by Anders Carlsson.
+
+        * fast/sub-pixel/input-caret-on-subpixel-bound.html: Don't wait for a timer to fire
+        after removing the last character.
+
+2015-04-14  Alexey Proskuryakov  <ap@apple.com>
+
         Two flaky tests in http/tests/security/mixedContent
         https://bugs.webkit.org/show_bug.cgi?id=143695
 
diff --git a/LayoutTests/fast/sub-pixel/input-caret-on-subpixel-bound.html b/LayoutTests/fast/sub-pixel/input-caret-on-subpixel-bound.html
index aa2a3f9..2dd08db 100644
--- a/LayoutTests/fast/sub-pixel/input-caret-on-subpixel-bound.html
+++ b/LayoutTests/fast/sub-pixel/input-caret-on-subpixel-bound.html
@@ -35,10 +35,10 @@
             el.focus();
             var step = function() {
                 var value = el.value;
-                if (value) {
+                if (el.value)
                     el.value = value.substr(0, value.length - 1);
+                if (el.value)
                     window.setTimeout(step, 10);
-                }
                 else if (window.testRunner)
                     testRunner.notifyDone();
             };