offapi/com/sun/star/accessibility/XAccessibleAction.idl |   10 ++++------
 test/source/a11y/accessibletestbase.cxx                 |    2 +-
 2 files changed, 5 insertions(+), 7 deletions(-)

New commits:
commit 5495a2ef9c5e0a457c44e2fe0f6a62effda43ebc
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Tue Jul 16 11:08:13 2024 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Tue Jul 16 12:47:45 2024 +0200

    a11y tests: Double the timeout for UI tests
    
    Increase the timeout for a11y UI tests from 1 to 2 min,
    to see whether this helps with one slow Windows Jenkins
    builder running into the timeout every once in a while,
    as was mentioned in ESC calls.
    
    Example from [1]:
    
        timeout waiting for dialog 'Bookmark' to show up
        warn:vcl.schedule:6600:11456:vcl/source/app/scheduler.cxx:524: Uncaught 
exception during Task::Invoke()!
        abort() has been 
calledwarn:sal.cppunittester:6600:11456:sal/cppunittester/cppunittester.cxx:601:
 CAUGHT SIGABRT:
        [...]
    
    [1] https://ci.libreoffice.org/job/gerrit_windows/181051/console
    
    Change-Id: I5b16ea8c538f6e2b5a30ebac714e177e696aac92
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/170569
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>
    Tested-by: Jenkins

diff --git a/test/source/a11y/accessibletestbase.cxx 
b/test/source/a11y/accessibletestbase.cxx
index 27bea36f6084..5d8f50c90f10 100644
--- a/test/source/a11y/accessibletestbase.cxx
+++ b/test/source/a11y/accessibletestbase.cxx
@@ -436,7 +436,7 @@ test::AccessibleTestBase::awaitDialog(const 
std::u16string_view name,
             mxToolkit->addTopWindowListener(mxTopWindowListener);
 
             maTimeoutTimer.SetInvokeHandler(LINK(this, ListenerHelper, 
timeoutTimerHandler));
-            maTimeoutTimer.SetTimeout(60000);
+            maTimeoutTimer.SetTimeout(120000);
             maTimeoutTimer.Start();
 
             maIdleHandler.SetInvokeHandler(LINK(this, ListenerHelper, 
idleHandler));
commit 4f63dd8b374ce6e5c6fd783625c287eef6079265
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Tue Jul 16 10:51:59 2024 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Tue Jul 16 12:47:34 2024 +0200

    a11y: Clean up XAccessibleAction doc
    
    The `XAccessibleAction::getAccessibleActionDescription`
    and `XAcccessibleAction::doAccessibleAction` doc mentions
    that an `IndexOutOfBoundsException` exception is thrown
    when the index is invalid, which is indeed the expected
    behaviour. However, they mentioned in addition that false
    or an empty string is returned, which doesn't make sense
    when exception gets thrown, so drop that.
    
    Also slightly improve wording at 2 other places.
    
    Change-Id: Idcc1fd669f32ff5a160cb35d2fbf6c2d23f65ab9
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/170567
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>
    Tested-by: Jenkins

diff --git a/offapi/com/sun/star/accessibility/XAccessibleAction.idl 
b/offapi/com/sun/star/accessibility/XAccessibleAction.idl
index bae281c8e98f..e35d049e1b44 100644
--- a/offapi/com/sun/star/accessibility/XAccessibleAction.idl
+++ b/offapi/com/sun/star/accessibility/XAccessibleAction.idl
@@ -47,8 +47,7 @@ interface XAccessibleAction : 
::com::sun::star::uno::XInterface
     /** Perform the specified Action on the object.
 
         @param nIndex
-            This index specifies the action to perform.  If it lies outside
-            the valid range `FALSE` is returned and no action is performed.
+            This index specifies the action to perform.
         @return
             Returns `TRUE` if the action was successfully performed.  If
             the action could not be performed successfully `FALSE` is
@@ -66,14 +65,13 @@ interface XAccessibleAction : 
::com::sun::star::uno::XInterface
 
         @param nIndex
             This index specifies the action of which to return a
-            description.  If it lies outside the valid range an empty string
-            is returned.
+            description.
 
         @return
             The returned value is a localized string of the specified action.
 
         @throws IndexOutOfBoundsException
-            If the given index lies not in the valid range then an
+            If the given index does not lie in the valid range then a
             ::com::sun::star::lang::IndexOutOfBoundsException
             exception is thrown.
     */
@@ -86,7 +84,7 @@ interface XAccessibleAction : 
::com::sun::star::uno::XInterface
         more information about how key bindings are represented.
 
         @param nIndex
-            This index specifies the action of which to return the key
+            This index specifies the action for which to return the key
             binding.
 
         @return

Reply via email to