Closed
Bug 965238
Opened 10 years ago
Closed 10 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•10 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•10 years ago
|
||
This should do the trick, right?
Attachment #8367301 -
Flags: review?(bugs)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Comment 3•10 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•10 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•10 years ago
|
Attachment #8367301 -
Flags: checkin+
Comment 5•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/97540c3ca68d
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Updated•10 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•