[Gnash-dev] Re: tests about this context in onUnload handler

[Sandro Santilli]

[ Added gnash-dev in Cc, as this seems of broader interest ]

On Wed, Aug 29, 2007 at 11:25:41AM +0800, zou lunkai wrote:
> Hello, strk
> 
> > Should be checked what happens with 'this' variables in onUnload methods. 
> > (from wiki)
> 
> Already tested, see testsuite/misc-ming.all/unload_movieclip_test1.c

Ok, good. This is still related to soft ref.
The problem is that as_value considers a ref "danglign" when the sprites's 
isUnloaded() function returns true.
In turn, isUnloaded() returns true too early, which is *before* the unload
event handlers are invoked (because they are queued!).

Instead, we may add an isFinishedUnloading() method for sprite_instance,
which should return true *after* all unload event handlers have been invoked.

Isn't straightforward with current design, as we don't konw when the unload
events are going to be executed, and the executor doesn't even know it's
an unload event hanlder (maybe we might add this knowledge in the ExecutableCode
class, which starts having a wrong name)


--strk;

 ()   ASCII Ribbon Campaign
 /\   Keep it simple!