Looks good to me.
http://codereview.appspot.com/18041/diff/1/11 File ../trunk/java/common/src/main/java/org/apache/shindig/protocol/DefaultHandlerRegistry.java (right): http://codereview.appspot.com/18041/diff/1/11#newcode193 Line 193: sortedSet = Sets.<RestPath>newTreeSet(); <RestPath> is unnecessary here http://codereview.appspot.com/18041/diff/1/11#newcode286 Line 286: RpcInvocationHandler handler; final http://codereview.appspot.com/18041/diff/1/11#newcode411 Line 411: private String operationPath; List<Part>, where Part has a PartType and a String? All these could be made final also (albeit with some code tweaks) http://codereview.appspot.com/18041
