> So all layout panels leak in IE (all versions) when they're never
> attached to the document.

Turns out that, even if attached but then detached in the same event loop (which
is what Dirk's leak test does), this causes a leak, because:

* layoutPanel.add(child), schedules a deferred layout command
* RootPanel.add(layoutPanel)
  * LayoutImpl.onAttach
* RootPanel.remove(layoutPanel)
  * layoutPanel.onDetach, clears __layer

Next event loop, layout command runs:

* LayoutImpl.layout
  * Sets __layer -- never unset

So, I think some tracking of attached/detached is needed, and __layer shouldn't
be set if the layout is unattached. Make sense? Not sure if that will have
unintended consequences.

- Stephen

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

Reply via email to