Update Caja to r4052

Early patch; not yet complete.

In addition to the API update, changes CajaContentRewriter to fetch using HttpFetcher.

[Jasvir, 2010-04-25 20:40:26]

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

http://codereview.appspot.com/846046/diff/1/8#newcode180
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/CajaContentRewriter.java:180:
} else if ("javascript".equalsIgnoreCase(uri.getScheme())) {
This rev of Caja no longer requires specific handling of javascript: urls.

http://codereview.appspot.com/846046/diff/1/5
File
java/gadgets/src/test/java/org/apache/shindig/gadgets/parse/AbstractParsingTestBase.java
(right):

http://codereview.appspot.com/846046/diff/1/5#newcode50
java/gadgets/src/test/java/org/apache/shindig/gadgets/parse/AbstractParsingTestBase.java:50:
is = new FileInputStream(new
File("/Users/fargo/dev/shindig4/java/gadgets/src/test/resources/" + path));
Debugging?

http://codereview.appspot.com/846046/diff/1/15
File pom.xml (right):

http://codereview.appspot.com/846046/diff/1/15#newcode1381
pom.xml:1381: <version>r3950</version>
Update rev?

[Jasvir, 2010-05-04 17:46:12]

ping?

[johnfargo, 2010-05-04 18:41:51]

On 2010/04/25 20:40:26, jasvir wrote:
> http://codereview.appspot.com/846046/diff/1/8
> File
>
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/CajaContentRewriter.java
> (right):
> 
> http://codereview.appspot.com/846046/diff/1/8#newcode180
>
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/CajaContentRewriter.java:180:
> } else if ("javascript".equalsIgnoreCase(uri.getScheme())) {
> This rev of Caja no longer requires specific handling of javascript: urls.

else-block removed.

> 
> http://codereview.appspot.com/846046/diff/1/5
> File
>
java/gadgets/src/test/java/org/apache/shindig/gadgets/parse/AbstractParsingTestBase.java
> (right):
> 
> http://codereview.appspot.com/846046/diff/1/5#newcode50
>
java/gadgets/src/test/java/org/apache/shindig/gadgets/parse/AbstractParsingTestBase.java:50:
> is = new FileInputStream(new
> File("/Users/fargo/dev/shindig4/java/gadgets/src/test/resources/" + path));
> Debugging?

Yep, will be removed before commit.

> 
> http://codereview.appspot.com/846046/diff/1/15
> File pom.xml (right):
> 
> http://codereview.appspot.com/846046/diff/1/15#newcode1381
> pom.xml:1381: <version>r3950</version>
> Update rev?

Strange, should have been in there already. Updated patch en route shortly.

[Jasvir, 2010-05-04 19:59:19]

http://codereview.appspot.com/846046/diff/1/15
File pom.xml (right):

http://codereview.appspot.com/846046/diff/1/15#newcode1381
pom.xml:1381: <version>r3950</version>
You'll also need htmlparser which has now been split out into a separate maven
dependency:
      <dependency>
        <groupId>caja</groupId>
        <artifactId>htmlparser</artifactId>
        <version>r4052</version>
        <scope>compile</scope>
      </dependency>


On 2010/04/25 20:40:26, jasvir wrote:
> Update rev?

[fargo, 2010-05-04 20:00:41]

Adding and trying out the tests once more..

On Tue, May 4, 2010 at 12:59 PM, <jasvir@gmail.com> wrote:

>
> http://codereview.appspot.com/846046/diff/1/15
> File pom.xml (right):
>
> http://codereview.appspot.com/846046/diff/1/15#newcode1381
> pom.xml:1381: <version>r3950</version>
> You'll also need htmlparser which has now been split out into a separate
> maven dependency:
>      <dependency>
>        <groupId>caja</groupId>
>        <artifactId>htmlparser</artifactId>
>        <version>r4052</version>
>        <scope>compile</scope>
>      </dependency>
>
>
>
> On 2010/04/25 20:40:26, jasvir wrote:
>
>> Update rev?
>>
>
> http://codereview.appspot.com/846046/show
>