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?
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/ nsAccessibility Service. cpp TY_ATK abled()
@@ +1845,5 @@
> }
> +
> +#ifdef MOZ_ACCESSIBILI
> +bool
> +ShouldA11yBeEn
Would probably be better to move this into our atk layer no?
::: accessible/ src/base/ nsAccessibility Service. h bled();
@@ +58,5 @@
>
> +/**
> + * Is platform accessibility enabled.
> + */
> +bool ShouldA11yBeEna
Despite the name is Linux/dbus specific right?