google-code-export / gwt-test-utils

Automatically exported from code.google.com/p/gwt-test-utils
1 stars 0 forks source link

Patch for (user-)DOMImpl.getEventListener missing #65

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
On the user-side DOMImpl, the method getEventListener is not patched. When 
using the RootPanel widget, an exception with the following cause gets thrown:

Caused by: java.lang.UnsatisfiedLinkError: 
com.google.gwt.user.client.impl.DOMImpl.getEventListener(Lcom/google/gwt/dom/cli
ent/Element;)Lcom/google/gwt/user/client/EventListener;
    at com.google.gwt.user.client.impl.DOMImpl.getEventListener(Native Method)
    at com.google.gwt.user.client.DOM.getEventListener(DOM.java:794)
    at com.google.gwt.user.client.Event.getEventListener(Event.java:512)
    at com.google.gwt.user.client.ui.RootPanel.isElementChildOfWidget(RootPanel.java:283)
    at com.google.gwt.user.client.ui.RootPanel.detachOnWindowClose(RootPanel.java:137)
    at com.google.gwt.user.client.ui.RootPanel.get(RootPanel.java:211)
    at com.google.gwt.user.client.ui.RootPanel.get(RootPanel.java:151)

Applied patch:

@PatchMethod(override = true)
static EventListener getEventListener(final DOMImpl domImpl, final Element 
elem) {
  return JavaScriptObjects.getObject(elem, JsoProperties.ELEM_EVENTLISTENER);
}

Version: gwt-test-utils-0.28.6-20110809.075218-9

Original issue reported on code.google.com by stefan.s...@googlemail.com on 11 Aug 2011 at 11:56

GoogleCodeExporter commented 9 years ago

Original comment by gael.laz...@gmail.com on 11 Aug 2011 at 11:58

GoogleCodeExporter commented 9 years ago
I applied your patch and deployed new snapshots. Could you please give me some 
feedback ?

Thanks !

Original comment by gael.laz...@gmail.com on 14 Aug 2011 at 5:23

GoogleCodeExporter commented 9 years ago
Fix works fine on this issue.

Original comment by stefan.s...@googlemail.com on 15 Aug 2011 at 1:09

GoogleCodeExporter commented 9 years ago
Thank you again for your patch :)

Original comment by gael.laz...@gmail.com on 15 Aug 2011 at 1:45

GoogleCodeExporter commented 9 years ago
Thx !

Original comment by gael.laz...@gmail.com on 15 Aug 2011 at 1:58