LGTM
On 2011/04/05 17:35:36, fredsa wrote:
http://gwt-code-reviews.appspot.com/1398802/diff/1/user/src/com/google/gwt/event/dom/client/DragExitEvent.java
File user/src/com/google/gwt/event/dom/client/DragExitEvent.java
(right):
http://gwt-code-reviews.appspot.com/1398802/diff/1/user/src/com/google/gwt/event/dom/client/DragExitEvent.java#newcode41
user/src/com/google/gwt/event/dom/client/DragExitEvent.java:41: *
{@link
DomEvent#fireNativeEvent(com.google.gwt.dom.client.NativeEvent,
com.google.gwt.event.shared.HasHandlers)}
On 2011/04/04 14:43:23, pdr wrote:
> Line > 100 chars.
Done.
http://gwt-code-reviews.appspot.com/1398802/diff/1/user/test/com/google/gwt/user/client/DragAndDropEventsSinkTest.java
File
user/test/com/google/gwt/user/client/DragAndDropEventsSinkTest.java
(right):
http://gwt-code-reviews.appspot.com/1398802/diff/1/user/test/com/google/gwt/user/client/DragAndDropEventsSinkTest.java#newcode176
user/test/com/google/gwt/user/client/DragAndDropEventsSinkTest.java:176:
delayTestFinish(100);
On 2011/04/04 14:43:23, pdr wrote:
> This is just 100ms, any reason not to bump that to 1000ms / 1s?
Done.
http://gwt-code-reviews.appspot.com/1398802/
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
