Closed Bug 1213327 Opened 9 years ago Closed 9 years ago

expose IAccessibleHyperlink on proxied accessibles

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: tbsaunde, Assigned: tbsaunde)

Details

Attachments

(3 files)

      No description provided.
This is not a nice solution, however its basically the only option.  We need to
store the set of interfaces the proxy implementes, preferably without
increasing the size of ProxyAccessibleWrap.
Attachment #8671960 - Flags: review?(dbolter)
Attachment #8671960 - Flags: review?(dbolter) → review+
Attachment #8671961 - Flags: review?(dbolter) → review+
Comment on attachment 8671962 [details] [diff] [review]
expose iAccessibleHyperlink on proxied accessibles

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

::: accessible/windows/ia2/ia2AccessibleHyperlink.cpp
@@ +28,5 @@
>    if (IID_IAccessibleHyperlink == iid) {
> +    auto accWrap = static_cast<AccessibleWrap*>(this);
> +    if (accWrap->IsProxy() ?
> +        !(accWrap->ProxyInterfaces() & Interfaces::HYPERLINK) :
> +        !accWrap->IsLink())

A ternary operator as a condition eh? (I don't mind)
Attachment #8671962 - Flags: review?(dbolter) → review+
Assignee: nobody → tbsaunde+mozbugs
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: