762755 - de-ns-ify nsXULMenuAccessible and nsXULListboxAccessible classes

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Comment 1

[Mark Capella [:capella]]

Attachment: Patch (v1)

Woo hoo! A two-fer-one !

Comment 2

[alexander :surkov (:asurkov)]

Comment on attachment 631546 [details] [diff] [review]
Patch (v1)

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

::: accessible/src/atk/nsXULListboxAccessibleWrap.h
@@ +11,5 @@
>  
> +namespace mozilla {
> +namespace a11y {
> +
> +typedef class XULListBoxAccessible XULListBoxAccessibleWrap;

ok, if you like though I didn't think to uppercase b in box (however you keep other classes untouched like XULListitemAccessible

::: accessible/src/xul/nsXULListboxAccessible.cpp
@@ +893,5 @@
>    return NativelyUnavailable() || (mParent && mParent->NativelyUnavailable()) ?
>      states::UNAVAILABLE : states::FOCUSABLE | states::SELECTABLE;
>  }
>  
> +NS_IMETHODIMP XULListitemAccessible::GetActionName(PRUint8 aIndex, nsAString& aName)

keep NS_IMETHODIMP on separate line

::: accessible/src/xul/nsXULListboxAccessible.h
@@ +143,1 @@
>                                  public nsIAccessibleTableCell

wrong indent

::: accessible/src/xul/nsXULMenuAccessible.cpp
@@ +289,5 @@
>    // That indicates we don't walk anonymous children for menuitems
>    return false;
>  }
>  
> +NS_IMETHODIMP XULMenuitemAccessible::DoAction(PRUint8 index)

return type on own line

@@ +300,5 @@
>    return NS_ERROR_INVALID_ARG;
>  }
>  
>  /** select us! close combo box if necessary*/
> +NS_IMETHODIMP XULMenuitemAccessible::GetActionName(PRUint8 aIndex, nsAString& aName)

same

@@ +396,5 @@
>  {
>    return roles::SEPARATOR;
>  }
>  
> +NS_IMETHODIMP XULMenuSeparatorAccessible::DoAction(PRUint8 index)

same and below

Comment 3

[Mark Capella [:capella]]

Attachment: Patch (v2)

fyi, final for push.

Comment 4

[Mark Capella [:capella]]

fyi on WIN, hg rename FOO foo generates a renaming thing to itself kind of error...

hg rename FOO xxx, followed by
hg rename xxx foo achieves the goal

Comment 5

[Mark Capella [:capella]]

push to try:
https://tbpl.mozilla.org/?tree=Try&rev=6b0a0f95c64d

Comment 6

[Mark Capella [:capella]]

https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=8ca3fe220709

Comment 7

[Ryan VanderMeulen (PTO, back 6-April)]

https://hg.mozilla.org/mozilla-central/rev/8ca3fe220709