include/vcl/accessibility/AccessibleBrowseBoxBase.hxx | 3 - vcl/source/accessibility/AccessibleBrowseBoxBase.cxx | 35 ------------------ 2 files changed, 38 deletions(-)
New commits: commit 78fe1ba99e2c9764afefc5b632ddfd037935c0e6 Author: Michael Weghorn <m.wegh...@posteo.de> AuthorDate: Tue Apr 29 19:11:40 2025 +0200 Commit: Michael Weghorn <m.wegh...@posteo.de> CommitDate: Tue Apr 29 23:12:43 2025 +0200 vcl a11y: Drop AccessibleBrowseBoxBase::getAccessibleIndexInParent The base class implementation in OAccessibleComponentHelper::getAccessibleIndexInParent already implements the same logic. Change-Id: I3b9e4b2aa04bb24bb0355897ea18eb41cad9ef61 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/184794 Tested-by: Jenkins Reviewed-by: Michael Weghorn <m.wegh...@posteo.de> diff --git a/include/vcl/accessibility/AccessibleBrowseBoxBase.hxx b/include/vcl/accessibility/AccessibleBrowseBoxBase.hxx index 54b56ac50ba7..feccc6c80a6a 100644 --- a/include/vcl/accessibility/AccessibleBrowseBoxBase.hxx +++ b/include/vcl/accessibility/AccessibleBrowseBoxBase.hxx @@ -103,9 +103,6 @@ public: /** @return A reference to the parent accessible object. */ virtual css::uno::Reference<css::accessibility::XAccessible > SAL_CALL getAccessibleParent() override; - /** @return The index of this object among the parent's children. */ - virtual sal_Int64 SAL_CALL getAccessibleIndexInParent() override; - /** @return The description of this object. */ diff --git a/vcl/source/accessibility/AccessibleBrowseBoxBase.cxx b/vcl/source/accessibility/AccessibleBrowseBoxBase.cxx index b87f6844d73c..58cbd24104a9 100644 --- a/vcl/source/accessibility/AccessibleBrowseBoxBase.cxx +++ b/vcl/source/accessibility/AccessibleBrowseBoxBase.cxx @@ -114,41 +114,6 @@ Reference< css::accessibility::XAccessible > SAL_CALL AccessibleBrowseBoxBase::g return mxParent; } -sal_Int64 SAL_CALL AccessibleBrowseBoxBase::getAccessibleIndexInParent() -{ - ::osl::MutexGuard aGuard( getMutex() ); - ensureIsAlive(); - - // -1 for child not found/no parent (according to specification) - sal_Int64 nRet = -1; - - css::uno::Reference< uno::XInterface > xMeMyselfAndI( static_cast< css::accessibility::XAccessibleContext* >( this ), uno::UNO_QUERY ); - - // iterate over parent's children and search for this object - if( mxParent.is() ) - { - css::uno::Reference< css::accessibility::XAccessibleContext > - xParentContext( mxParent->getAccessibleContext() ); - if( xParentContext.is() ) - { - css::uno::Reference< uno::XInterface > xChild; - - sal_Int64 nChildCount = xParentContext->getAccessibleChildCount(); - for( sal_Int64 nChild = 0; nChild < nChildCount; ++nChild ) - { - xChild.set(xParentContext->getAccessibleChild( nChild ), css::uno::UNO_QUERY); - - if ( xMeMyselfAndI.get() == xChild.get() ) - { - nRet = nChild; - break; - } - } - } - } - return nRet; -} - OUString SAL_CALL AccessibleBrowseBoxBase::getAccessibleDescription() { ::osl::MutexGuard aGuard( getMutex() );