Reviewers: pdr,

Description:
Fixing a bug in Image where IE throws a native JavaScript exception if
an image is attached and detached immediately. The problem occurs when
we try to dispatch a synthetic load event on an unattached element,
which IE doesn't like. Now, if we detect that the widget is detached, we
mark an expando so that we trigger the event the next time the image is
attached.

Also, I fixed a bug where we were firing multiple load events after
multiple synchronous changes to a clipped state when we should only fire
one load event. The test cases were incorrectly testing that we received
multiple events, but all four load events would fire AFTER the final
synchronous change, which is not useful, and contradictary to how load
events are fired if you change the src of an img multiple times in a
single event loop.


Please review this at http://gwt-code-reviews.appspot.com/1421801/

Affected files:
  M user/src/com/google/gwt/user/client/ui/Image.java
  M user/test/com/google/gwt/user/client/ui/ImageTest.java


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

Reply via email to