1127402 - implement proxying relations

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Comment 1

[Trevor Saunders (:tbsaunde)]

Attachment: proxy RelationByType method

Comment 2

[David Bolter [:davidb] (NeedInfo me for attention)]

Comment on attachment 8556530 [details] [diff] [review]
proxy RelationByType method

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

r=me. It took me a while to read this code and there was some 'hmm I assume this is right' that happened. Also, I think I have achieved my aspirations of being a human lint.

::: accessible/atk/AccessibleWrap.cpp
@@ +991,5 @@
> +      atkRelation = atk_relation_new(wrappers.Elements(), wrappers.Length(),
> +                                     atkType);
> +      atk_relation_set_add(relation_set, atkRelation);
> +      g_object_unref(atkRelation);
> +  }

nit: indent above } two spaces.

::: accessible/ipc/DocAccessibleChild.cpp
@@ +174,5 @@
> +    return false;
> +
> +#define RELATIONTYPE(gecko, s, a, m, i) AddRelation(acc, RelationType::gecko, aRelations);
> +
> +#include "RelationTypeMap.h"

nit: Why the spacer line?

::: accessible/ipc/PDocAccessible.ipdl
@@ +62,1 @@
>  prio(high) sync TextSubstring(uint64_t aID, int32_t aStartOffset, int32_t

might as well indent the above two lines.

::: accessible/ipc/ProxyAccessible.cpp
@@ +105,5 @@
> +  size_t targetCount = targetIDs.Length();
> +  nsTArray<ProxyAccessible*> targets(targetCount);
> +      for (size_t i = 0; i < targetCount; i++)
> +      if (ProxyAccessible* proxy = mDoc->GetAccessible(targetIDs[i]))
> +      targets.AppendElement(proxy);

the 'for' is indented 4 spaces too many, the 'if' in indented 2 spaces too many.

@@ +131,5 @@
> +
> +    size_t targetCount = ipcRelations[i].Targets().Length();
> +    nsTArray<ProxyAccessible*> targets(targetCount);
> +    for (size_t j = 0; j < targetCount; j++)
> +      if (ProxyAccessible* proxy = mDoc->GetAccessible(ipcRelations[i].Targets()[j]))

line above needs indent.

Comment 3

[Trevor Saunders (:tbsaunde)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5186614bfe01

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5186614bfe01