1040023 - Broadcast event queues must hold their targets weakly

Status: RESOLVED INCOMPLETE

(Firefox Graveyard :: Shumway, defect, P2)

Description

[Till Schneidereit [:till]]

Currently, any DisplayObject that has any frame event listeners is strongly held by the broadcastEventDispatchQueue. Instead, they have to be weakly held.

Consider this test case (only works in the IDE or debug player):

var clip:MovieClip = new MovieClip();
clip.addEventListener(Event.FRAME_CONSTRUCTED, function() {trace('constructed')});
clip = null;
flash.system.System.gc();

In the Flash Player, this doesn't trace at all, whereas we keep tracing indefinitely.

Comment 1

[Michael Bebenita [:mbx]]

I will try to see if I can use MovieClip reference counts to remove the event handlers from the `broadcastEventDispatchQueue`.

Comment 2

[Till Schneidereit [:till]]

We might not need to fix this for ads: they play for 30 seconds and then stop, so there's probably not much need for weak listeners, etc. Moving to m5 for now.