LGTM

http://codereview.appspot.com/4591043/diff/17001/src/com/google/caja/parser/html/Nodes.java
File src/com/google/caja/parser/html/Nodes.java (right):

http://codereview.appspot.com/4591043/diff/17001/src/com/google/caja/parser/html/Nodes.java#newcode646
src/com/google/caja/parser/html/Nodes.java:646: } while (problemIndex !=
-1);
With the null break above, this appears to terminate however we may want
to consider adding a max counter on this loop.

http://codereview.appspot.com/4591043/

Reply via email to