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)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
mozilla44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: tbsaunde, Unassigned)
Details
Attachments
(2 files)
1.05 KB,
patch
|
Details | Diff | Splinter Review | |
873 bytes,
patch
|
tbsaunde
:
review+
|
Details | Diff | Splinter Review |
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 ;)
Updated•9 years ago
|
Summary: mkae getChildCountCB correct deal with outerdoc accessibles with proxy children → make getChildCountCB correct deal with outerdoc accessibles with proxy children
Updated•9 years ago
|
Summary: make getChildCountCB correct deal with outerdoc accessibles with proxy children → make getChildCountCB correctly deal with outerdoc accessibles with proxy children
Comment 1•9 years ago
|
||
Attachment #8664424 -
Flags: review?(tbsaunde+mozbugs)
Reporter | ||
Comment 2•9 years ago
|
||
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)
Comment 3•9 years ago
|
||
Attachment #8664467 -
Flags: review?(tbsaunde+mozbugs)
Reporter | ||
Updated•9 years ago
|
Attachment #8664467 -
Flags: review?(tbsaunde+mozbugs) → review+
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c8146f364b03
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in
before you can comment on or make changes to this bug.
Description
•