From 6f31670783b3445a8073bd49fe4c041b655e91b3 Mon Sep 17 00:00:00 2001 From: Caolán McNamara Date: Thu, 23 Nov 2017 14:05:26 +0000 Subject: set new document window as parent for dialogs during load MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit and exit typedetection early and completely if application quits while detecting During typedetection, before loading proper, we have the hidden window as parent so warnings are not modal to existing windows and they are cancelled on exit. Once we do have a window, then reinit interaction handler to have that window as the parent for any further dialogs. Change-Id: I5c6711557266bf7d1eb9291f1c454cbfaf766886 Reviewed-on: https://gerrit.libreoffice.org/45148 Tested-by: Jenkins Reviewed-by: Caolán McNamara Tested-by: Caolán McNamara --- filter/CppunitTest_filter_priority.mk | 1 + filter/source/config/cache/typedetection.cxx | 12 +++++--- filter/source/config/cache/typedetection.hxx | 41 +++++++++++++++++++++++++++- 3 files changed, 49 insertions(+), 5 deletions(-) (limited to 'filter') diff --git a/filter/CppunitTest_filter_priority.mk b/filter/CppunitTest_filter_priority.mk index e9bc7de8f196..5f9042dd0859 100644 --- a/filter/CppunitTest_filter_priority.mk +++ b/filter/CppunitTest_filter_priority.mk @@ -11,6 +11,7 @@ $(eval $(call gb_CppunitTest_CppunitTest,filter_priority)) $(eval $(call gb_CppunitTest_use_sdk_api,filter_priority)) $(eval $(call gb_CppunitTest_use_ure,filter_priority)) +$(eval $(call gb_CppunitTest_use_vcl,filter_priority)) $(eval $(call gb_CppunitTest_use_configuration,filter_priority)) diff --git a/filter/source/config/cache/typedetection.cxx b/filter/source/config/cache/typedetection.cxx index 3355618042c0..7b867177412c 100644 --- a/filter/source/config/cache/typedetection.cxx +++ b/filter/source/config/cache/typedetection.cxx @@ -21,6 +21,7 @@ #include "constant.hxx" #include +#include #include #include @@ -50,7 +51,10 @@ namespace filter{ TypeDetection::TypeDetection(const css::uno::Reference< css::uno::XComponentContext >& rxContext) : m_xContext(rxContext) + , m_xTerminateListener(new TerminateDetection(this)) + , m_bCancel(false) { + css::frame::Desktop::create(m_xContext)->addTerminateListener(m_xTerminateListener.get()); BaseContainer::init(rxContext , TypeDetection::impl_getImplementationName() , TypeDetection::impl_getSupportedServiceNames(), @@ -60,6 +64,7 @@ TypeDetection::TypeDetection(const css::uno::Reference< css::uno::XComponentCont TypeDetection::~TypeDetection() { + css::frame::Desktop::create(m_xContext)->removeTerminateListener(m_xTerminateListener.get()); } @@ -425,18 +430,17 @@ OUString SAL_CALL TypeDetection::queryTypeByDescriptor(css::uno::Sequence< css:: if (lFlatTypes.size()>0) sType = impl_detectTypeFlatAndDeep(stlDescriptor, lFlatTypes, bAllowDeep, lUsedDetectors, sLastChance); - // flat detection failed // pure deep detection failed // => ask might existing InteractionHandler // means: ask user for its decision - if (sType.isEmpty()) + if (sType.isEmpty() && !m_bCancel) sType = impl_askUserForTypeAndFilterIfAllowed(stlDescriptor); // no real detected type - but a might valid one. // update descriptor and set last chance for return. - if (sType.isEmpty() && !sLastChance.isEmpty()) + if (sType.isEmpty() && !sLastChance.isEmpty() && !m_bCancel) { OSL_FAIL("set first flat detected type without a registered deep detection service as \"last chance\" ... nevertheless some other deep detections said \"NO\". I TRY IT!"); sType = sLastChance; @@ -896,7 +900,7 @@ OUString TypeDetection::impl_detectTypeFlatAndDeep( utl::MediaDescriptor& r // obtained from the cache => ignore it, and continue with search for (FlatDetection::const_iterator pFlatIt = lFlatTypes.begin(); - pFlatIt != lFlatTypes.end() ; + pFlatIt != lFlatTypes.end() && !m_bCancel; ++pFlatIt ) { const FlatDetectionInfo& aFlatTypeInfo = *pFlatIt; diff --git a/filter/source/config/cache/typedetection.hxx b/filter/source/config/cache/typedetection.hxx index 18185134af5a..830badc54f63 100644 --- a/filter/source/config/cache/typedetection.hxx +++ b/filter/source/config/cache/typedetection.hxx @@ -21,14 +21,16 @@ #include "basecontainer.hxx" #include +#include #include #include #include +#include #include - namespace filter{ namespace config { +class TerminateDetection; /** @short implements the service TypeDetection. */ @@ -39,6 +41,8 @@ class TypeDetection : public ::cppu::ImplInheritanceHelper< BaseContainer // native interface css::uno::Reference< css::uno::XComponentContext > m_xContext; + rtl::Reference m_xTerminateListener; + bool m_bCancel; public: @@ -53,6 +57,11 @@ public: */ explicit TypeDetection(const css::uno::Reference< css::uno::XComponentContext >& rxContext); + void cancel() + { + m_bCancel = true; + } + /** @short standard dtor. */ @@ -362,6 +371,36 @@ public: static css::uno::Reference< css::uno::XInterface > impl_createInstance(const css::uno::Reference< css::lang::XMultiServiceFactory >& xSMGR); }; +class TerminateDetection : public cppu::WeakComponentImplHelper +{ +private: + osl::Mutex m_aLock; + TypeDetection* m_pTypeDetection; + +public: + + using cppu::WeakComponentImplHelperBase::disposing; + virtual void SAL_CALL disposing(const css::lang::EventObject&) override + { + } + + // XTerminateListener + virtual void SAL_CALL queryTermination(const css::lang::EventObject&) override + { + m_pTypeDetection->cancel(); + } + + virtual void SAL_CALL notifyTermination(const css::lang::EventObject&) override + { + } + + TerminateDetection(TypeDetection* pTypeDetection) + : cppu::WeakComponentImplHelper(m_aLock) + , m_pTypeDetection(pTypeDetection) + { + } +}; + }} #endif // INCLUDED_FILTER_SOURCE_CONFIG_CACHE_TYPEDETECTION_HXX -- cgit