Open
Bug 735921
Opened 12 years ago
Updated 2 years ago
kill nsAccessibleWrap::ShutdownAtkObject()
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
NEW
People
(Reporter: tbsaunde, Unassigned)
Details
Attachments
(1 file)
|
3.05 KB,
patch
|
eeejay
:
review+
ginnchen+exoracle
:
review-
|
Details | Diff | Splinter Review |
No description provided.
|
Reporter | |
Comment 1•12 years ago
|
||
|
|
Reporter | |
Updated•12 years ago
|
Attachment #606025 -
Flags: review?(eitan)
Does nsApplicationAccessibleWrap() has a mDoc? Can we get to Shutdown() if it doesn't?
|
||
Comment 3•12 years ago
|
||
Comment on attachment 606025 [details] [diff] [review] kill nsAccessibleWrap::ShutdownAtkObject() This looks correct.
Attachment #606025 -
Flags: review?(eitan) → review+
|
||
Updated•12 years ago
|
Attachment #606025 -
Flags: review?(ginn.chen)
|
|
Reporter | |
Comment 4•12 years ago
|
||
(In reply to Ginn Chen from comment #2) > Does nsApplicationAccessibleWrap() has a mDoc? > Can we get to Shutdown() if it doesn't? we shouldn't need to, we explicitly shutdown() the application accessible in nsAccessNode::ShutdownXPAcessibility()
Comment on attachment 606025 [details] [diff] [review] kill nsAccessibleWrap::ShutdownAtkObject() In ShutdownXPAccessibility() it calls nsApplicationAccessible::Shutdown(), but it doesn't go to nsAccessibleWrap::Shutdown(). So we leaks.
Attachment #606025 -
Flags: review?(ginn.chen) → review-
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•