Closed
Bug 257092
Opened 20 years ago
Closed 19 years ago
GOK can't grab HTML content and XUL Content
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: Louie.Zhao, Assigned: Louie.Zhao)
Details
(Keywords: access)
Attachments
(1 file)
3.54 KB,
patch
|
aaronlev
:
review+
Henry.Jia
:
superreview+
|
Details | Diff | Splinter Review |
GOK can't grab HTML Content (e.g. links in webpage) and some XUL Content (e.g. right part of Preference dialog). This bug is severe because it blocks any bug fix for HTML/XUL element interaction with GOK UI-Grab. Atk object "HTML_CONTAINER" shouldn't have AccessibleText/AccessibleEditableText interface. Otherwise GOK won't go over the children of "HTML_CONTAINER".
Assignee | ||
Comment 1•20 years ago
|
||
In this patch, fix a bug of link. When invoking function of nsHTMLLinkAccessible[Wrap], mLinkContent must be initialized first.
Assignee | ||
Updated•20 years ago
|
Attachment #157128 -
Flags: review?(aaronleventhal)
Comment 2•20 years ago
|
||
Comment on attachment 157128 [details] [diff] [review] patch v1 + nsCOMPtr<nsIAccessibleText> accessInterfaceText; + QueryInterface(NS_GET_IID(nsIAccessibleText), + getter_AddRefs(accessInterfaceText)); Isn't it simpler to do this? nsCOMPtr<nsIAccessibleText> accessInterfaceText = do_QueryInterface(this); I see that pattern throughout the ATK code. Change it if you want, but that's not a condition of r=. I am interested to know why it is different from other parts of Mozilla.
Attachment #157128 -
Flags: review?(aaronleventhal) → review+
Assignee | ||
Updated•20 years ago
|
Attachment #157128 -
Flags: superreview?(Henry.Jia)
Comment on attachment 157128 [details] [diff] [review] patch v1 sr = Henry
Attachment #157128 -
Flags: superreview?(Henry.Jia) → superreview+
Assignee | ||
Comment 4•20 years ago
|
||
Thanks for sr & r. (I can't checkin now because of waiting for my cvs account update) This patch follows the pattern throughout the ATK code. We can change QueryInterface(NS_GET_IID(... to doQueryInterface(... later for all ATK code.
Keywords: access
Assignee | ||
Comment 5•19 years ago
|
||
patch already in trunk. Close this bug.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•