diff options
| author | Noel Grandin <noelgrandin@gmail.com> | 2022-07-03 20:29:28 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2022-07-08 10:10:11 +0200 |
| commit | 8d8e6c84e512c1a8b33aac75965b84481d1a1d13 (patch) | |
| tree | 9ed209d057081d4283eb44b51c0d9af18f976eab /vcl/osx/a11yrolehelper.mm | |
| parent | 911ac42485b690df5cbbff6e3c04b111c1723aca (diff) | |
[API CHANGE] Drop css::accessibility::XAccessibleStateSet
which is internal API, unused (as far as I can tell) by external
users.
This state is purely a bitset
(as implemented by utl::AccessibleStateSetHelper)
so we can just return it as a 64-bit value.
This shaves significant time off the performance profiles
of code that loads very complex shapes, because this state
is frequently used, and we no longer need to allocate a return
value on the heap for every call.
Change-Id: Icf1b3bd367c256646ae9015f9127025f59459c2c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136786
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'vcl/osx/a11yrolehelper.mm')
| -rw-r--r-- | vcl/osx/a11yrolehelper.mm | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/vcl/osx/a11yrolehelper.mm b/vcl/osx/a11yrolehelper.mm index e42b2d53db8b..0ea7075277eb 100644 --- a/vcl/osx/a11yrolehelper.mm +++ b/vcl/osx/a11yrolehelper.mm @@ -156,7 +156,8 @@ using namespace ::com::sun::star::uno; if ( rxAccessible.is() ) { Reference < XAccessibleContext > rxAccessibleContext = rxAccessible -> getAccessibleContext(); if ( rxAccessibleContext.is() && rxAccessibleContext -> getAccessibleRole() == AccessibleRole::TEXT ) { - if ( ! rxAccessibleContext -> getAccessibleStateSet() -> contains ( AccessibleStateType::EDITABLE ) ) { + sal_Int64 nStateSet = rxAccessibleContext -> getAccessibleStateSet(); + if ( !(nStateSet & AccessibleStateType::EDITABLE ) ) { [ nativeRole release ]; nativeRole = NSAccessibilityPopUpButtonRole; } |