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)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

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

Details

(Keywords: access)

Attachments

(1 file)

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".
Attached patch patch v1Splinter Review
In this patch, fix a bug of link. When invoking function of
nsHTMLLinkAccessible[Wrap], mLinkContent must be initialized first.
Attachment #157128 - Flags: review?(aaronleventhal)
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+
Attachment #157128 - Flags: superreview?(Henry.Jia)
Comment on attachment 157128 [details] [diff] [review]
patch v1

sr = Henry
Attachment #157128 - Flags: superreview?(Henry.Jia) → superreview+
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
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.

Attachment

General

Creator:
Created:
Updated:
Size: