Closed
Bug 965238
Opened 12 years ago
Closed 12 years ago
--disable-accessibility build fails with: layout/xul/nsMenuFrame.cpp:682:64: error: no member named 'anonid' in 'nsGkAtoms'
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla29
People
(Reporter: MatsPalmgren_bugz, Assigned: Gijs)
References
Details
(Keywords: regression)
Attachments
(1 file)
1.94 KB,
patch
|
enndeakin
:
review+
Gijs
:
checkin+
|
Details | Diff | Splinter Review |
layout/xul/nsMenuFrame.cpp:682:64: error: no member named 'anonid' in 'nsGkAtoms'
doc->GetAnonymousElementByAttribute(mContent, nsGkAtoms::anonid, id);
~~~~~~~~~~~^
Reporter | ||
Comment 1•12 years ago
|
||
Looks like bug 941409 broke this. I guess we need to move "anonid"
out of the "#ifdef ACCESSIBILITY" block in nsGkAtomList.h.
Blocks: 941409
Keywords: regression
Assignee | ||
Comment 2•12 years ago
|
||
This should do the trick, right?
Attachment #8367301 -
Flags: review?(bugs)
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Comment 3•12 years ago
|
||
Comment on attachment 8367301 [details] [diff] [review]
trivial patch to move anonid atom out of a11y ifdef,
Yes, let's just fix this
Attachment #8367301 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 4•12 years ago
|
||
I should read bugmail better. Anyhow:
remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/523a1736de8f
And then backed out and relanded to fix the commit msg:
remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/866823b9979e
remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/97540c3ca68d
Assignee | ||
Updated•12 years ago
|
Attachment #8367301 -
Flags: checkin+
Comment 5•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Updated•12 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•