From a5b47041066011eb590a020bb38035442fce48af Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Mon, 19 Oct 2015 14:32:16 -0700 Subject: Merge to XFA: Remove dead code that was reactivated when fixing overrides. BUG=pdfium:205 TBR=tsepez@chromium.org Review URL: https://codereview.chromium.org/1409743004 . (cherry picked from commit 9ddafc82bddb984ae46ee0df801ba20b446d5158) Review URL: https://codereview.chromium.org/1411423002 . --- fpdfsdk/src/formfiller/FFL_ComboBox.cpp | 18 ----------- fpdfsdk/src/formfiller/FFL_FormFiller.cpp | 2 -- fpdfsdk/src/formfiller/FFL_IFormFiller.cpp | 17 +--------- fpdfsdk/src/formfiller/FFL_ListBox.cpp | 18 ----------- fpdfsdk/src/pdfwindow/PWL_Edit.cpp | 51 ++++++------------------------ fpdfsdk/src/pdfwindow/PWL_ListBox.cpp | 25 +++++++-------- 6 files changed, 22 insertions(+), 109 deletions(-) (limited to 'fpdfsdk/src') diff --git a/fpdfsdk/src/formfiller/FFL_ComboBox.cpp b/fpdfsdk/src/formfiller/FFL_ComboBox.cpp index ea8d295dff..b4619cb32d 100644 --- a/fpdfsdk/src/formfiller/FFL_ComboBox.cpp +++ b/fpdfsdk/src/formfiller/FFL_ComboBox.cpp @@ -256,24 +256,6 @@ CPWL_Wnd* CFFL_ComboBox::ResetPDFWindow(CPDFSDK_PageView* pPageView, return pRet; } -void CFFL_ComboBox::OnKeyStroke(FX_BOOL bKeyDown, FX_UINT nFlag) { - ASSERT(m_pWidget != NULL); - - int nFlags = m_pWidget->GetFieldFlags(); - - if (nFlags & FIELDFLAG_COMMITONSELCHANGE) { - if (m_bValid) { - CPDFSDK_PageView* pPageView = GetCurPageView(); - ASSERT(pPageView != NULL); - - if (CommitData(pPageView, nFlag)) { - DestroyPDFWindow(pPageView); - m_bValid = FALSE; - } - } - } -} - FX_BOOL CFFL_ComboBox::IsFieldFull(CPDFSDK_PageView* pPageView) { if (CPWL_ComboBox* pComboBox = (CPWL_ComboBox*)GetPDFWindow(pPageView, FALSE)) { diff --git a/fpdfsdk/src/formfiller/FFL_FormFiller.cpp b/fpdfsdk/src/formfiller/FFL_FormFiller.cpp index 30a297dd68..b6224294df 100644 --- a/fpdfsdk/src/formfiller/FFL_FormFiller.cpp +++ b/fpdfsdk/src/formfiller/FFL_FormFiller.cpp @@ -602,8 +602,6 @@ IFX_SystemHandler* CFFL_FormFiller::GetSystemHandler() const { return m_pApp->GetSysHandler(); } -void CFFL_FormFiller::OnKeyStroke(FX_BOOL bKeyDown, FX_DWORD nFlag) {} - void CFFL_FormFiller::EscapeFiller(CPDFSDK_PageView* pPageView, FX_BOOL bDestroyPDFWindow) { m_bValid = FALSE; diff --git a/fpdfsdk/src/formfiller/FFL_IFormFiller.cpp b/fpdfsdk/src/formfiller/FFL_IFormFiller.cpp index c1450f6c84..6ef8785636 100644 --- a/fpdfsdk/src/formfiller/FFL_IFormFiller.cpp +++ b/fpdfsdk/src/formfiller/FFL_IFormFiller.cpp @@ -986,9 +986,7 @@ FX_BOOL CFFL_IFormFiller::IsValidAnnot(CPDFSDK_PageView* pPageView, return FALSE; } -void CFFL_IFormFiller::OnBeforeKeyStroke(FX_BOOL bEditOrList, - void* pPrivateData, - int32_t nKeyCode, +void CFFL_IFormFiller::OnBeforeKeyStroke(void* pPrivateData, CFX_WideString& strChange, const CFX_WideString& strChangeEx, int nSelStart, @@ -1078,16 +1076,3 @@ void CFFL_IFormFiller::OnBeforeKeyStroke(FX_BOOL bEditOrList, } } } - -void CFFL_IFormFiller::OnAfterKeyStroke(FX_BOOL bEditOrList, - void* pPrivateData, - FX_BOOL& bExit, - FX_DWORD nFlag) { - CFFL_PrivateData* pData = (CFFL_PrivateData*)pPrivateData; - ASSERT(pData->pWidget); - - CFFL_FormFiller* pFormFiller = GetFormFiller(pData->pWidget, FALSE); - - if (!bEditOrList) - pFormFiller->OnKeyStroke(bExit, nFlag); -} diff --git a/fpdfsdk/src/formfiller/FFL_ListBox.cpp b/fpdfsdk/src/formfiller/FFL_ListBox.cpp index 0f478fe7e0..343683016f 100644 --- a/fpdfsdk/src/formfiller/FFL_ListBox.cpp +++ b/fpdfsdk/src/formfiller/FFL_ListBox.cpp @@ -228,21 +228,3 @@ CPWL_Wnd* CFFL_ListBox::ResetPDFWindow(CPDFSDK_PageView* pPageView, return pRet; } - -void CFFL_ListBox::OnKeyStroke(FX_BOOL bKeyDown, FX_DWORD nFlag) { - ASSERT(m_pWidget != NULL); - - int nFlags = m_pWidget->GetFieldFlags(); - - if (nFlags & FIELDFLAG_COMMITONSELCHANGE) { - if (m_bValid) { - CPDFSDK_PageView* pPageView = GetCurPageView(); - ASSERT(pPageView != NULL); - - if (CommitData(pPageView, nFlag)) { - DestroyPDFWindow(pPageView); - m_bValid = FALSE; - } - } - } -} diff --git a/fpdfsdk/src/pdfwindow/PWL_Edit.cpp b/fpdfsdk/src/pdfwindow/PWL_Edit.cpp index 3271c80fdb..8e45060ea9 100644 --- a/fpdfsdk/src/pdfwindow/PWL_Edit.cpp +++ b/fpdfsdk/src/pdfwindow/PWL_Edit.cpp @@ -163,7 +163,7 @@ void CPWL_Edit::PasteText() { int nSelStart = 0; int nSelEnd = 0; GetSel(nSelStart, nSelEnd); - m_pFillerNotify->OnBeforeKeyStroke(TRUE, GetAttachedData(), 0, swClipboard, + m_pFillerNotify->OnBeforeKeyStroke(GetAttachedData(), swClipboard, strChangeEx, nSelStart, nSelEnd, TRUE, bRC, bExit, 0); if (!bRC) @@ -176,13 +176,6 @@ void CPWL_Edit::PasteText() { Clear(); InsertText(swClipboard.c_str()); } - - if (m_pFillerNotify) { - FX_BOOL bExit = FALSE; - m_pFillerNotify->OnAfterKeyStroke(TRUE, GetAttachedData(), bExit, 0); - if (bExit) - return; - } } void CPWL_Edit::CutText() { @@ -883,9 +876,9 @@ FX_BOOL CPWL_Edit::OnKeyDown(FX_WORD nChar, FX_DWORD nFlag) { if (nSelStart == nSelEnd) nSelEnd = nSelStart + 1; - m_pFillerNotify->OnBeforeKeyStroke( - TRUE, GetAttachedData(), FWL_VKEY_Delete, strChange, strChangeEx, - nSelStart, nSelEnd, TRUE, bRC, bExit, nFlag); + m_pFillerNotify->OnBeforeKeyStroke(GetAttachedData(), strChange, + strChangeEx, nSelStart, nSelEnd, TRUE, + bRC, bExit, nFlag); if (!bRC) return FALSE; if (bExit) @@ -895,15 +888,6 @@ FX_BOOL CPWL_Edit::OnKeyDown(FX_WORD nChar, FX_DWORD nFlag) { FX_BOOL bRet = CPWL_EditCtrl::OnKeyDown(nChar, nFlag); - if (nChar == FWL_VKEY_Delete) { - if (m_pFillerNotify) { - FX_BOOL bExit = FALSE; - m_pFillerNotify->OnAfterKeyStroke(TRUE, GetAttachedData(), bExit, nFlag); - if (bExit) - return FALSE; - } - } - // In case of implementation swallow the OnKeyDown event. if (IsProceedtoOnChar(nChar, nFlag)) return TRUE; @@ -940,9 +924,8 @@ FX_BOOL CPWL_Edit::IsProceedtoOnChar(FX_WORD nKeyCode, FX_DWORD nFlag) { case FWL_VKEY_Space: return TRUE; default: - break; + return FALSE; } - return FALSE; } FX_BOOL CPWL_Edit::OnChar(FX_WORD nChar, FX_DWORD nFlag) { @@ -952,11 +935,9 @@ FX_BOOL CPWL_Edit::OnChar(FX_WORD nChar, FX_DWORD nFlag) { FX_BOOL bRC = TRUE; FX_BOOL bExit = FALSE; - FX_BOOL bCtrl = IsCTRLpressed(nFlag); - if (!bCtrl) { + if (!IsCTRLpressed(nFlag)) { if (m_pFillerNotify) { CFX_WideString swChange; - int32_t nKeyCode; int nSelStart = 0; int nSelEnd = 0; @@ -964,23 +945,20 @@ FX_BOOL CPWL_Edit::OnChar(FX_WORD nChar, FX_DWORD nFlag) { switch (nChar) { case FWL_VKEY_Back: - nKeyCode = nChar; if (nSelStart == nSelEnd) nSelStart = nSelEnd - 1; break; case FWL_VKEY_Return: - nKeyCode = nChar; break; default: - nKeyCode = 0; swChange += nChar; break; } CFX_WideString strChangeEx; - m_pFillerNotify->OnBeforeKeyStroke(TRUE, GetAttachedData(), nKeyCode, - swChange, strChangeEx, nSelStart, - nSelEnd, TRUE, bRC, bExit, nFlag); + m_pFillerNotify->OnBeforeKeyStroke(GetAttachedData(), swChange, + strChangeEx, nSelStart, nSelEnd, TRUE, + bRC, bExit, nFlag); } } @@ -996,17 +974,8 @@ FX_BOOL CPWL_Edit::OnChar(FX_WORD nChar, FX_DWORD nFlag) { SetCharSet(nNewCharSet); } } - FX_BOOL bRet = CPWL_EditCtrl::OnChar(nChar, nFlag); - - if (!bCtrl) { - if (m_pFillerNotify) { - m_pFillerNotify->OnAfterKeyStroke(TRUE, GetAttachedData(), bExit, nFlag); - if (bExit) - return FALSE; - } - } - return bRet; + return CPWL_EditCtrl::OnChar(nChar, nFlag); } FX_BOOL CPWL_Edit::OnMouseWheel(short zDelta, diff --git a/fpdfsdk/src/pdfwindow/PWL_ListBox.cpp b/fpdfsdk/src/pdfwindow/PWL_ListBox.cpp index b252bffb8e..4bdcb5d1f3 100644 --- a/fpdfsdk/src/pdfwindow/PWL_ListBox.cpp +++ b/fpdfsdk/src/pdfwindow/PWL_ListBox.cpp @@ -374,20 +374,17 @@ void CPWL_ListBox::RePosChildWnd() { void CPWL_ListBox::OnNotifySelChanged(FX_BOOL bKeyDown, FX_BOOL& bExit, FX_DWORD nFlag) { - if (m_pFillerNotify) { - FX_BOOL bRC = TRUE; - CFX_WideString swChange = GetText(); - CFX_WideString strChangeEx; - int nSelStart = 0; - int nSelEnd = swChange.GetLength(); - m_pFillerNotify->OnBeforeKeyStroke(FALSE, GetAttachedData(), 0, swChange, - strChangeEx, nSelStart, nSelEnd, - bKeyDown, bRC, bExit, nFlag); - if (bExit) - return; - - m_pFillerNotify->OnAfterKeyStroke(FALSE, GetAttachedData(), bExit, nFlag); - } + if (!m_pFillerNotify) + return; + + FX_BOOL bRC = TRUE; + CFX_WideString swChange = GetText(); + CFX_WideString strChangeEx; + int nSelStart = 0; + int nSelEnd = swChange.GetLength(); + m_pFillerNotify->OnBeforeKeyStroke(GetAttachedData(), swChange, strChangeEx, + nSelStart, nSelEnd, bKeyDown, bRC, bExit, + nFlag); } CPDF_Rect CPWL_ListBox::GetFocusRect() const { -- cgit v1.2.3