LGTM

http://codereview.appspot.com/136053/diff/9157/9203
File src/com/google/caja/parser/js/Directive.java (right):

http://codereview.appspot.com/136053/diff/9157/9203#newcode103
src/com/google/caja/parser/js/Directive.java:103: // Escaping has
modified the directive. Render nothing.
maybe add a comment pointing to the longer discussion in the bug.

http://codereview.appspot.com/136053/diff/9157/9203#newcode107
src/com/google/caja/parser/js/Directive.java:107:
rc.getOut().consume(";");
cool

http://codereview.appspot.com/136053/diff/9157/9175
File tests/com/google/caja/parser/js/parsertest6.js (right):

http://codereview.appspot.com/136053/diff/9157/9175#newcode3
tests/com/google/caja/parser/js/parsertest6.js:3: "use f\u0008oo";  //
Should not be rendered
cool

http://codereview.appspot.com/136053

Reply via email to