LGTM modulo comments.
https://codereview.appspot.com/6782129/diff/1/src/com/google/caja/lang/html/HtmlDefinitions.java File src/com/google/caja/lang/html/HtmlDefinitions.java (right): https://codereview.appspot.com/6782129/diff/1/src/com/google/caja/lang/html/HtmlDefinitions.java#newcode482 src/com/google/caja/lang/html/HtmlDefinitions.java:482: if (currentDate.indexOf("*/") >= 0) { I think this is stale code; the stuff below may originally have been in a /* */ delimited comment. With a single-line comment, we should test for \n as a terminator. https://codereview.appspot.com/6782129/diff/1/src/com/google/caja/lang/html/HtmlDefinitions.java#newcode583 src/com/google/caja/lang/html/HtmlDefinitions.java:583: // TODO(kpreid): This is duplicative of Builder.build and looks like it You said this TODO has been done. https://codereview.appspot.com/6782129/
