Closed Bug 239866 Opened 20 years ago Closed 20 years ago

Wrong information exported for HyperText to gnopernicus

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: Louie.Zhao, Assigned: Louie.Zhao)

Details

Attachments

(1 file)

patch v1
2.37 KB, patch
yuanyi21
: review+
Henry.Jia
: superreview+
chofmann
: approval1.7+
Details | Diff | Splinter Review 
Mozilla sometimes report wrong information for HyperText (combination of link
and text), which includes:
1. ignore wrapped link (e.g. link wrapped by <H>)
2. When the cursor is at the tail of "text", Gnopernicus can't report the
content of it
3. "Static text" was concerned as "link" in some cases.

"nsAccessibleHyperText::GetLink" makes assumption that link node is the parent
of text node. Actually link node may be grandparent (or grand-grand-parent, ...)
of text node (e.g. in the testcase, the structure of "Item 3" is "A(link) ->
B(bond) -> Text"). This patch changes the code for "searching link node for text
node".
Attached patch patch v1 — — Splinter Review 
This patch has worked in Sun's internal mozilla codebase for a period of time.
Attachment #145603 - Flags: review?(kyle.yuan)
Comment on attachment 145603 [details] [diff] [review]
patch v1

r=kyle
Attachment #145603 - Flags: review?(kyle.yuan) → review+
Attachment #145603 - Flags: superreview?(Henry.Jia)
Attachment #145603 - Flags: superreview?(Henry.Jia) → superreview+
Attachment #145603 - Flags: approval1.7?
Comment on attachment 145603 [details] [diff] [review]
patch v1

a=chofmann for 1.7
Attachment #145603 - Flags: approval1.7? → approval1.7+
Assignee: aaronleventhal → Louie.Zhao
checked in by louie.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.