The default bug view has changed. See this FAQ.

remove ROLE_SYSTEM_ definitions

RESOLVED FIXED in mozilla15

Status

()

Core
Disability Access APIs
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: surkov, Assigned: maxli)

Tracking

(Blocks: 1 bug)

unspecified
mozilla15
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=eitan@monotonous.org][lang=c++])

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
they were introduced in 2006 to fix tbox bustage (http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=/cvsroot&subdir=mozilla/accessible/src/msaa&command=DIFF_FRAMESET&file=nsRoleMap.h&rev2=1.2&rev1=1.1), I don't think we need them nowdays
(Reporter)

Comment 1

5 years ago
they are in msaa/nsAccessibleWrap.cpp

Comment 2

5 years ago
VC7.1 has them, but VC6 didn't.
(Reporter)

Comment 3

5 years ago
(In reply to neil@parkwaycc.co.uk from comment #2)
> VC7.1 has them, but VC6 didn't.

ok, thank you. so we are on safe side then: vc6 is not officially supported for trunk (https://developer.mozilla.org/En/Developer_Guide/Build_Instructions/Windows_Prerequisites)
(Assignee)

Comment 4

5 years ago
Created attachment 619806 [details] [diff] [review]
Patch v1
Assignee: nobody → maxli
Attachment #619806 - Flags: review?(eitan)
Comment on attachment 619806 [details] [diff] [review]
Patch v1

Review of attachment 619806 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good. Thanks!
Attachment #619806 - Flags: review?(eitan) → review+
(Reporter)

Comment 6

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/03ca8a717fd4
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/03ca8a717fd4
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.