Opened 9 years ago

Last modified 9 years ago

#15851 closed defect

_Widget.destroy(): release all handles returned from _Widget.on() — at Version 2

Reported by: bill Owned by: bill
Priority: undecided Milestone: 1.8.1
Component: Dijit Version: 1.8.0
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

To safeguard against memory leaks, and to avoid any funny business when calling destroy(true) (which destroys the widget but preserves the DOM), maybe _Widget.on() should call this.own() on every handle it returns.

Also, Destroyable probably shouldn't set handle._odh (storing it's own handle inside the original handle) because it creates an unnecessary limitation that a given handle can only be owned by one Destroyable object. Rather, _odb can be a variable in the closure.

Change History (2)

comment:1 Changed 9 years ago by bill

Milestone: tbd2.0

comment:2 Changed 9 years ago by bill

Description: modified (diff)
Note: See TracTickets for help on using tickets.