Opened 10 years ago

Closed 9 years ago

#9887 closed defect (fixed)

dojox FLVideo: detroy() doesn't remove widget

Reported by: petzy Owned by: Mike Wilcox
Priority: high Milestone: 1.6
Component: Dojox Version: 1.3.2
Keywords: dojox flvideo Cc:
Blocked By: Blocking:

Description

DojoX Audio/Video? Version 0.1 destroy() function doesn't remove the widget

Sample code:

var video = document.createElement('div');

video.id = "vid";

p.appendChild(video);

var newVideo = new dojox.av.FLVideo({initialVolume:.7, mediaUrl:'video/sample.flv', autoPlay:true, isDebug:true}, "vid");

do something here

var oldVideo = dijit.byId('vid');

oldVideo.destroy();

oldVideo is not destroyed or affected at all

Change History (4)

comment:1 Changed 10 years ago by Mike Wilcox

Owner: changed from anonymous to Mike Wilcox

comment:2 Changed 10 years ago by Mike Wilcox

Milestone: tbdfuture

comment:3 Changed 9 years ago by Mike Wilcox

Component: GeneralDojox
Milestone: future1.6

There was a bug in FLVideo where it was not cleaned up completely when destroyed. I'll commit that. But the example is not correct. It should be newVideo.destroy(); I'm not sure what the example supplied would do.

comment:4 Changed 9 years ago by Mike Wilcox

Resolution: fixed
Status: newclosed

(In [23406]) Fixes #9887 - doing a more thorough destroy by cleaning up a timer and connect.

Note: See TracTickets for help on using tickets.