LGTM


http://codereview.appspot.com/126062/diff/2028/3041
File src/com/google/caja/plugin/templates/SafeHtmlMaker.java (right):

http://codereview.appspot.com/126062/diff/2028/3041#newcode60
Line 60: * The {...@link #make()} method of this class produces two pieces
of content: safe
Add as much commentary on this as you can, please.

http://codereview.appspot.com/126062/diff/2028/3049
File src/com/google/caja/service/CajaArguments.java (right):

http://codereview.appspot.com/126062/diff/2028/3049#newcode59
Line 59: * The Caja language transform that is being requested. Valid
values are:
Maybe point out that the Cajita transform only applies to .js files,
while the valija transform only applies to .html (though it can refer to
.css and .js)?

http://codereview.appspot.com/126062/diff/2028/3044
File src/com/google/caja/service/ContentHandler.java (right):

http://codereview.appspot.com/126062/diff/2028/3044#newcode56
Line 56: * @param checker Used to check whheter two content-types are
compatible
whether

http://codereview.appspot.com/126062

Reply via email to