Upgrade to caja r4291; Add unit tests

This change upgrades to caja r4291 and introduces a few unit tests for CajaContentRewriter.

This version of caja will preserve XMLNS declarations in the DOM Parser which is required by  SHINDIG-1290.

https://issues.apache.org/jira/browse/SHINDIG-1290
http://codereview.appspot.com/2183044/

[Jasvir, 2010-10-01 19:54:46]

http://codereview.appspot.com/2353041/diff/1/java/gadgets/src/test/java/org/a...
File
java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/CajaContentRewriterTest.java
(right):

http://codereview.appspot.com/2353041/diff/1/java/gadgets/src/test/java/org/a...
java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/CajaContentRewriterTest.java:140:
private void testMarkup(String markup, String expected) throws GadgetException{
Golden tests are fragile if new errors and warnings are introduced.  Does it
make sense to split "expected" into a expected output and an array of messages
that actual .contains()?

[chirag, 2010-10-01 22:48:27]

Make tests less fragile

[chirag, 2010-10-01 22:48:50]

http://codereview.appspot.com/2353041/diff/1/java/gadgets/src/test/java/org/a...
File
java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/CajaContentRewriterTest.java
(right):

http://codereview.appspot.com/2353041/diff/1/java/gadgets/src/test/java/org/a...
java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/CajaContentRewriterTest.java:140:
private void testMarkup(String markup, String expected) throws GadgetException{
On 2010/10/01 19:54:46, jasvir wrote:
> Golden tests are fragile if new errors and warnings are introduced.  Does it
> make sense to split "expected" into a expected output and an array of messages
> that actual .contains()?

Done.