723796 - rename nsAccessible::GetAllowsAnonChildAccessibles

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

rename nsAccessible::GetAllowsAnonChildAccessibles to something more nice.

Options:
AreAnonChildrenAllowed - short but not too correct
CanHaveChildrenInAnonContent - doesn't describe that class expects or not expect the anonymous content be accessible
AreChildrenInAnonContentAllowed
AllowsChildrenInAnonContent

Trevor, ideas?

Comment 1

[Trevor Saunders (:tbsaunde)]

(In reply to alexander :surkov from comment #0)
> rename nsAccessible::GetAllowsAnonChildAccessibles to something more nice.
> 
> Options:
> AreAnonChildrenAllowed - short but not too correct
> CanHaveChildrenInAnonContent - doesn't describe that class expects or not
> expect the anonymous content be accessible
> AreChildrenInAnonContentAllowed
> AllowsChildrenInAnonContent

how about CanHaveChildrenForAnonContent()?

> Trevor, ideas?

how about kill it and use a bit in mFlags for the same purpose?

Comment 2

[alexander :surkov (:asurkov)]

(In reply to Trevor Saunders (:tbsaunde) from comment #1)
> (In reply to alexander :surkov from comment #0)
> > rename nsAccessible::GetAllowsAnonChildAccessibles to something more nice.
> > 
> > Options:
> > AreAnonChildrenAllowed - short but not too correct
> > CanHaveChildrenInAnonContent - doesn't describe that class expects or not
> > expect the anonymous content be accessible
> > AreChildrenInAnonContentAllowed
> > AllowsChildrenInAnonContent
> 
> how about CanHaveChildrenForAnonContent()?

mm, ok.

> > Trevor, ideas?
> 
> how about kill it and use a bit in mFlags for the same purpose?

somebody should run through implementations and make sure accessible is recreated when attribute are changed (it'd be nice to write mochitests). Anyway, if we see that accessible is not recreated then it can be a problem (in case if it's children dont' get updated).

Comment 3

[alexander :surkov (:asurkov)]

after chat on irc Trevor and I are fine with CanHaveAnonChildren() name.

Comment 4

[joseph]

Attachment: GetAllowsAnonChildAccessibles renamed to CanHaveAnonChildren

Comment 5

[alexander :surkov (:asurkov)]

Comment on attachment 594702 [details] [diff] [review]
GetAllowsAnonChildAccessibles renamed to CanHaveAnonChildren

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

it appears you didn't change all entries, for example, nsXULComboboxAccessible::GetAllowsAnonChildAccessibles()

::: accessible/src/base/nsAccessible.h
@@ +387,5 @@
>     */
>    virtual nsresult HandleAccEvent(AccEvent* aAccEvent);
>  
>    /**
>     * Return true if there are accessible children in anonymous content

while you're here then please change the comment like:
Return true if this accessible allows accessible children from anonymous subtree.

::: accessible/src/mac/nsAccessibleWrap.h
@@ +98,5 @@
>    /**
>     * Returns this accessible's all children, adhering to "flat" accessibles by 
>     * not returning their children.
>     */
> +  void GetUnignoredChildren(nsTArray< nsAccessibile* > & aChildrenArray);

why is this change?

Comment 6

[joseph]

Attachment: This time, all GetAllowsAnonChildAccessibles() methods renamed to CanHaveAnonChildren()

Comment 7

[alexander :surkov (:asurkov)]

Comment on attachment 595165 [details] [diff] [review]
This time, all GetAllowsAnonChildAccessibles() methods renamed to CanHaveAnonChildren()

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

r=me

Comment 8

[alexander :surkov (:asurkov)]

inbound land https://hg.mozilla.org/integration/mozilla-inbound/rev/417bf64cf918

Comment 9

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/417bf64cf918

Congratulations on your first patch in the tree! Hope to see you on IRC (http://irc.mozilla.org/) in #developers soon. If you'd like to fix another bug (we'd love it if you did!) but need some inspiration, pop on & say hi - and we'll find something for you :-)

Comment 10

[alexander :surkov (:asurkov)]

(In reply to Ed Morley [:edmorley] from comment #9)
> https://hg.mozilla.org/mozilla-central/rev/417bf64cf918
> 
> Congratulations on your first patch in the tree! Hope to see you on IRC
> (http://irc.mozilla.org/) in #developers soon. If you'd like to fix another
> bug (we'd love it if you did!) but need some inspiration, pop on & say hi -
> and we'll find something for you :-)

and #accessibility channel too :)

Thanks, Joseph, for working on this!