final nits, should just be 1 more iteration.
http://codereview.appspot.com/1712043/diff/19001/20005 File java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/AccelUriManager.java (right): http://codereview.appspot.com/1712043/diff/19001/20005#newcode32 java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/AccelUriManager.java:32: final String PROXY_PATH_PARAM = DefaultProxyUriManager.PROXY_PATH_PARAM; can be public, should be static. http://codereview.appspot.com/1712043/diff/19001/20006 File java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultAccelUriManager.java (right): http://codereview.appspot.com/1712043/diff/19001/20006#newcode46 java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultAccelUriManager.java:46: ACCEL_PATH = config.getString(CONTAINER, DefaultProxyUriManager.PROXY_PATH_PARAM); nit: for local non-static variables, use camelCase variables starting w/ lowerCase letters http://codereview.appspot.com/1712043/diff/19001/20003 File java/gadgets/src/test/java/org/apache/shindig/gadgets/uri/AccelUriManagerTest.java (right): http://codereview.appspot.com/1712043/diff/19001/20003#newcode27 java/gadgets/src/test/java/org/apache/shindig/gadgets/uri/AccelUriManagerTest.java:27: * TODO: Complete or remove. I'd remove this now; AccelUriManager is not testable. DefaultAccelUriManager however is ;) http://codereview.appspot.com/1712043/show
