LGTM

http://gwt-code-reviews.appspot.com/1296801/diff/28001/29002
File user/src/com/google/gwt/canvas/client/Canvas.java (right):

http://gwt-code-reviews.appspot.com/1296801/diff/28001/29002#newcode64
user/src/com/google/gwt/canvas/client/Canvas.java:64:
CanvasElementSupportDetector.class);
You shouldn't GWT.create() on every call to isSupported.  Create a
static variable and do a null check here, GWT.creating only the first
time. In fact, you might just cache the return value after its called
once.

Sorry I missed this before.

http://gwt-code-reviews.appspot.com/1296801/show

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

Reply via email to