http://codereview.appspot.com/136053/diff/1/19
File tests/com/google/caja/demos/applet/CajaAppletTest.java (left):
http://codereview.appspot.com/136053/diff/1/19#oldcode62
Line 62: TestUtil.readResource(getClass(),
"caja-applet-cajita-golden.js"),
Rather than throwing out a useful test, can you run it twice, once with
"use cajita" at the top.
http://codereview.appspot.com/136053/diff/1/20
File tests/com/google/caja/demos/benchmarks/BenchmarkRunner.java
(right):
http://codereview.appspot.com/136053/diff/1/20#newcode185
Line 185: + " 'require cajita';"
What is 'require strict' supposed to do?
http://codereview.appspot.com/136053/diff/1/21
File tests/com/google/caja/demos/benchmarks/BenchmarkSize.java (right):
http://codereview.appspot.com/136053/diff/1/21#newcode200
Line 200: + " 'require cajita';"
ditto
http://codereview.appspot.com/136053/diff/1/6
File tests/com/google/caja/opensocial/DefaultGadgetRewriterTest.java
(right):
http://codereview.appspot.com/136053/diff/1/6#newcode102
Line 102: assertRewritePasses("SocialHelloWorld.xml",
MessageLevel.WARNING);
So this no longer tests it in Cajita mode?
http://codereview.appspot.com/136053/diff/1/12
File tests/com/google/caja/plugin/domita_test.html (right):
http://codereview.appspot.com/136053/diff/1/12#newcode211
Line 211: loadScript('valija.co.js');
Why are we loading valija.co.js even when it's not valija?
http://codereview.appspot.com/136053/diff/1/16
File tests/com/google/caja/plugin/domita_test_untrusted.html (right):
http://codereview.appspot.com/136053/diff/1/16#newcode463
Line 463: /*@@cajaLanguageSelectionStatements@@*/
Ok. Maybe a pointer to the code that does the substitution.
http://codereview.appspot.com/136053/diff/1/15
File tests/com/google/caja/plugin/jsunit.js (right):
http://codereview.appspot.com/136053/diff/1/15#newcode223
Line 223: }
cool
http://codereview.appspot.com/136053
