Comment 17 for bug 857153

Revision history for this message
In , Bolterbugz (bolterbugz) wrote :

Comment on attachment 572408
patch

Review of attachment 572408:
-----------------------------------------------------------------

Please don't forget to add mgorse as an author if you've built on his work.

::: accessible/src/base/nsAccessibilityService.cpp
@@ +1845,5 @@
> }
> +
> +#ifdef MOZ_ACCESSIBILITY_ATK
> +bool
> +ShouldA11yBeEnabled()

Would probably be better to move this into our atk layer no?

::: accessible/src/base/nsAccessibilityService.h
@@ +58,5 @@
>
> +/**
> + * Is platform accessibility enabled.
> + */
> +bool ShouldA11yBeEnabled();

Despite the name is Linux/dbus specific right?