You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by jo...@apache.org on 2010/02/22 18:08:25 UTC

svn commit: r912659 - in /shindig/trunk/java: common/src/main/java/org/apache/shindig/common/servlet/ gadgets/src/main/java/org/apache/shindig/gadgets/parse/caja/ gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ gadgets/src/main/java/org/apach...

Author: johnh
Date: Mon Feb 22 17:08:25 2010
New Revision: 912659

URL: http://svn.apache.org/viewvc?rev=912659&view=rev
Log:
Minor cleanups, and adding a few new Param enums.


Modified:
    shindig/trunk/java/common/src/main/java/org/apache/shindig/common/servlet/HttpUtil.java
    shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/caja/CajaCssParser.java
    shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/MakeRequestServlet.java
    shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/UriCommon.java
    shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/image/JPEGOptimizerTest.java

Modified: shindig/trunk/java/common/src/main/java/org/apache/shindig/common/servlet/HttpUtil.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/common/src/main/java/org/apache/shindig/common/servlet/HttpUtil.java?rev=912659&r1=912658&r2=912659&view=diff
==============================================================================
--- shindig/trunk/java/common/src/main/java/org/apache/shindig/common/servlet/HttpUtil.java (original)
+++ shindig/trunk/java/common/src/main/java/org/apache/shindig/common/servlet/HttpUtil.java Mon Feb 22 17:08:25 2010
@@ -105,6 +105,6 @@
   }
 
   public static void setDefaultTtl(int defaultTtl) {
-    defaultTtl = defaultTtl;
+    HttpUtil.defaultTtl = defaultTtl;
   }
 }

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/caja/CajaCssParser.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/caja/CajaCssParser.java?rev=912659&r1=912658&r2=912659&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/caja/CajaCssParser.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/caja/CajaCssParser.java Mon Feb 22 17:08:25 2010
@@ -47,7 +47,6 @@
 import java.io.Writer;
 import java.net.URI;
 import java.util.Collections;
-import java.util.logging.Logger;
 
 /** A CSS DOM parser using Caja. */
 public class CajaCssParser {

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/MakeRequestServlet.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/MakeRequestServlet.java?rev=912659&r1=912658&r2=912659&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/MakeRequestServlet.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/MakeRequestServlet.java Mon Feb 22 17:08:25 2010
@@ -32,7 +32,7 @@
  *
  * GET and POST are supported so as to facilitate improved browser caching.
  *
- * Currently this just delegates to ProxyHandler, which deals with both
+ * Currently this just delegates to MakeRequestHandler, which deals with both
  * makeRequest and open proxy calls.
  */
 public class MakeRequestServlet extends InjectedServlet {

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/UriCommon.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/UriCommon.java?rev=912659&r1=912658&r2=912659&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/UriCommon.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/UriCommon.java Mon Feb 22 17:08:25 2010
@@ -22,6 +22,7 @@
   public static final String USER_PREF_PREFIX = "up_";
   public enum Param {
     URL("url"),
+    GADGET("gadget"),
     CONTAINER("container"),
     VIEW("view"),
     LANG("lang"),
@@ -31,7 +32,10 @@
     VERSION("v"),
     SECURITY_TOKEN("st"),
     MODULE_ID("mid"),
-    LIBS("libs");
+    REFRESH("refresh"),
+    LIBS("libs"),
+    JSON("json"),
+    TYPE("type");
    
     private final String key;
     private Param(String key) {

Modified: shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/image/JPEGOptimizerTest.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/image/JPEGOptimizerTest.java?rev=912659&r1=912658&r2=912659&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/image/JPEGOptimizerTest.java (original)
+++ shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/rewrite/image/JPEGOptimizerTest.java Mon Feb 22 17:08:25 2010
@@ -19,7 +19,6 @@
 
 import org.apache.sanselan.ImageReadException;
 import org.apache.shindig.gadgets.http.HttpResponse;
-import org.junit.Ignore;
 import org.junit.Test;
 
 import java.io.IOException;