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 <thestig@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org>
diff --git a/fxjs/cjs_document.cpp b/fxjs/cjs_document.cpp index c839e9f..35bb240 100644 --- a/fxjs/cjs_document.cpp +++ b/fxjs/cjs_document.cpp
@@ -440,15 +440,10 @@ // 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(); }
diff --git a/testing/resources/javascript/bug_679642_expected.txt b/testing/resources/javascript/bug_679642_expected.txt index 9e99cc6..b576eae 100644 --- a/testing/resources/javascript/bug_679642_expected.txt +++ b/testing/resources/javascript/bug_679642_expected.txt
@@ -1,3 +1,2 @@ Alert: Starting ... Alert: Firing ... -Alert: failed: Field.page: Object no longer exists.
diff --git a/testing/resources/javascript/bug_679643_expected.txt b/testing/resources/javascript/bug_679643_expected.txt index 36d4a31..b576eae 100644 --- a/testing/resources/javascript/bug_679643_expected.txt +++ b/testing/resources/javascript/bug_679643_expected.txt
@@ -1,3 +1,2 @@ Alert: Starting ... Alert: Firing ... -Alert: failed: Annot.name: Object no longer exists.
diff --git a/testing/resources/javascript/xfa_specific/resolve_nodes_1_expected.txt b/testing/resources/javascript/xfa_specific/resolve_nodes_1_expected.txt index d4cc442..335a4e2 100644 --- a/testing/resources/javascript/xfa_specific/resolve_nodes_1_expected.txt +++ b/testing/resources/javascript/xfa_specific/resolve_nodes_1_expected.txt
@@ -1 +1 @@ -: \Find 1 items +: Find 12 items