Fix changes to caja api calling convention that break caja gadgets

* removes libs=caja from url params in sample container - the behavior for this appears to have changed in shindig
* changes to CajaContentRewriter to ensure error messages from caja are properly escaped
* changes to CajaContentRewriter to handle the new static html output

[Paul Lindner, 2009-07-22 20:06:16]

Nothing obviously wrong here.

Please create a jira issue and I can apply this patch.

Thanks!
Paul

[etnu00, 2009-07-22 20:22:18]

http://codereview.appspot.com/96079/diff/1/2
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/CajaContentRewriter.java
(right):

http://codereview.appspot.com/96079/diff/1/2#newcode93
Line 93: String origContent = content.getContent();
So this code is still serializing the DOM, caja is parsing it, then
re-serializing it, then we're parsing again. This is the primary reason why caja
still can't be used in production on a lot of properties. I was under the
impression that we were going to fix this so that caja understood a DOM as
input.

http://codereview.appspot.com/96079/diff/1/2#newcode102
Line 102: Document doc = content.getDocument();
These two lines will result yet another parse. content.setContent("") should be
removed. If a problem occurs when cajoling, handle it (put it in a finally block
if you want to be really sure).

[Jasvir, 2009-07-22 21:22:05]

Snapshot

http://codereview.appspot.com/96079/diff/1/2
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/CajaContentRewriter.java
(right):

http://codereview.appspot.com/96079/diff/1/2#newcode93
Line 93: String origContent = content.getContent();
Ok.  The next release of Caja will switch to accepting and outputting DOM nodes
directly.

http://codereview.appspot.com/96079/diff/1/2#newcode102
Line 102: Document doc = content.getDocument();
On 2009/07/22 20:22:18, etnu00 wrote:
> These two lines will result yet another parse. content.setContent("") should
be
> removed. If a problem occurs when cajoling, handle it (put it in a finally
block
> if you want to be really sure).

Done.