LGTM w/ nits.
http://gwt-code-reviews.appspot.com/1384806/diff/2001/user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java File user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java (right): http://gwt-code-reviews.appspot.com/1384806/diff/2001/user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java#newcode96 user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java:96: // implementations in the tests Use a block comment for multi-line comments to allow eclipse to re-flow the text. http://gwt-code-reviews.appspot.com/1384806/diff/2001/user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java#newcode132 user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java:132: public void testInvokeInheritedMethod() { Sort methods. http://gwt-code-reviews.appspot.com/1384806/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors