nsBlockAccessible inheritence incorrect with respect to documents

RESOLVED FIXED

Status

()

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

People

(Reporter: Aaron Leventhal, Assigned: Aaron Leventhal)

Tracking

Trunk
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
nsDocAccessible should inherit from nsBlockAccessible.
nsOuterDocAccessible should inherit from nsAccessibleWrap.

It is nsDocAccessible that might have overlapping children. nsOuterDocAccessible
only ever has 1 child, the nsDocAccessible.

This was a mistake I created when I mad 2 accessible objects for the inner and
outer document objects.
(Assignee)

Comment 1

14 years ago
Created attachment 124239 [details] [diff] [review]
Fixes inheritence
(Assignee)

Updated

14 years ago
Attachment #124239 - Flags: superreview?(Henry.Jia)
Attachment #124239 - Flags: review?(kyle.yuan)

Comment 2

14 years ago
Comment on attachment 124239 [details] [diff] [review]
Fixes inheritence

r=kyle
Attachment #124239 - Flags: review?(kyle.yuan) → review+
(Assignee)

Updated

14 years ago
Attachment #124239 - Flags: superreview?(Henry.Jia) → superreview?(alecf)

Comment 3

14 years ago
Comment on attachment 124239 [details] [diff] [review]
Fixes inheritence

sr=alecf
Attachment #124239 - Flags: superreview?(alecf) → superreview+
(Assignee)

Comment 4

14 years ago
checked in
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

14 years ago
Comment on attachment 124239 [details] [diff] [review]
Fixes inheritence

Seeking a= for accessibility patch that has baked on the trunk for 2 weeks.
Attachment #124239 - Flags: approval1.4?

Comment 6

14 years ago
Comment on attachment 124239 [details] [diff] [review]
Fixes inheritence

moving approval request forward.
Attachment #124239 - Flags: approval1.4? → approval1.4.x?

Comment 7

14 years ago
Comment on attachment 124239 [details] [diff] [review]
Fixes inheritence

a=mkaply for 1.4.1.
Attachment #124239 - Flags: approval1.4.x? → approval1.4.x+

Comment 8

14 years ago
please add the fixed1.4.1 keyword when this is checked in.
Flags: blocking1.4.x+

Comment 9

14 years ago
According to Bonsai, this approved patch is still not checked in for 1.4.1.
(Assignee)

Comment 10

14 years ago
I'm not planning to check it in.

I never got enough approvals on other accessibilty bugs to make it worth it. This alone isn't going 
to make 1.4 accessible.

On with accessibility in the trunk I say. Own it and check it in, if you want. Sun might be interested.

Comment 11

14 years ago
CCing mkaply, since he has set the blocking1.4.x+ flag.

Comment 12

14 years ago
Same as Aaron said, we are also interested to move *all* of the accessibility
patches from trunk to 1.4 branch. Only 3 patches won't make the accessibility
feature much better.

Comment 13

14 years ago
Which patches are these? Shouldn't they be "blocking1.4.x+", too?

Comment 14

14 years ago
The list is fairly long:

a= bug 207171: nsBlockAccessible inheritence is wrong, affects
AccessibleObjectFromPoint() -- Must add #include "nsIContent.h to
nsOuterDocAccessible.h
a= bug 207486: should not report state_expanded for non-container tree nodes
a= bug 207554: crash in nshtmlselectoptionaccessible::getaccnextsibling() -
http://climate.netscape.com/reports/SingleIncidentInfo.cfm?dynamicBBID=20543829
a? bug 207950: add capabilities to ISimpleDOMNode -- has what Freedom Sci needs
(patch is messed up, need to make sure it gets get_localInterface)
a? bug 207487: not reporting accessible name for xul <textbox>es
a? bug 208265: accessibility can crash after profile manager window closes
a? bug 208781: No accessible focus events fired for list items in profile manager.
a? bug 208812: fire internal nsIObserver accessibility event for internal xpcom
accessibility clients, don't forget to add nsIAccessibleCaret.h to
widget/src/windows
a= bug 208460: get rid of nsIDOM3Node.h dependency
a? bug 208922: Make nsIAccessNode consumable for in process accessibility clients
a? bug 210069: fix in process accessibility events to not leak
a? bug 209990: fixes more instability
a? bug 208898: Fire accessible events for interal accessibiltiy clients for all
new documents and windows that are created
a? bug 208803: need xp way to destroy doc accessibles in cache when docs go
away. Also works with internal accessibility clients. 

We still have 10 bugs in the waiting list. (I've marked most of them
blocking1.4.x? today).

Updated

14 years ago
Flags: blocking1.4.x+

Updated

14 years ago
Attachment #124239 - Flags: approval1.4.x+
You need to log in before you can comment on or make changes to this bug.