LGTM.  Please follow up with a message to the list that the API has
changed, and give examples.


http://codereview.appspot.com/136053/diff/7001/7039
File src/com/google/caja/cajita-module.js (right):

http://codereview.appspot.com/136053/diff/7001/7039#newcode270
src/com/google/caja/cajita-module.js:270: // this should not happen
Throw an informative exception.

http://codereview.appspot.com/136053/diff/7001/7059
File src/com/google/caja/service/HtmlHandler.java (right):

http://codereview.appspot.com/136053/diff/7001/7059#newcode67
src/com/google/caja/service/HtmlHandler.java:67:
"http://caja.appspot.com/cajoler";;
Good catch

http://codereview.appspot.com/136053/diff/7001/7007
File tests/com/google/caja/a.js (right):

http://codereview.appspot.com/136053/diff/7001/7007#newcode23
tests/com/google/caja/a.js:23: x + y
semicolon

http://codereview.appspot.com/136053/diff/7001/7022
File tests/com/google/caja/c.js (right):

http://codereview.appspot.com/136053/diff/7001/7022#newcode23
tests/com/google/caja/c.js:23: x + 1
semicolon

http://codereview.appspot.com/136053

Reply via email to