LGTM
http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/src/com/google/gwt/event/shared/HandlerManager.java File user/src/com/google/gwt/event/shared/HandlerManager.java (right): http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/src/com/google/gwt/event/shared/HandlerManager.java#newcode49 user/src/com/google/gwt/event/shared/HandlerManager.java:49: @com.google.web.bindery.event.shared.SimpleEventBus::doRemove(Lcom/google/web/bindery/event/shared/Event$Type;Ljava/lang/Object;Ljava/lang/Object;) Can you use the JSNI wildcard syntax here? http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/test/com/google/web/bindery/event/shared/BarEvent.java File user/test/com/google/web/bindery/event/shared/BarEvent.java (right): http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/test/com/google/web/bindery/event/shared/BarEvent.java#newcode17 user/test/com/google/web/bindery/event/shared/BarEvent.java:17: Extra blank line. http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/test/com/google/web/bindery/event/shared/EventBusTestBase.java File user/test/com/google/web/bindery/event/shared/EventBusTestBase.java (right): http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/test/com/google/web/bindery/event/shared/EventBusTestBase.java#newcode19 user/test/com/google/web/bindery/event/shared/EventBusTestBase.java:19: Extra blank lines. http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/test/com/google/web/bindery/event/shared/EventSharedSuite.java File user/test/com/google/web/bindery/event/shared/EventSharedSuite.java (right): http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/test/com/google/web/bindery/event/shared/EventSharedSuite.java#newcode29 user/test/com/google/web/bindery/event/shared/EventSharedSuite.java:29: suite.addTestSuite(com.google.web.bindery.event.shared.ResettableEventBusTest.class); Use import statements? http://gwt-code-reviews.appspot.com/1394803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
