Change nsIAccessibleObjectAttributeChangedEvent.changedAttribute from an nsIAtom to an AString

RESOLVED FIXED in Firefox 57

Status

()

Core
Disability Access APIs
RESOLVED FIXED
6 months ago
6 months ago

People

(Reporter: njn, Assigned: njn)

Tracking

unspecified
mozilla57
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 months ago
This removes one use of nsIAtom within scripts, which is good (see bug
1392883). All the uses of this attribute within scripts just converted it to a
string anyway.
(Assignee)

Comment 1

6 months ago
Created attachment 8900204 [details] [diff] [review]
Change nsIAccessibleObjectAttributeChangedEvent.changedAttribute from an nsIAtom to an AString
Attachment #8900204 - Flags: review?(aklotz)

Comment 2

6 months ago
Comment on attachment 8900204 [details] [diff] [review]
Change nsIAccessibleObjectAttributeChangedEvent.changedAttribute from an nsIAtom to an AString

Stealing review from Aaron, who is primarily working on our E10S Windows MSCOM stuff anyway. Thanks for the cleanup!
Attachment #8900204 - Flags: review?(aklotz) → review+

Comment 3

6 months ago
Thanks Marco!
(Assignee)

Comment 4

6 months ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/73a7172afa6573133131c3e876fd55a5e4d514f1
Bug 1392993 - Change nsIAccessibleObjectAttributeChangedEvent.changedAttribute from an nsIAtom to an AString. r=MarcoZ.

Comment 5

6 months ago
Pushed by nnethercote@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/73a7172afa65
Change nsIAccessibleObjectAttributeChangedEvent.changedAttribute from an nsIAtom to an AString. r=MarcoZ.
https://hg.mozilla.org/mozilla-central/rev/73a7172afa65
Status: ASSIGNED → RESOLVED
Last Resolved: 6 months ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.