[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - editeng/source include/editeng svx/source

[Michael Stahl]

 editeng/source/accessibility/AccessibleContextBase.cxx |   15 ---------------
 include/editeng/AccessibleContextBase.hxx              |    6 ------
 svx/source/accessibility/AccessibleShape.cxx           |    6 +++++-
 3 files changed, 5 insertions(+), 22 deletions(-)

New commits:
commit 0dd9e51abc5705b7ae253d3f9b284136cf1597d6
Author: Michael Stahl <mstahl at redhat.com>
Date:   Thu Dec 15 22:46:08 2016 +0100

    tdf#104488 editeng,svx: throw less Accessible exceptions
    
    For some of these functions it doesn't make sense to throw
    DisposedException as they can return a sensible default value.
    
    Particularly AccessibleShape::getAccessibleChildCount() was throwing an
    exception that was never caught.
    
    (cherry picked from commit 5a3b2111172f106378cbf78aba0fde1db3642275)
    
    loplugin:unnecessaryoverride
    (cherry picked from commit 10bf49c18d56f198e63514953a1c6e2ea17fcc9c)
    
    Change-Id: I000149cf9bb0fd13f69650ad8224a8daf26f7bee
    Reviewed-on: https://gerrit.libreoffice.org/32187
    Reviewed-by: Michael Stahl <mstahl at redhat.com>
    Tested-by: Michael Stahl <mstahl at redhat.com>

diff --git a/editeng/source/accessibility/AccessibleContextBase.cxx b/editeng/source/accessibility/AccessibleContextBase.cxx
index f93e443..32912aa 100644
--- a/editeng/source/accessibility/AccessibleContextBase.cxx
+++ b/editeng/source/accessibility/AccessibleContextBase.cxx
@@ -175,7 +175,6 @@ uno::Reference< XAccessibleContext> SAL_CALL
     AccessibleContextBase::getAccessibleContext()
     throw (uno::RuntimeException, std::exception)
 {
-    ThrowIfDisposed ();
     return this;
 }
 
@@ -188,7 +187,6 @@ sal_Int32 SAL_CALL
        AccessibleContextBase::getAccessibleChildCount()
     throw (uno::RuntimeException, std::exception)
 {
-    ThrowIfDisposed ();
     return 0;
 }
 
@@ -420,7 +418,6 @@ void SAL_CALL AccessibleContextBase::removeAccessibleEventListener (
 OUString SAL_CALL AccessibleContextBase::getImplementationName()
     throw (css::uno::RuntimeException, std::exception)
 {
-    ThrowIfDisposed ();
     return OUString("AccessibleContextBase");
 }
 
@@ -434,7 +431,6 @@ uno::Sequence< OUString > SAL_CALL
        AccessibleContextBase::getSupportedServiceNames()
     throw (css::uno::RuntimeException, std::exception)
 {
-    ThrowIfDisposed ();
     return {
         "com.sun.star.accessibility.Accessible",
         "com.sun.star.accessibility.AccessibleContext"};
@@ -443,17 +439,6 @@ uno::Sequence< OUString > SAL_CALL
 
 // XTypeProvider
 
-uno::Sequence< css::uno::Type>
-    AccessibleContextBase::getTypes()
-    throw (css::uno::RuntimeException, std::exception)
-{
-    ThrowIfDisposed ();
-
-    // This class supports no interfaces on its own.  Just return those
-    // supported by the base class.
-    return WeakComponentImplHelper::getTypes();
-}
-
 uno::Sequence<sal_Int8> SAL_CALL
     AccessibleContextBase::getImplementationId()
     throw (css::uno::RuntimeException, std::exception)
diff --git a/include/editeng/AccessibleContextBase.hxx b/include/editeng/AccessibleContextBase.hxx
index dc0c19a..24bb87e 100644
--- a/include/editeng/AccessibleContextBase.hxx
+++ b/include/editeng/AccessibleContextBase.hxx
@@ -266,12 +266,6 @@ public:
 
     //=====  XTypeProvider  ===================================================
 
-    /** Returns a sequence of all supported interfaces.
-    */
-    virtual css::uno::Sequence< css::uno::Type> SAL_CALL
-        getTypes()
-        throw (css::uno::RuntimeException, std::exception) override;
-
     /** Returns a implementation id.
     */
     virtual css::uno::Sequence<sal_Int8> SAL_CALL
diff --git a/svx/source/accessibility/AccessibleShape.cxx b/svx/source/accessibility/AccessibleShape.cxx
index bcbebd3..3821216 100644
--- a/svx/source/accessibility/AccessibleShape.cxx
+++ b/svx/source/accessibility/AccessibleShape.cxx
@@ -326,7 +326,11 @@ sal_Int32 SAL_CALL
        AccessibleShape::getAccessibleChildCount ()
     throw (css::uno::RuntimeException, std::exception)
 {
-    ThrowIfDisposed ();
+    if (IsDisposed())
+    {
+        return 0;
+    }
+
     sal_Int32 nChildCount = 0;
 
     // Add the number of shapes that are children of this shape.