Closed Bug 1207253 Opened 9 years ago Closed 9 years ago

make getChildCountCB correctly deal with outerdoc accessibles with proxy children

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: tbsaunde, Unassigned)

Details

Attachments

(2 files)

Consider the case that GetChildCountCB is called on an atk object with an accessible not a proxy.  That accessible might be a outerDoc accessible where the child is a proxied document.  to handle that in the accessible case we need to see if we can convert the Accessible* to an OuterDocAccessible* and then get something when we call RemoteChildDoc() on it.

smaug can you fix this? it should be easy ;)
Summary: mkae getChildCountCB correct deal with outerdoc accessibles with proxy children → make getChildCountCB correct deal with outerdoc accessibles with proxy children
Summary: make getChildCountCB correct deal with outerdoc accessibles with proxy children → make getChildCountCB correctly deal with outerdoc accessibles with proxy children
Attached patch v1Splinter Review
Attachment #8664424 - Flags: review?(tbsaunde+mozbugs)
Comment on attachment 8664424 [details] [diff] [review]
v1

>-    return static_cast<gint>(accWrap->EmbeddedChildCount());
>+    uint32_t count = accWrap->EmbeddedChildCount();
>+    if (count) {
>+      return static_cast<gint>(count);
>+    }
>+
>+    OuterDocAccessible* outerDoc = accWrap->AsOuterDoc();
>+    if (outerDoc) {
>+      proxy = outerDoc->RemoteChildDoc();
>+    }

oops, I should be more careful, this will return the number of children of the child, not the number of children.  if RemoteChildDoc() returns a proxy we should just return 1, because there is one child that RemoteChildDoc.
Attachment #8664424 - Flags: review?(tbsaunde+mozbugs)
Attached patch v2Splinter Review
Attachment #8664467 - Flags: review?(tbsaunde+mozbugs)
Attachment #8664467 - Flags: review?(tbsaunde+mozbugs) → review+
https://hg.mozilla.org/mozilla-central/rev/c8146f364b03
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: