[Libreoffice-commits] core.git: vcl/source

[Noel Grandin (via logerrit)]

 vcl/source/uitest/uno/uitest_uno.cxx |    5 +++++
 1 file changed, 5 insertions(+)

New commits:
commit 08e1cdef0ddfd3c88483b084d8e1a28d01980364
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Fri May 20 09:47:26 2022 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Fri May 20 17:18:28 2022 +0200

    revert some questionable changes "new loplugin:unnecessary locking"
    
    in commit f2c02331a7dc0a924bbf30cbc279e92621e89590
    
    which, as llunak points out, are questionable
    
    Change-Id: Ifb29429631c12c4f1c0b4da044a4d599592d3642
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/134660
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/vcl/source/uitest/uno/uitest_uno.cxx 
b/vcl/source/uitest/uno/uitest_uno.cxx
index 48fa2ef67967..4e5a6e0e0391 100644
--- a/vcl/source/uitest/uno/uitest_uno.cxx
+++ b/vcl/source/uitest/uno/uitest_uno.cxx
@@ -66,28 +66,33 @@ UITestUnoObj::UITestUnoObj():
 
 sal_Bool SAL_CALL UITestUnoObj::executeCommand(const OUString& rCommand)
 {
+    SolarMutexGuard aGuard;
     return UITest::executeCommand(rCommand);
 }
 
 sal_Bool SAL_CALL UITestUnoObj::executeCommandWithParameters(const OUString& 
rCommand,
     const css::uno::Sequence< css::beans::PropertyValue >& rArgs)
 {
+    SolarMutexGuard aGuard;
     return UITest::executeCommandWithParameters(rCommand,rArgs);
 }
 
 sal_Bool SAL_CALL UITestUnoObj::executeDialog(const OUString& rCommand)
 {
+    SolarMutexGuard aGuard;
     return UITest::executeDialog(rCommand);
 }
 
 css::uno::Reference<css::ui::test::XUIObject> SAL_CALL 
UITestUnoObj::getTopFocusWindow()
 {
+    SolarMutexGuard aGuard;
     std::unique_ptr<UIObject> pObj = UITest::getFocusTopWindow();
     return new UIObjectUnoObj(std::move(pObj));
 }
 
 css::uno::Reference<css::ui::test::XUIObject> SAL_CALL 
UITestUnoObj::getFloatWindow()
 {
+    SolarMutexGuard aGuard;
     std::unique_ptr<UIObject> pObj = UITest::getFloatWindow();
     return new UIObjectUnoObj(std::move(pObj));
 }