[Laszlo-dev] [Fwd: For Review: Change 20071003-bargull-8 Summary: basewindow does not properly update "windowlist"]

[André Bargull]

Max, I still need your approval for this change before I can commit it. 
It's already QA reviewed by Phil R.

-------- Original-Message --------
Subject: 	For Review: Change 20071003-bargull-8 Summary: basewindow does 
not properly update "windowlist"
Date: 	Wed, 03 Oct 2007 18:08:57 +0200
From: 	André Bargull <a.bargull at intensis.de>
To: 	Max Carlson <max at openlaszlo.org>, promanik at laszlosystems.com
CC: 	laszlo-dev at openlaszlo.org



[ This change was accidentally partially committed for LPP-4758, so you 
MUST review this change against rev. #6558 to see all changes! Sorry for 
my stupid mistake! (+.+)(-.-)(_ _) ]

Change 20071003-bargull-8 by bargull at dell--p4--2-53 on 2007-10-03 17:45:40
in /home/Admin/src/svn/openlaszlo/trunk
for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: basewindow does not properly update "windowlist"

New Features:

Bugs Fixed:
LPP-4617 - basewindow does not properly update "windowlist"

Technical Reviewer: max
QA Reviewer: promanik
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
Added check in basewindow#setVisible(..) to ensure the window is part of 
its parent's windowlist.
Updated basewindow#sendInFrontOf(..) and basewindow#sendBehind(..) to 
protect against bad values.


Tests:
see testcase at LPP-4617

Files:
M lps/components/base/basewindow.lzx

Changeset: 
http://svn.openlaszlo.org/openlaszlo/patches/20071003-bargull-8.tar