commit 44005193b7eba10e4193135cb51ee25ff0f16b2b
author Lei Zhang <thestig@chromium.org> Fri Apr 17 01:35:43 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Fri Apr 17 01:35:43 2020 +0000
tree 55fdec167fcefdd860bb619a70d5d509c9a238ca
parent 6021a9f4d0407fc4499cb36858aca69544d17e6a

Call FORM_DoDocumentAAction() in EmbedderTest::CloseDocument().

The |FPDFDOC_AACTION_WC| action is meant to run when the document is
closing. Tie this to CloseDocument(), rather than TearDown().

Change-Id: Ic7dd9c42430e834dec5fdff1480133cbcc391d08
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/68854
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

testing/embedder_test.cpp

diff --git a/testing/embedder_test.cpp b/testing/embedder_test.cpp
index caba758..69842a4 100644
--- a/testing/embedder_test.cpp
+++ b/testing/embedder_test.cpp
@@ -86,10 +86,8 @@
   EXPECT_EQ(0U, page_map_.size());
   EXPECT_EQ(0U, saved_page_map_.size());
 
-  if (document_) {
-    FORM_DoDocumentAAction(form_handle_, FPDFDOC_AACTION_WC);
+  if (document_)
     CloseDocument();
-  }
 
   FPDFAvail_Destroy(avail_);
   FPDF_DestroyLibrary();
@@ -229,6 +227,7 @@
 }
 
 void EmbedderTest::CloseDocument() {
+  FORM_DoDocumentAAction(form_handle_, FPDFDOC_AACTION_WC);
   FPDFDOC_ExitFormFillEnvironment(form_handle_);
   form_handle_ = nullptr;