1336637 - [e10s] Support accessible xpcom relation methods on Windows

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Comment 1

[Eitan Isaacson [:eeejay]]

Attachment: Support relation methods in xpc on windows. r?aklotz

Don't know if I am doing memory management right..

Comment 2

[Trevor Saunders (:tbsaunde)]

Comment on attachment 8833550 [details] [diff] [review]
Support relation methods in xpc on windows. r?aklotz

>+ProxyAccessible::RelationByType(RelationType aType) const
>+{
>+  nsTArray<ProxyAccessible*> proxies;
>+
>+  RefPtr<IAccessible> acc;
>+  if (!GetCOMInterface((void**)getter_AddRefs(acc))) {
>+    return Move(proxies);
>+  }
>+
>+  RefPtr<IAccessible2_2> acc2;
>+  if (FAILED(acc->QueryInterface(IID_IAccessible2_2, (void**)getter_AddRefs(acc2)))) {
>+    return Move(proxies);

you can simplify that with the templated QueryInterface defined in this file.

>+  }
>+
>+  _bstr_t relationType;
>+  for (uint32_t idx = 0; idx < ArrayLength(sRelationTypePairs); idx++) {
>+    if (aType == sRelationTypePairs[idx].first) {
>+      relationType = sRelationTypePairs[idx].second;
>+      break;
>+    }
>+  }
>+  if (!relationType) {

blank line  after } please

>+    return Move(proxies);

just return nsTArray<ProxyAccessible*>() might be nicer.

>+  }
>+
>+  IUnknown** targets;
>+  long nTargets = 0;
>+  HRESULT hr = acc2->get_relationTargetsOfType(relationType, 0, &targets, &nTargets);

I think you want LONG_MAX, 0 is just going to always get you 0 targets.

Comment 3

[Eitan Isaacson [:eeejay]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=bfc8030fd32634f70ae431be68615b0c9e318b4c

Enabled the test as well.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b3ec4674a7fe
Support relation methods in xpc on windows. r=tbsaunde

Comment 5

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/b3ec4674a7fe