Instead of having a setHandlerManager() method, which can be called
twice, I think we should have a protected createHandlerManager() method
that is called from ensureHandlerManager().  That way, we can ensure it
is only called once:

HandlerManager createHandlerManager() {
  return new HandlerManager(this);
}

I think the protected getHandlerManager() method makes sense.


http://gwt-code-reviews.appspot.com/138801/diff/1/2
File user/src/com/google/gwt/user/client/ui/Widget.java (right):

http://gwt-code-reviews.appspot.com/138801/diff/1/2#newcode247
Line 247: * Returns the handler manager
Checkstyle requires a period at the end of this sentence.

http://gwt-code-reviews.appspot.com/138801

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to