[Laszlo-dev] For Review: Change 20071025-bargull-2 Summary: Bugs related to "destroy"

[Max Carlson]

Approved!

André Bargull wrote:
> Change 20071025-bargull-2 by bargull at dell--p4--2-53 on 2007-10-25 23:59:07
> in /home/Admin/src/svn/openlaszlo/trunk
> for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: Bugs related to "destroy"
> 
> New Features:
> 
> Bugs Fixed:
> LPP-4879 - "Memory leak in "LzUtils.callback""
> 
> Technical Reviewer: max
> QA Reviewer: promanik
> Doc Reviewer: (pending)
> 
> Documentation:
> 
> Release Notes:
> 
> Details:
> "LzView#destroy(..)" must call "this.sprite.destroy(true)", otherwise 
> subsprites weren't destroyed. Whereas subsprites means a sprite's 
> "__children" (DHTML-only).
> Removed "LzNode#__LZsetDefaultHandler(..)": this method is no longer 
> used and ptw says it has been overlooked when LPP-3167 was implemented.
> Removed "LzSprite#predestroy()" (as): this method was defined two times 
> in LzSprite (as), so I've just removed one definition.
> Fixed "LzSprite#destroy(..)" (as): unregistering delegates did not work, 
> moved to "LzSprite.predestroy(..)".
> Updated several components to call "super.destroy.apply(this, 
> arguments)" instead of solely "super.destroy()".
> 
> 
> Tests:
> 
> Files:
> M WEB-INF/lps/lfc/kernel/swf/LzSprite.as
> M WEB-INF/lps/lfc/core/LzNode.lzs
> M WEB-INF/lps/lfc/views/LaszloView.lzs
> M lps/components/lz/basefloatinglist.lzx
> M lps/components/base/basetree.lzx
> M lps/components/base/basecomponent.lzx
> M lps/components/base/basewindow.lzx
> M lps/components/base/submit.lzx
> M lps/components/base/baseformitem.lzx
> M lps/components/base/basetabs.lzx
> 
> Changeset: 
> http://svn.openlaszlo.org/openlaszlo/patches/20071025-bargull-2.tar
> 

-- 
Regards,
Max Carlson
OpenLaszlo.org