Thanks, but didn't you just caught an issue?

http://codereview.appspot.com/1841045/diff/34001/24002
File
java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/ContentTypeCharsetRemoverRewriterTest.java
(right):

http://codereview.appspot.com/1841045/diff/34001/24002#newcode114
java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/ContentTypeCharsetRemoverRewriterTest.java:114:
+ "<META Content=\"text/html ; charsett=\'hello\'; hello=world\" "
I don't think this should be recognized as charset, it should not be
removed.

http://codereview.appspot.com/1841045/

Reply via email to