From a7ff4dc7c27c7940daec9cf740f4b7e7638a45ec Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Mon, 14 May 2018 20:13:40 +0000 Subject: Cleanup CPDF_Form parsing code This CL folds the StartParse() method of CPDF_Form into the ParserContent method. The no arguments ParseContent is removed and ParseContentWithParams renamed to ParseContent. The callsites are updated to pass the nullptr's. Bug: chromium:813349 Change-Id: I304b77aef1de1b9aa20e4a3044db5023f5701584 Reviewed-on: https://pdfium-review.googlesource.com/32511 Reviewed-by: Henrique Nakashima Commit-Queue: dsinclair --- core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp') diff --git a/core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp b/core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp index e9b3d15d30..d97e8abf0b 100644 --- a/core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp +++ b/core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp @@ -298,7 +298,7 @@ TEST_F(CPDF_PageContentGeneratorTest, ProcessEmptyForm) { // Create an empty form. auto pTestForm = pdfium::MakeUnique(pDoc.get(), nullptr, pStream.get()); - pTestForm->ParseContent(); + pTestForm->ParseContent(nullptr, nullptr, nullptr, nullptr); ASSERT_TRUE(pTestForm->IsParsed()); // The generated stream for the empty form should be an empty string. @@ -324,7 +324,7 @@ TEST_F(CPDF_PageContentGeneratorTest, ProcessFormWithPath) { // Create a form with a non-empty stream. auto pTestForm = pdfium::MakeUnique(pDoc.get(), nullptr, pStream.get()); - pTestForm->ParseContent(); + pTestForm->ParseContent(nullptr, nullptr, nullptr, nullptr); ASSERT_TRUE(pTestForm->IsParsed()); CPDF_PageContentGenerator generator(pTestForm.get()); -- cgit v1.2.3