use less exception throwing for flow control

Change-Id: I7ad023479229f89918e588eb8dc7431b5830b45d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171813
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 16 insertions, 5 deletions

diff --git a/vbahelper/source/vbahelper/vbadocumentsbase.cxx b/vbahelper/source/vbahelper/vbadocumentsbase.cxx
index 4821fc5c1c09..ec7694c59294 100644
--- a/vbahelper/source/vbahelper/vbadocumentsbase.cxx
+++ b/vbahelper/source/vbahelper/vbadocumentsbase.cxx
@@ -206,12 +206,23 @@ void lclSetupComponent( const uno::Reference< lang::XComponent >& rxComponent, b
     {
     }
 
-    if( !bInteractive ) try
+    if( bInteractive )
+        return;
+    try
     {
-        uno::Reference< frame::XModel > xModel( rxComponent, uno::UNO_QUERY_THROW );
-        uno::Reference< frame::XController > xController( xModel->getCurrentController(), uno::UNO_SET_THROW );
-        uno::Reference< frame::XFrame > xFrame( xController->getFrame(), uno::UNO_SET_THROW );
-        uno::Reference< awt::XWindow >( xFrame->getContainerWindow(), uno::UNO_SET_THROW )->setEnable( false );
+        uno::Reference< frame::XModel > xModel( rxComponent, uno::UNO_QUERY );
+        if (!xModel)
+            return;
+        uno::Reference< frame::XController > xController( xModel->getCurrentController() );
+        if (!xController)
+            return;
+        uno::Reference< frame::XFrame > xFrame( xController->getFrame() );
+        if (!xFrame)
+            return;
+        uno::Reference< awt::XWindow > xWindow( xFrame->getContainerWindow() );
+        if (!xWindow)
+            return;
+        xWindow->setEnable( false );
     }
     catch( uno::Exception& )
     {