LGTM++
Ok to check in after dealing with existing comments. http://codereview.appspot.com/91107/diff/1028/18 File src/com/google/caja/parser/quasiliteral/CajitaRewriter.java (right): http://codereview.appspot.com/91107/diff/1028/18#newcode501 Line 501: substitutes="___.markFuncFreeze(function() {\n" Usually, we don't bother to terminate quasi expressions with "\n" ... it's probably good to remain consistent. http://codereview.appspot.com/91107/diff/1028/18#newcode504 Line 504: + " @body\n" Missing semicolon. http://codereview.appspot.com/91107/diff/1028/18#newcode514 Line 514: + " })();") Indent this line one step out. http://codereview.appspot.com/91107/diff/1028/18#newcode537 Line 537: // error messages were logged in the function fetctStaticModule Trailing whitespace on this line. Also, fetchStaticModule mis-spelled. http://codereview.appspot.com/91107/diff/1028/16 File tests/com/google/caja/parser/quasiliteral/CajitaRewriterTest.java (right): http://codereview.appspot.com/91107/diff/1028/16#newcode2304 Line 2304: MessageLevel.FATAL_ERROR); Maybe add a test for the "cannot load dynamic" failure condition? http://codereview.appspot.com/91107
