[Libreoffice-commits] core.git: Branch 'feature/cib_contract891' - framework/source
Miklos Vajna (via logerrit)
logerrit at kemper.freedesktop.org
Fri Oct 11 09:20:37 UTC 2019
Rebased ref, commits from common ancestor:
commit b4158aedf18394c695a14708bc8ff14c601b1d3e
Author: Miklos Vajna <vmiklos at collabora.com>
AuthorDate: Wed Dec 19 17:47:57 2018 +0100
Commit: Thorsten Behrens <Thorsten.Behrens at CIB.de>
CommitDate: Fri Oct 11 11:19:13 2019 +0200
framework: allow loading a component on the main thread
The user-visible problem was that embedded (OLE) objects contained in a
document that was loaded on a thread were not editable.
This works in the loaded-with-UI case because the Windows version of the
SalData constructor in vcl calls CoInitialize() (which sets the
concurrency model of the main thread to STA) and then later the
OleComponent constructor in embeddedobj calls OleInitialize(), which
just realizes that the concurrency model is already set, and OLE editing
works.
However, if the document is loaded on a thread, things are different.
The concurrency model of the thread is set to MTA in
oslWorkerWrapperFunction() in sal, so the later OleInitialize() will
fail with RPC_E_CHANGED_MODE, as it's not possible to set the
concurrency model of a thread once it's set.
Solve the problem by providing in opt-in way to execute the actual
import on the main thread, since remote UNO clients always invoke
Desktop::loadComponentFromURL() on a thread.
Reviewed-on: https://gerrit.libreoffice.org/65453
Reviewed-by: Miklos Vajna <vmiklos at collabora.com>
Tested-by: Jenkins
(cherry picked from commit 2dc3a6c273cb82506842864481d78df7294debbf)
Conflicts:
framework/source/services/frame.cxx
Change-Id: I94f2721b599c3ae3e2ebc1c90dea649a69d51ef7
diff --git a/framework/source/services/frame.cxx b/framework/source/services/frame.cxx
index f3209b346544..0da8c99d0607 100644
--- a/framework/source/services/frame.cxx
+++ b/framework/source/services/frame.cxx
@@ -20,6 +20,8 @@
#include <services/frame.hxx>
#include <dispatch/dispatchprovider.hxx>
+#include <boost/bind.hpp>
+
#include <dispatch/interceptionhelper.hxx>
#include <dispatch/closedispatcher.hxx>
#include <dispatch/windowcommanddispatch.hxx>
@@ -82,7 +84,9 @@
#include <comphelper/processfactory.hxx>
#include <unotools/moduleoptions.hxx>
#include <tools/diagnose_ex.h>
+#include <unotools/cmdoptions.hxx>
+#include <vcl/threadex.hxx>
#include <vcl/menu.hxx>
namespace framework{
@@ -320,7 +324,10 @@ css::uno::Reference< css::lang::XComponent > SAL_CALL Frame::loadComponentFromUR
css::uno::Reference< css::uno::XComponentContext > xContext = m_xContext;
aReadLock.unlock();
- return LoadEnv::loadComponentFromURL(xThis, xContext, sURL, sTargetFrameName, nSearchFlags, lArguments);
+ return vcl::solarthread::syncExecute(
+ boost::bind(&LoadEnv::loadComponentFromURL, xThis,
+ xContext, sURL, sTargetFrameName,
+ nSearchFlags, lArguments));
}
/*-****************************************************************************************************//**
More information about the Libreoffice-commits
mailing list