loadScript may be asynchronous, so give the impl a bit of time to do the callback
authorJaroslav Tulach <jaroslav.tulach@netbeans.org>
Wed, 05 Feb 2014 13:37:51 +0100
changeset 52975669f440267
parent 528 289d1a80c244
child 530 ce0da3e90343
loadScript may be asynchronous, so give the impl a bit of time to do the callback
json-tck/src/main/java/net/java/html/js/tests/JavaScriptBodyTest.java
json-tck/src/main/java/net/java/html/js/tests/Later.java
     1.1 --- a/json-tck/src/main/java/net/java/html/js/tests/JavaScriptBodyTest.java	Wed Feb 05 13:26:19 2014 +0100
     1.2 +++ b/json-tck/src/main/java/net/java/html/js/tests/JavaScriptBodyTest.java	Wed Feb 05 13:37:51 2014 +0100
     1.3 @@ -293,6 +293,9 @@
     1.4          p.loadScript(new StringReader(
     1.5              "window.later();"
     1.6          ));
     1.7 +        for (int i = 0; i < 100 && l.call != 42; i++) {
     1.8 +            Thread.sleep(50);
     1.9 +        }
    1.10          assert l.call == 42 : "Method was called: " + l.call;
    1.11      }
    1.12      
     2.1 --- a/json-tck/src/main/java/net/java/html/js/tests/Later.java	Wed Feb 05 13:26:19 2014 +0100
     2.2 +++ b/json-tck/src/main/java/net/java/html/js/tests/Later.java	Wed Feb 05 13:37:51 2014 +0100
     2.3 @@ -49,7 +49,7 @@
     2.4   * @author Jaroslav Tulach <jtulach@netbeans.org>
     2.5   */
     2.6  public final class Later {
     2.7 -    int call;
     2.8 +    volatile int call;
     2.9  
    2.10      @JavaScriptBody(args = {  }, javacall = true, body = 
    2.11          "var self = this;"