summaryrefslogtreecommitdiff
path: root/xfa/fwl
diff options
context:
space:
mode:
authorHenrique Nakashima <hnakashima@chromium.org>2018-04-27 21:10:57 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-04-27 21:10:57 +0000
commit5ee10423d237c5bd22b8db6fe6d7b53edd0906dc (patch)
tree6d18497746c4913e1e7fc9522919a03a86366e9a /xfa/fwl
parent724f61671bec82f521be75b0ace069fb3130902a (diff)
downloadpdfium-5ee10423d237c5bd22b8db6fe6d7b53edd0906dc.tar.xz
Update caret after changes that could move lines in XFA edit.
This fixes issues when a line break is deleted and the caret remains where it was, now between two lines. Bug: chromium:836361 Change-Id: I6cd40402a97348bcfc4c9291e7acb044a1e68113 Reviewed-on: https://pdfium-review.googlesource.com/31391 Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Reviewed-by: dsinclair <dsinclair@chromium.org> Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa/fwl')
-rw-r--r--xfa/fwl/cfwl_edit.cpp6
1 files changed, 5 insertions, 1 deletions
diff --git a/xfa/fwl/cfwl_edit.cpp b/xfa/fwl/cfwl_edit.cpp
index 8ebc155a80..dd7bcfae3a 100644
--- a/xfa/fwl/cfwl_edit.cpp
+++ b/xfa/fwl/cfwl_edit.cpp
@@ -236,7 +236,9 @@ Optional<WideString> CFWL_Edit::Cut() {
if (!m_EdtEngine.HasSelection())
return {};
- return {m_EdtEngine.DeleteSelectedText()};
+ WideString cut_text = m_EdtEngine.DeleteSelectedText();
+ UpdateCaret();
+ return {cut_text};
}
bool CFWL_Edit::Paste(const WideString& wsPaste) {
@@ -1225,6 +1227,7 @@ void CFWL_Edit::OnKeyDown(CFWL_MessageKey* pMsg) {
}
m_EdtEngine.Delete(m_CursorPosition, 1);
+ UpdateCaret();
break;
}
case FWL_VKEY_Insert:
@@ -1254,6 +1257,7 @@ void CFWL_Edit::OnChar(CFWL_MessageKey* pMsg) {
if (m_CursorPosition > 0) {
SetCursorPosition(m_CursorPosition - 1);
m_EdtEngine.Delete(m_CursorPosition, 1);
+ UpdateCaret();
}
break;
case L'\n':