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.
http://codereview.appspot.com/846046/show
