desktop/source/lib/init.cxx | 36 +++++++++++++++++++++----
include/LibreOfficeKit/LibreOfficeKit.h | 1
include/LibreOfficeKit/LibreOfficeKit.hxx | 6 ++--
include/vcl/ITiledRenderable.hxx | 12 +++-----
sc/inc/docuno.hxx | 6 ++--
sc/qa/unit/tiledrendering/tiledrendering.cxx | 6 ++--
sc/source/ui/unoobj/docuno.cxx | 34 ++++++-----------------
sd/qa/unit/tiledrendering/tiledrendering.cxx | 6 ++--
sd/source/ui/inc/unomodel.hxx | 3 --
sd/source/ui/unoidl/unomodel.cxx | 36 ++++++-------------------
sw/inc/unotxdoc.hxx | 4 +-
sw/qa/extras/tiledrendering/tiledrendering.cxx | 6 ++--
sw/source/uibase/uno/unotxdoc.cxx | 30 ++++++--------------
vcl/source/window/window.cxx | 8 ++++-
14 files changed, 96 insertions(+), 98 deletions(-)
New commits:
commit 5024d7dbb364dec4fc90979251e5f29bc5ed3f5b
Author: Pranav Kant <pran...@collabora.co.uk>
Date: Wed Feb 14 17:33:16 2018 +0530
lok IME: support dialogs as well
Change-Id: Ic78da45dadaa5a4e1ca78e20d04974108581121e
(cherry picked from commit 44fa8ae7d9bb3a28d860b2cc5871d6a6ccfc8411)
Reviewed-on: https://gerrit.libreoffice.org/49728
Reviewed-by: pranavk <pran...@collabora.co.uk>
Tested-by: pranavk <pran...@collabora.co.uk>
diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index eafdf21f88ce..8c51972b861d 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -569,6 +569,7 @@ static void doc_postKeyEvent(LibreOfficeKitDocument* pThis,
int nCharCode,
int nKeyCode);
static void doc_postExtTextInputEvent(LibreOfficeKitDocument* pThis,
+ unsigned nWindowId,
int nType,
const char* pText);
static void doc_postWindowKeyEvent(LibreOfficeKitDocument* pThis,
@@ -2285,18 +2286,43 @@ static void doc_postKeyEvent(LibreOfficeKitDocument*
pThis, int nType, int nChar
pDoc->postKeyEvent(nType, nCharCode, nKeyCode);
}
-static void doc_postExtTextInputEvent(LibreOfficeKitDocument* pThis, int
nType, const char* pText)
+static void doc_postExtTextInputEvent(LibreOfficeKitDocument* pThis, unsigned
nWindowId, int nType, const char* pText)
{
SolarMutexGuard aGuard;
+ VclPtr<vcl::Window> pWindow;
+ if (nWindowId == 0)
+ {
+ ITiledRenderable* pDoc = getTiledRenderable(pThis);
+ if (!pDoc)
+ {
+ gImpl->maLastExceptionMsg = "Document doesn't support tiled
rendering";
+ return;
+ }
+ pWindow = pDoc->getDocWindow();
+ }
+ else
+ {
+ pWindow = vcl::Window::FindLOKWindow(nWindowId);
+ }
- ITiledRenderable* pDoc = getTiledRenderable(pThis);
- if (!pDoc)
+ if (!pWindow)
{
- gImpl->maLastExceptionMsg = "Document doesn't support tiled rendering";
+ gImpl->maLastExceptionMsg = "No window found for window id: " +
OUString::number(nWindowId);
return;
}
- pDoc->postExtTextInputEvent(nType, OUString::fromUtf8(OString(pText,
strlen(pText))));
+ switch (nType)
+ {
+ case LOK_EXT_TEXTINPUT:
+ pWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_EXTTEXTINPUT,
+ OUString::fromUtf8(OString(pText,
strlen(pText))));
+ break;
+ case LOK_EXT_TEXTINPUT_END:
+ pWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_ENDEXTTEXTINPUT, "");
+ break;
+ default:
+ assert(false && "Unhandled External Text input event!");
+ }
}
static void doc_postWindowKeyEvent(LibreOfficeKitDocument* /*pThis*/, unsigned
nLOKWindowId, int nType, int nCharCode, int nKeyCode)
diff --git a/include/LibreOfficeKit/LibreOfficeKit.h
b/include/LibreOfficeKit/LibreOfficeKit.h
index bb99e04abe17..2af965e4a45d 100644
--- a/include/LibreOfficeKit/LibreOfficeKit.h
+++ b/include/LibreOfficeKit/LibreOfficeKit.h
@@ -291,6 +291,7 @@ struct _LibreOfficeKitDocumentClass
/// @see lok::Document::postExtTextInputEvent
void (*postExtTextInputEvent) (LibreOfficeKitDocument* pThis,
+ unsigned nWindowId,
int nType,
const char* pText);
diff --git a/include/LibreOfficeKit/LibreOfficeKit.hxx
b/include/LibreOfficeKit/LibreOfficeKit.hxx
index c076c0f0b58f..fabd3b29e687 100644
--- a/include/LibreOfficeKit/LibreOfficeKit.hxx
+++ b/include/LibreOfficeKit/LibreOfficeKit.hxx
@@ -539,12 +539,14 @@ public:
/**
* Post the text input from external input window, like IME
*
+ * @param nWindowId Specify the window id to post the input event to. If
+ * nWindow is 0, the event is posted into the document
* @param nType see LibreOfficeKitExtTextInputType
* @param pText Text for LOK_EXT_TEXTINPUT
*/
- void postExtTextInputEvent(int nType, const char* pText)
+ void postExtTextInputEvent(unsigned nWindowId, int nType, const char*
pText)
{
- mpDoc->pClass->postExtTextInputEvent(mpDoc, nType, pText);
+ mpDoc->pClass->postExtTextInputEvent(mpDoc, nWindowId, nType, pText);
}
#endif // defined LOK_USE_UNSTABLE_API || defined LIBO_INTERNAL_ONLY
diff --git a/include/vcl/ITiledRenderable.hxx b/include/vcl/ITiledRenderable.hxx
index 96d021935745..b91a67fdc0a3 100644
--- a/include/vcl/ITiledRenderable.hxx
+++ b/include/vcl/ITiledRenderable.hxx
@@ -85,6 +85,11 @@ public:
}
/**
+ * Get the vcl::Window for the document being edited
+ */
+ virtual VclPtr<vcl::Window> getDocWindow() = 0;
+
+ /**
* Get the hash of the currently displayed part, i.e. sheet in a
spreadsheet
* or slide in a presentation.
*/
@@ -110,13 +115,6 @@ public:
virtual void postKeyEvent(int nType, int nCharCode, int nKeyCode) = 0;
/**
- * Posts an external text input event
- *
- * @see lok::Document::postExtTextInputEvent().
- */
- virtual void postExtTextInputEvent(int nType, const OUString& rText) = 0;
-
- /**
* Posts a mouse event on the document.
*
* @see lok::Document::postMouseEvent().
diff --git a/sc/inc/docuno.hxx b/sc/inc/docuno.hxx
index 4c4e74583023..0b91ec3ddcfc 100644
--- a/sc/inc/docuno.hxx
+++ b/sc/inc/docuno.hxx
@@ -379,15 +379,15 @@ public:
/// @see vcl::ITiledRenderable::getPartHash().
virtual OUString getPartHash( int nPart ) override;
+ /// @see vcl::ITiledRenderable::getDocWindow().
+ virtual VclPtr<vcl::Window> getDocWindow() override;
+
/// @see vcl::ITiledRenderable::initializeForTiledRendering().
virtual void initializeForTiledRendering(const
css::uno::Sequence<css::beans::PropertyValue>& rArguments) override;
/// @see vcl::ITiledRenderable::postKeyEvent().
virtual void postKeyEvent(int nType, int nCharCode, int nKeyCode) override;
- /// @see vcl::ITiledRenderable::postExtTextInputEvent().
- virtual void postExtTextInputEvent(int nType, const OUString& rText)
override;
-
/// @see vcl::ITiledRenderable::postMouseEvent().
virtual void postMouseEvent(int nType, int nX, int nY, int nCount, int
nButtons, int nModifier) override;
diff --git a/sc/qa/unit/tiledrendering/tiledrendering.cxx
b/sc/qa/unit/tiledrendering/tiledrendering.cxx
index 801f7ad55288..74d2d3dc5f0a 100644
--- a/sc/qa/unit/tiledrendering/tiledrendering.cxx
+++ b/sc/qa/unit/tiledrendering/tiledrendering.cxx
@@ -32,6 +32,7 @@
#include <comphelper/propertyvalue.hxx>
#include <sfx2/lokhelper.hxx>
#include <svx/svdpage.hxx>
+#include <vcl/vclevent.hxx>
#include <chrono>
#include <tabvwsh.hxx>
@@ -1566,6 +1567,7 @@ void ScTiledRenderingTest::testIMESupport()
comphelper::LibreOfficeKit::setActive();
ScModelObj* pModelObj = createDoc("empty.ods");
+ VclPtr<vcl::Window> pDocWindow = pModelObj->getDocWindow();
ScDocument* pDoc = pModelObj->GetDocument();
ScTabViewShell* pView =
dynamic_cast<ScTabViewShell*>(SfxViewShell::Current());
@@ -1581,9 +1583,9 @@ void ScTiledRenderingTest::testIMESupport()
});
for (const auto& aInput: aInputs)
{
- pModelObj->postExtTextInputEvent(LOK_EXT_TEXTINPUT, aInput);
+ pDocWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_EXTTEXTINPUT,
aInput);
}
- pModelObj->postExtTextInputEvent(LOK_EXT_TEXTINPUT_END, "");
+ pDocWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_ENDEXTTEXTINPUT, "");
// commit the string to the cell
pModelObj->postKeyEvent(LOK_KEYEVENT_KEYINPUT, 0, awt::Key::RETURN);
diff --git a/sc/source/ui/unoobj/docuno.cxx b/sc/source/ui/unoobj/docuno.cxx
index fc7c6407915c..63a7b757bf9a 100644
--- a/sc/source/ui/unoobj/docuno.cxx
+++ b/sc/source/ui/unoobj/docuno.cxx
@@ -546,6 +546,16 @@ OUString ScModelObj::getPartHash( int nPart )
return (pViewData->GetDocument()->GetHashCode(nPart, nHashCode) ?
OUString::number(nHashCode) : OUString());
}
+VclPtr<vcl::Window> ScModelObj::getDocWindow()
+{
+ SolarMutexGuard aGuard;
+ ScViewData* pViewData = ScDocShell::GetViewData();
+ VclPtr<vcl::Window> pWindow;
+ if (pViewData)
+ pWindow = pViewData->GetActiveWin();
+ return pWindow;
+}
+
Size ScModelObj::getDocumentSize()
{
Size aSize(10, 10); // minimum size
@@ -616,30 +626,6 @@ void ScModelObj::postKeyEvent(int nType, int nCharCode,
int nKeyCode)
}
}
-void ScModelObj::postExtTextInputEvent(int nType, const OUString& rText)
-{
- SolarMutexGuard aGuard;
-
- ScViewData* pViewData = ScDocShell::GetViewData();
- vcl::Window* pWindow = pViewData->GetActiveWin();
-
- if (!pWindow)
- return;
-
- CommandExtTextInputData aTextInputData(rText, nullptr, 0, 0, false);
- switch (nType)
- {
- case LOK_EXT_TEXTINPUT:
- pWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_EXTTEXTINPUT, rText);
- break;
- case LOK_EXT_TEXTINPUT_END:
- pWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_ENDEXTTEXTINPUT, "");
- break;
- default:
- assert(false && "Unhandled External Text input event!");
- }
-}
-
void ScModelObj::postMouseEvent(int nType, int nX, int nY, int nCount, int
nButtons, int nModifier)
{
SolarMutexGuard aGuard;
diff --git a/sd/qa/unit/tiledrendering/tiledrendering.cxx
b/sd/qa/unit/tiledrendering/tiledrendering.cxx
index 4cdec1234c8e..e23ac2449841 100644
--- a/sd/qa/unit/tiledrendering/tiledrendering.cxx
+++ b/sd/qa/unit/tiledrendering/tiledrendering.cxx
@@ -49,6 +49,7 @@
#include <svx/svxids.hrc>
#include <DrawViewShell.hxx>
#include <pres.hxx>
+#include <vcl/vclevent.hxx>
#include <chrono>
@@ -1926,6 +1927,7 @@ void SdTiledRenderingTest::testIMESupport()
comphelper::LibreOfficeKit::setActive();
SdXImpressDocument* pXImpressDocument = createDoc("dummy.odp");
+ VclPtr<vcl::Window> pDocWindow = pXImpressDocument->getDocWindow();
sd::ViewShell* pViewShell =
pXImpressDocument->GetDocShell()->GetViewShell();
SdrObject* pObject = pViewShell->GetActualPage()->GetObj(0);
SdrTextObj* pTextObj = static_cast<SdrTextObj*>(pObject);
@@ -1944,9 +1946,9 @@ void SdTiledRenderingTest::testIMESupport()
});
for (const auto& aInput: aInputs)
{
- pXImpressDocument->postExtTextInputEvent(LOK_EXT_TEXTINPUT, aInput);
+ pDocWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_EXTTEXTINPUT,
aInput);
}
- pXImpressDocument->postExtTextInputEvent(LOK_EXT_TEXTINPUT_END, "");
+ pDocWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_ENDEXTTEXTINPUT, "");
// the cursor should be at position 3rd
EditView& rEditView = pView->GetTextEditOutlinerView()->GetEditView();
diff --git a/sd/source/ui/inc/unomodel.hxx b/sd/source/ui/inc/unomodel.hxx
index 1e4bbcbab7a8..aff7c906ab39 100644
--- a/sd/source/ui/inc/unomodel.hxx
+++ b/sd/source/ui/inc/unomodel.hxx
@@ -242,6 +242,7 @@ public:
virtual int getParts() override;
virtual OUString getPartName( int nPart ) override;
virtual OUString getPartHash( int nPart ) override;
+ virtual VclPtr<vcl::Window> getDocWindow() override;
virtual void setPartMode( int nPartMode ) override;
@@ -249,8 +250,6 @@ public:
virtual void initializeForTiledRendering(const
css::uno::Sequence<css::beans::PropertyValue>& rArguments) override;
/// @see vcl::ITiledRenderable::postKeyEvent().
virtual void postKeyEvent(int nType, int nCharCode, int nKeyCode) override;
- /// @see vcl::ITiledRenderable::postExtTextInputEvent().
- virtual void postExtTextInputEvent(int nType, const OUString& rText)
override;
/// @see vcl::ITiledRenderable::postMouseEvent().
virtual void postMouseEvent(int nType, int nX, int nY, int nCount, int
nButtons, int nModifier) override;
/// @see vcl::ITiledRenderable::setTextSelection().
diff --git a/sd/source/ui/unoidl/unomodel.cxx b/sd/source/ui/unoidl/unomodel.cxx
index 8d865516d032..4933b87742f9 100644
--- a/sd/source/ui/unoidl/unomodel.cxx
+++ b/sd/source/ui/unoidl/unomodel.cxx
@@ -2367,6 +2367,16 @@ OUString SdXImpressDocument::getPartHash( int nPart )
return OUString::number(pPage->GetHashCode());
}
+VclPtr<vcl::Window> SdXImpressDocument::getDocWindow()
+{
+ SolarMutexGuard aGuard;
+ DrawViewShell* pViewShell = GetViewShell();
+ VclPtr<vcl::Window> pWindow;
+ if (pViewShell)
+ pWindow = pViewShell->GetActiveWindow();
+ return pWindow;
+}
+
void SdXImpressDocument::setPartMode( int nPartMode )
{
DrawViewShell* pViewSh = GetViewShell();
@@ -2517,32 +2527,6 @@ void SdXImpressDocument::postKeyEvent(int nType, int
nCharCode, int nKeyCode)
}
}
-void SdXImpressDocument::postExtTextInputEvent(int nType, const OUString&
rText)
-{
- SolarMutexGuard aGuard;
-
- DrawViewShell* pViewShell = GetViewShell();
- if (!pViewShell)
- return;
-
- vcl::Window* pWindow = pViewShell->GetActiveWindow();
- if (!pWindow)
- return;
-
- CommandExtTextInputData aTextInputData(rText, nullptr, 0, 0, false);
- switch (nType)
- {
- case LOK_EXT_TEXTINPUT:
- pWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_EXTTEXTINPUT, rText);
- break;
- case LOK_EXT_TEXTINPUT_END:
- pWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_ENDEXTTEXTINPUT, "");
- break;
- default:
- assert(false && "Unhandled External Text input event!");
- }
-}
-
void SdXImpressDocument::postMouseEvent(int nType, int nX, int nY, int nCount,
int nButtons, int nModifier)
{
SolarMutexGuard aGuard;
diff --git a/sw/inc/unotxdoc.hxx b/sw/inc/unotxdoc.hxx
index a0cbcb0d63de..9cd47293c63c 100644
--- a/sw/inc/unotxdoc.hxx
+++ b/sw/inc/unotxdoc.hxx
@@ -415,12 +415,12 @@ public:
virtual OUString getPartName(int nPart) override;
/// @see vcl::ITiledRenderable::getPartHash().
virtual OUString getPartHash(int nPart) override;
+ /// @see vcl::ITiledRenderable::getDocWindow().
+ virtual VclPtr<vcl::Window> getDocWindow() override;
/// @see vcl::ITiledRenderable::initializeForTiledRendering().
virtual void initializeForTiledRendering(const
css::uno::Sequence<css::beans::PropertyValue>& rArguments) override;
/// @see vcl::ITiledRenderable::postKeyEvent().
virtual void postKeyEvent(int nType, int nCharCode, int nKeyCode) override;
- /// @see vcl::ITiledRenderable::postExtTextInputEvent().
- virtual void postExtTextInputEvent(int nType, const OUString& rText)
override;
/// @see vcl::ITiledRenderable::postMouseEvent().
virtual void postMouseEvent(int nType, int nX, int nY, int nCount, int
nButtons, int nModifier) override;
/// @see vcl::ITiledRenderable::setTextSelection().
diff --git a/sw/qa/extras/tiledrendering/tiledrendering.cxx
b/sw/qa/extras/tiledrendering/tiledrendering.cxx
index 05c352926d73..bb539a326149 100644
--- a/sw/qa/extras/tiledrendering/tiledrendering.cxx
+++ b/sw/qa/extras/tiledrendering/tiledrendering.cxx
@@ -38,6 +38,7 @@
#include <sfx2/dispatch.hxx>
#include <redline.hxx>
#include <IDocumentRedlineAccess.hxx>
+#include <vcl/vclevent.hxx>
#include <flddat.hxx>
static const char* DATA_DIRECTORY = "/sw/qa/extras/tiledrendering/data/";
@@ -2042,6 +2043,7 @@ void SwTiledRenderingTest::testIMESupport()
{
comphelper::LibreOfficeKit::setActive();
SwXTextDocument* pXTextDocument = createDoc("dummy.fodt");
+ VclPtr<vcl::Window> pDocWindow = pXTextDocument->getDocWindow();
SwView* pView = dynamic_cast<SwView*>(SfxViewShell::Current());
SwWrtShell* pWrtShell = pView->GetWrtShellPtr();
@@ -2055,9 +2057,9 @@ void SwTiledRenderingTest::testIMESupport()
});
for (const auto& aInput: aInputs)
{
- pXTextDocument->postExtTextInputEvent(LOK_EXT_TEXTINPUT, aInput);
+ pDocWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_EXTTEXTINPUT,
aInput);
}
- pXTextDocument->postExtTextInputEvent(LOK_EXT_TEXTINPUT_END, "");
+ pDocWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_ENDEXTTEXTINPUT, "");
// the cursor should be at position 2nd
SwShellCursor* pShellCursor = pWrtShell->getShellCursor(false);
diff --git a/sw/source/uibase/uno/unotxdoc.cxx
b/sw/source/uibase/uno/unotxdoc.cxx
index 77393d59bdaa..43101853987a 100644
--- a/sw/source/uibase/uno/unotxdoc.cxx
+++ b/sw/source/uibase/uno/unotxdoc.cxx
@@ -3402,6 +3402,16 @@ OUString SwXTextDocument::getPartHash(int nPart)
return OUString::number(sPart.hashCode());
}
+VclPtr<vcl::Window> SwXTextDocument::getDocWindow()
+{
+ SolarMutexGuard aGuard;
+ VclPtr<vcl::Window> pWindow;
+ SwView* pView = pDocShell->GetView();
+ if (pView)
+ pWindow = &(pView->GetEditWin());
+ return pWindow;
+}
+
void SwXTextDocument::initializeForTiledRendering(const
css::uno::Sequence<css::beans::PropertyValue>& rArguments)
{
SolarMutexGuard aGuard;
@@ -3501,26 +3511,6 @@ void SwXTextDocument::postKeyEvent(int nType, int
nCharCode, int nKeyCode)
}
}
-void SwXTextDocument::postExtTextInputEvent(int nType, const OUString& rText)
-{
- SolarMutexGuard aGuard;
-
- vcl::Window* pWindow = &(pDocShell->GetView()->GetEditWin());
-
- CommandExtTextInputData aTextInputData(rText, nullptr, 0, 0, false);
- switch (nType)
- {
- case LOK_EXT_TEXTINPUT:
- pWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_EXTTEXTINPUT, rText);
- break;
- case LOK_EXT_TEXTINPUT_END:
- pWindow->PostExtTextInputEvent(VCLEVENT_WINDOW_ENDEXTTEXTINPUT, "");
- break;
- default:
- assert(false && "Unhandled External Text input event!");
- }
-}
-
void SwXTextDocument::postMouseEvent(int nType, int nX, int nY, int nCount,
int nButtons, int nModifier)
{
SolarMutexGuard aGuard;
diff --git a/vcl/source/window/window.cxx b/vcl/source/window/window.cxx
index 6106198490e2..9f1e2c8c2c36 100644
--- a/vcl/source/window/window.cxx
+++ b/vcl/source/window/window.cxx
@@ -2109,8 +2109,14 @@ void Window::PostExtTextInputEvent(int nType, const
OUString& rText)
{
case VCLEVENT_WINDOW_EXTTEXTINPUT:
{
- SalExtTextInputEvent aEvent { rText, nullptr, rText.getLength(), 0 };
+ std::unique_ptr<ExtTextInputAttr[]> pAttr(new
ExtTextInputAttr[rText.getLength()]);
+ for (int i = 0; i < rText.getLength(); ++i) {
+ pAttr[i] = ExtTextInputAttr::NONE;
+ }
+ SalExtTextInputEvent aEvent { rText, pAttr.get(), rText.getLength(),
EXTTEXTINPUT_CURSOR_OVERWRITE };
ImplWindowFrameProc(this, SalEvent::ExtTextInput, &aEvent);
+ SalExtTextInputPosEvent evt;
+ ImplWindowFrameProc(this, SalEvent::ExtTextInputPos, &evt);
}
break;
case VCLEVENT_WINDOW_ENDEXTTEXTINPUT:
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
