[gwt-contrib] Re: Fixes a memory leak on IE with the cross-site fragment loading strategy. (issue544801)

2010-05-19 Thread jlabanca 

LGTM

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

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

[gwt-contrib] Re: Fixes a memory leak on IE with the cross-site fragment loading strategy. (issue544801)

2010-05-19 Thread spoon 

Thanks!


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

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

[gwt-contrib] Re: Fixes a memory leak on IE with the cross-site fragment loading strategy. (issue544801)

2010-05-18 Thread spoon 

This is ready for review. It's a tiny patch, and either of you would
seem appropriate to me.

Joel, this is a simpler memory leak than the one in the other patch I
sent you.

John, you recently reviewed a similar change to this class having to do
with handling on-load and on-error events.


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

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