Verify spellcheck attribute asynchronously

LayoutTests/editing/spelling/script-tests/spelling-attribute-change.js

 description('Tests if the spellchecker behavior change after the spellcheck attribute changed by the script.');
 
+jsTestIsAsync = true;
+
+if (window.internals) {
+    internals.settings.setUnifiedTextCheckerEnabled(true);
+    internals.settings.setAsynchronousSpellCheckingEnabled(true);
+}
+
 var parent = document.createElement("div");
 document.body.appendChild(parent);
 var sel = document.getSelection();
 
 function testSpellCheckingEnabled(target, enabled)
 {
     target.spellcheck = enabled;
 
     if (target.tagName == "SPAN") {
         target.appendChild(document.createTextNode("Hello,"));
         sel.setBaseAndExtent(target, 6, target, 6);
     } else if (target.tagName == "INPUT" || target.tagName == "TEXTAREA") {
         target.focus();
         document.execCommand("InsertText", false, "Hello,");
     }
 
     document.execCommand("InsertText", false, 'z');
     document.execCommand("InsertText", false, 'z');
     document.execCommand("InsertText", false, ' ');
 
     window.target = target;
     shouldBe("target.spellcheck", enabled ? "true" : "false");
-    shouldBe("internals.hasSpellingMarker(document, 6, 2)", enabled ? "true" : "false");
+    if (window.internals)
+        shouldBecomeEqual("internals.hasSpellingMarker(document, 6, 2)", enabled ? "true" : "false", done);
+    else
+        done();
 }
 
 function createElement(tagName, spellcheck)
 {
     var target = document.createElement(tagName);
     if (tagName == "SPAN")
         target.setAttribute("contentEditable", "true");
     if (spellcheck)
         target.setAttribute("spellcheck", spellcheck);
     return target;
 }
 
 function testFor(tagName, initialAttribute, expectation)
 {
     var target = createElement(tagName, initialAttribute);
     parent.appendChild(target);
 
     testSpellCheckingEnabled(target, expectation);
-    parent.innerHTML = "";
 }
 
-// default -> true
-testFor("SPAN", undefined, true);
-// default -> false
-testFor("SPAN", undefined, false);
-// true -> true
-testFor("SPAN", true, true);
-// true -> false
-testFor("SPAN", true, false);
-// false -> true
-testFor("SPAN", false, true);
-// false -> false
-testFor("SPAN", false, false);
+var tests = [
+    function() { testFor("SPAN", undefined, true); }, // default -> true
+    function() { testFor("SPAN", undefined, false); }, // default -> false
+    function() { testFor("SPAN", true, true); }, // true -> true

[tony, 2014/03/10 17:07:56]

I'm not sure there's a lot of value to these comments.  This could also be
["SPAN", true, true] for each test case and the closure created in done().

[groby-ooo-7-16, 2014/03/10 20:14:09]

And while you're simplifying: The testFor function is always called for the same
6 initial/expectation combos - might want to iterate through that in done(),
too.

[grzegorz, 2014/03/11 13:24:32]

Thanks. Please have a look at patch #2 if I properly applied those suggestions.

+    function() { testFor("SPAN", true, false); }, // true -> false
+    function() { testFor("SPAN", false, true); }, // false -> true
+    function() { testFor("SPAN", false, false); }, // false -> false
 
-// default -> true
-testFor("INPUT", undefined, true);
-// default -> false
-testFor("INPUT", undefined, false);
-// true -> true
-testFor("INPUT", true, true);
-// true -> false
-testFor("INPUT", true, false);
-// false -> true
-testFor("INPUT", false, true);
-// false -> false
-testFor("INPUT", false, false);
+    function() { testFor("INPUT", undefined, true); }, // default -> true
+    function() { testFor("INPUT", undefined, false); }, // default -> false
+    function() { testFor("INPUT", true, true); }, // true -> true
+    function() { testFor("INPUT", true, false); }, // true -> false
+    function() { testFor("INPUT", false, true); }, // false -> true
+    function() { testFor("INPUT", false, false); }, // false -> false
 
-// default -> true
-testFor("TEXTAREA", undefined, true);
-// default -> false
-testFor("TEXTAREA", undefined, false);
-// true -> true
-testFor("TEXTAREA", true, true);
-// true -> false
-testFor("TEXTAREA", true, false);
-// false -> true
-testFor("TEXTAREA", false, true);
-// false -> false
-testFor("TEXTAREA", false, false);
+    function() { testFor("TEXTAREA", undefined, true); }, // default -> true
+    function() { testFor("TEXTAREA", undefined, false); }, // default -> false
+    function() { testFor("TEXTAREA", true, true); }, // true -> true
+    function() { testFor("TEXTAREA", true, false); }, // true -> false
+    function() { testFor("TEXTAREA", false, true); }, // false -> true
+    function() { testFor("TEXTAREA", false, false); }, // false -> false
+];
+
+function done()
+{
+    var next = tests.shift();
+    if (next)
+        return setTimeout(next, 0);
+
+    finishJSTest();
+}
+done();
 
 var successfullyParsed = true;