LGTM.

http://codereview.appspot.com/2006042/diff/20002/21007
File
java/gadgets/src/test/java/org/apache/shindig/gadgets/parse/caja/VanillaCajaHtmlParserTest.java
(right):

http://codereview.appspot.com/2006042/diff/20002/21007#newcode56
java/gadgets/src/test/java/org/apache/shindig/gadgets/parse/caja/VanillaCajaHtmlParserTest.java:56:
// Bad behavior by Caja DomParser. Bug to be raised with Caja team.
Nitpick: Did you want to put a TODO: here and in the other bad test case
as well, so that these can be spotted easily?

http://codereview.appspot.com/2006042/

Reply via email to