From 04c2dbb1908006830b6ee45ea85f6cde1a9fc4cb Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Tue, 2 Oct 2018 21:23:50 +0000 Subject: Remove ability to delete annot in CJS_Document::removeField() This path has only been present under XFA, and has resulted in several bugs, including the referenced one. There is a breakage in some XFA functionality, but I'm not confident we can do this without re-engineering all of fpdfsdk widget ownership. Bug: 891210 Change-Id: I5aa158fa5359b1c8338046eefb25fb25df9ace48 Reviewed-on: https://pdfium-review.googlesource.com/c/43311 Reviewed-by: Lei Zhang Commit-Queue: Tom Sepez --- fxjs/cjs_document.cpp | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) (limited to 'fxjs') diff --git a/fxjs/cjs_document.cpp b/fxjs/cjs_document.cpp index c839e9f487..35bb240671 100644 --- a/fxjs/cjs_document.cpp +++ b/fxjs/cjs_document.cpp @@ -440,15 +440,10 @@ CJS_Result CJS_Document::removeField( // do not create one. We may be in the process of tearing down the document // and creating a new pageview at this point will cause bad things. CPDFSDK_PageView* pPageView = m_pFormFillEnv->GetPageView(pPage, false); - if (pPageView) { -#ifdef PDF_ENABLE_XFA - pPageView->DeleteAnnot(pWidget); -#endif // PDF_ENABLE_XFA + if (pPageView) pPageView->UpdateRects(aRefresh); - } } m_pFormFillEnv->SetChangeMark(); - return CJS_Result::Success(); } -- cgit v1.2.3