framework/inc/services/desktop.hxx | 12 ---
framework/source/services/desktop.cxx | 23 -------
svx/source/form/fmscriptingenv.cxx | 106 ----------------------------------
3 files changed, 141 deletions(-)
New commits:
commit a97e344ba06b92e5f1fc96062a5ad03fb732615e
Author: Jan-Marek Glogowski <jan-marek.glogow...@extern.cib.de>
AuthorDate: Mon Feb 17 00:12:12 2020 +0100
Commit: Jan-Marek Glogowski <glo...@fbihome.de>
CommitDate: Sun May 31 22:21:35 2020 +0200
Revert "tdf#88985 block app from exiting during macro execution"
This isn't needed anymore with the fixes for tdf#127205.
Reverts commit 68cf256f506d4601a2d2cf3ec2d56713afd491e6.
Change-Id: I1a7944632af8330f68d451d2c293143233039b34
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88837
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glo...@fbihome.de>
diff --git a/framework/inc/services/desktop.hxx
b/framework/inc/services/desktop.hxx
index 1f0f7911e98f..0cf9cb355fe4 100644
--- a/framework/inc/services/desktop.hxx
+++ b/framework/inc/services/desktop.hxx
@@ -418,18 +418,6 @@ class Desktop final : private cppu::BaseMutex,
*/
css::uno::Reference< css::frame::XTerminateListener > m_xQuickLauncher;
- /** special terminate listener active when a macro is executing.
- * Because basic runs Application::Yield internally the application
may quit
- * while running inside the internal basic event loop. So all the
basic
- * infrastructure may be deleted while the call is executing, leading
to
- * a variant of crashes. So this special terminate listener will
- * veto the current quit attempt, stop basic execution, which will
- * cause the inner event loop to quit, and on return to the outer
normal
- * application event loop then resend the quit attempt.
- * So these implementation must be a special terminate listener too
.-(
- */
- css::uno::Reference< css::frame::XTerminateListener >
m_xStarBasicQuitGuard;
-
/** special terminate listener which loads images asynchronous for
current open documents.
* Because internally it uses blocking system APIs... it can't be
guaranteed that
* running jobs can be cancelled successfully if the corresponding
document will be closed...
diff --git a/framework/source/services/desktop.cxx
b/framework/source/services/desktop.cxx
index 260ddcb5dd43..28f5015203e3 100644
--- a/framework/source/services/desktop.cxx
+++ b/framework/source/services/desktop.cxx
@@ -161,7 +161,6 @@ Desktop::Desktop( const css::uno::Reference<
css::uno::XComponentContext >& xCon
, m_xDispatchRecorderSupplier(
)
, m_xPipeTerminator (
)
, m_xQuickLauncher (
)
- , m_xStarBasicQuitGuard (
)
, m_xSWThreadManager (
)
, m_xSfxTerminator (
)
, m_xTitleNumberGenerator (
)
@@ -206,7 +205,6 @@ sal_Bool SAL_CALL Desktop::terminate()
css::uno::Reference< css::frame::XTerminateListener > xPipeTerminator =
m_xPipeTerminator;
css::uno::Reference< css::frame::XTerminateListener > xQuickLauncher =
m_xQuickLauncher;
- css::uno::Reference< css::frame::XTerminateListener > xStarBasicQuitGuard
= m_xStarBasicQuitGuard;
css::uno::Reference< css::frame::XTerminateListener > xSWThreadManager =
m_xSWThreadManager;
css::uno::Reference< css::frame::XTerminateListener > xSfxTerminator =
m_xSfxTerminator;
@@ -257,12 +255,6 @@ sal_Bool SAL_CALL Desktop::terminate()
lCalledTerminationListener.push_back( xQuickLauncher );
}
- if ( xStarBasicQuitGuard.is() )
- {
- xStarBasicQuitGuard->queryTermination( aEvent );
- lCalledTerminationListener.push_back( xStarBasicQuitGuard );
- }
-
if ( xSWThreadManager.is() )
{
xSWThreadManager->queryTermination( aEvent );
@@ -308,9 +300,6 @@ sal_Bool SAL_CALL Desktop::terminate()
if( bAskQuickStart && xQuickLauncher.is() )
xQuickLauncher->notifyTermination( aEvent );
- if ( xStarBasicQuitGuard.is() )
- xStarBasicQuitGuard->notifyTermination( aEvent );
-
if ( xSWThreadManager.is() )
xSWThreadManager->notifyTermination( aEvent );
@@ -416,11 +405,6 @@ void SAL_CALL Desktop::addTerminateListener( const
css::uno::Reference< css::fra
m_xQuickLauncher = xListener;
return;
}
- if( sImplementationName == "com.sun.star.comp.svx.StarBasicQuitGuard" )
- {
- m_xStarBasicQuitGuard = xListener;
- return;
- }
if( sImplementationName == "com.sun.star.util.comp.FinalThreadManager"
)
{
m_xSWThreadManager = xListener;
@@ -466,12 +450,6 @@ void SAL_CALL Desktop::removeTerminateListener( const
css::uno::Reference< css::
return;
}
- if( sImplementationName == "com.sun.star.comp.svx.StarBasicQuitGuard" )
- {
- m_xStarBasicQuitGuard.clear();
- return;
- }
-
if( sImplementationName == "com.sun.star.util.comp.FinalThreadManager"
)
{
m_xSWThreadManager.clear();
@@ -1105,7 +1083,6 @@ void SAL_CALL Desktop::disposing()
m_xPipeTerminator.clear();
m_xQuickLauncher.clear();
- m_xStarBasicQuitGuard.clear();
m_xSWThreadManager.clear();
// we need a copy because the disposing might call the removeEventListener
method
diff --git a/svx/source/form/fmscriptingenv.cxx
b/svx/source/form/fmscriptingenv.cxx
index 14374f12e874..6edc70460d56 100644
--- a/svx/source/form/fmscriptingenv.cxx
+++ b/svx/source/form/fmscriptingenv.cxx
@@ -737,109 +737,6 @@ namespace svxform
m_pScriptExecutor = nullptr;
}
- namespace {
-
- // tdf#88985 If LibreOffice tries to exit during the execution of a macro
- // then: detect the effort, stop basic execution, block until the macro
- // returns due to that stop, then restart the quit. This avoids the app
- // exiting and destroying itself until the macro is parked at a safe place
- // to do that.
- class QuitGuard
- {
- private:
-
- class TerminateListener : public
cppu::WeakComponentImplHelper<css::frame::XTerminateListener,
-
css::lang::XServiceInfo>
- {
- private:
- css::uno::Reference<css::frame::XDesktop2> m_xDesktop;
- osl::Mutex maMutex;
- bool mbQuitBlocked;
- public:
- // XTerminateListener
- virtual void SAL_CALL queryTermination(const
css::lang::EventObject& /*rEvent*/) override
- {
- mbQuitBlocked = true;
-#if HAVE_FEATURE_SCRIPTING
- StarBASIC::Stop();
-#endif
- throw css::frame::TerminationVetoException();
- }
-
- virtual void SAL_CALL notifyTermination(const
css::lang::EventObject& /*rEvent*/) override
- {
- mbQuitBlocked = false;
- }
-
- using cppu::WeakComponentImplHelperBase::disposing;
-
- virtual void SAL_CALL disposing(const css::lang::EventObject&
rEvent) override
- {
- const bool bShutDown = (rEvent.Source == m_xDesktop);
- if (bShutDown && m_xDesktop.is())
- {
- m_xDesktop->removeTerminateListener(this);
- m_xDesktop.clear();
- }
- }
-
- // XServiceInfo
- virtual OUString SAL_CALL getImplementationName() override
- {
- return "com.sun.star.comp.svx.StarBasicQuitGuard";
- }
-
- virtual sal_Bool SAL_CALL supportsService(OUString const &
ServiceName) override
- {
- return cppu::supportsService(this, ServiceName);
- }
-
- virtual css::uno::Sequence<OUString> SAL_CALL
getSupportedServiceNames() override
- {
- return { "com.sun.star.svx.StarBasicQuitGuard" };
- }
-
- public:
- TerminateListener()
- : cppu::WeakComponentImplHelper<css::frame::XTerminateListener,
-
css::lang::XServiceInfo>(maMutex)
- , mbQuitBlocked(false)
- {
- }
-
- void start()
- {
- css::uno::Reference<css::uno::XComponentContext>
xContext(comphelper::getProcessComponentContext());
- m_xDesktop = css::frame::Desktop::create(xContext);
- m_xDesktop->addTerminateListener(this);
- }
-
- void stop()
- {
- if (!m_xDesktop.is())
- return;
- m_xDesktop->removeTerminateListener(this);
- if (mbQuitBlocked)
- m_xDesktop->terminate();
- }
- };
-
- rtl::Reference<TerminateListener> mxListener;
- public:
- QuitGuard()
- : mxListener(new TerminateListener)
- {
- mxListener->start();
- }
-
- ~QuitGuard()
- {
- mxListener->stop();
- }
- };
-
- }
-
IMPL_LINK( FormScriptListener, OnAsyncScriptEvent, void*, p, void )
{
ScriptEvent* _pEvent = static_cast<ScriptEvent*>(p);
@@ -851,10 +748,7 @@ namespace svxform
::osl::ClearableMutexGuard aGuard( m_aMutex );
if ( !impl_isDisposed_nothrow() )
- {
- QuitGuard aQuitGuard;
impl_doFireScriptEvent_nothrow( aGuard, *_pEvent, nullptr );
- }
}
delete _pEvent;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
