Opened 12 years ago

Closed 12 years ago

#2691 closed defect (fixed)

a11y.js leaks in IE

Reported by: Michael Schall Owned by: Becky Gibson
Priority: high Milestone:
Component: Accessibility Version: 0.4.2
Keywords: Cc:
Blocked By: Blocking:

Description

Any call to a dom node's removeChild will leak in IE. In a11y.js leaks on the line:

dojo.body().removeChild(div);

It needs to be

dojo.html.destroyNode(div);

Attachments (1)

a11y.patch (302 bytes) - added by Michael Schall 12 years ago.
patch for leak

Download all attachments as: .zip

Change History (3)

Changed 12 years ago by Michael Schall

Attachment: a11y.patch added

patch for leak

comment:1 Changed 12 years ago by sjmiles

(In [8307]) Check for built-in constructors that do not expose "apply". Refs #2691.

comment:2 Changed 12 years ago by Becky Gibson

Resolution: fixed
Status: newclosed

(In [8860]) from schallm (cla on file) fixes #2691 - use remove node. Fixes #2177 - use dojo.doc(). refs #3196 (may still be applicable in dijit) check for null cStyle obj

Note: See TracTickets for help on using tickets.