Use shouldBecomeEqual in asynchronous spellchecking tests.

LayoutTests/editing/spelling/grammar-markers.html

Index: LayoutTests/editing/spelling/grammar-markers.html
diff --git a/LayoutTests/editing/spelling/grammar-markers.html b/LayoutTests/editing/spelling/grammar-markers.html
index 7d732a7212be6d15d4861e064180f2eab8e7e928..e07e5317ea4858741c7cd6cf35731004420d5d30 100644
--- a/LayoutTests/editing/spelling/grammar-markers.html
+++ b/LayoutTests/editing/spelling/grammar-markers.html
@@ -18,6 +18,15 @@ if (window.internals)
 if (window.testRunner)
     testRunner.dumpAsText(true);
 
+function verifyGrammarMarker(destination, ungrammaticalPhrase)
+{
+    if (window.internals && internals.hasGrammarMarker(document, destination.innerHTML.indexOf(ungrammaticalPhrase), ungrammaticalPhrase.length)) {
+        testPassed("ungrammatical phrase '" + ungrammaticalPhrase + "' on '" + destination.innerHTML + "'");
+        return true;
+    } else
+       return false;

[tony, 2013/09/20 17:10:24]

Nit: Looks like the indent is off by one.

+}
+
 var source = document.getElementById('source');
 window.getSelection().selectAllChildren(source);
 document.execCommand('Copy');
@@ -25,23 +34,7 @@ var target = document.getElementById('target');
 window.getSelection().selectAllChildren(target);
 document.execCommand('Paste');
 
-var retryCount = 10;
-var sleepPeriod = 1;
-var ungrammaticalPhrase = 'has';
-var hasMarker;
-function verifyMarker() {
-    hasMarker = internals.hasGrammarMarker(document, target.innerHTML.indexOf(ungrammaticalPhrase), ungrammaticalPhrase.length);
-    if (!hasMarker && --retryCount > 0) {
-        sleepPeriod *= 2;
-        window.setTimeout(verifyMarker, sleepPeriod);
-        return;
-    }
-
-    shouldBeTrue('hasMarker');
-    finishJSTest();
-};
-
-window.setTimeout(verifyMarker, sleepPeriod);
+shouldBecomeEqual('verifyGrammarMarker(target, "has")', 'true', finishJSTest);
 
 </script>
 <script src="../../fast/js/resources/js-test-post.js"></script>