From da8189e65547f87a9c7ba3ebb72f78ecdb47e120 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Fri, 30 Jan 2015 14:41:50 -0800 Subject: Merge to XFA: track formhandles in embeddder test class itself TBR=thestig@chromium.org Review URL: https://codereview.chromium.org/891763003 --- fpdfsdk/src/fpdftext_embeddertest.cpp | 16 ++++------------ fpdfsdk/src/fpdfview_embeddertest.cpp | 8 +++----- 2 files changed, 7 insertions(+), 17 deletions(-) (limited to 'fpdfsdk') diff --git a/fpdfsdk/src/fpdftext_embeddertest.cpp b/fpdfsdk/src/fpdftext_embeddertest.cpp index 1457832658..f0d9feb91d 100644 --- a/fpdfsdk/src/fpdftext_embeddertest.cpp +++ b/fpdfsdk/src/fpdftext_embeddertest.cpp @@ -30,8 +30,7 @@ class FPDFTextEmbeddertest : public EmbedderTest { TEST_F(FPDFTextEmbeddertest, Text) { EXPECT_TRUE(OpenDocument("testing/resources/hello_world.pdf")); - FPDF_FORMHANDLE form_handle = SetFormFillEnvironment(); - FPDF_PAGE page = LoadPage(0, form_handle); + FPDF_PAGE page = LoadPage(0); EXPECT_NE(nullptr, page); FPDF_TEXTPAGE textpage = FPDFText_LoadPage(page); @@ -135,13 +134,11 @@ TEST_F(FPDFTextEmbeddertest, Text) { EXPECT_EQ(0xbdbd, fixed_buffer[10]); FPDFText_ClosePage(textpage); - ClearFormFillEnvironment(form_handle); } TEST_F(FPDFTextEmbeddertest, TextSearch) { EXPECT_TRUE(OpenDocument("testing/resources/hello_world.pdf")); - FPDF_FORMHANDLE form_handle = SetFormFillEnvironment(); - FPDF_PAGE page = LoadPage(0, form_handle); + FPDF_PAGE page = LoadPage(0); EXPECT_NE(nullptr, page); FPDF_TEXTPAGE textpage = FPDFText_LoadPage(page); @@ -242,14 +239,12 @@ TEST_F(FPDFTextEmbeddertest, TextSearch) { FPDFText_FindClose(search); FPDFText_ClosePage(textpage); - ClearFormFillEnvironment(form_handle); } // Test that the page has characters despite a bad stream length. TEST_F(FPDFTextEmbeddertest, StreamLengthPastEndOfFile) { EXPECT_TRUE(OpenDocument("testing/resources/bug_57.pdf")); - FPDF_FORMHANDLE form_handle = SetFormFillEnvironment(); - FPDF_PAGE page = LoadPage(0, form_handle); + FPDF_PAGE page = LoadPage(0); EXPECT_NE(nullptr, page); FPDF_TEXTPAGE textpage = FPDFText_LoadPage(page); @@ -257,13 +252,11 @@ TEST_F(FPDFTextEmbeddertest, StreamLengthPastEndOfFile) { EXPECT_EQ(13, FPDFText_CountChars(textpage)); FPDFText_ClosePage(textpage); - ClearFormFillEnvironment(form_handle); } TEST_F(FPDFTextEmbeddertest, WebLinks) { EXPECT_TRUE(OpenDocument("testing/resources/weblinks.pdf")); - FPDF_FORMHANDLE form_handle = SetFormFillEnvironment(); - FPDF_PAGE page = LoadPage(0, form_handle); + FPDF_PAGE page = LoadPage(0); EXPECT_NE(nullptr, page); FPDF_TEXTPAGE textpage = FPDFText_LoadPage(page); @@ -365,5 +358,4 @@ TEST_F(FPDFTextEmbeddertest, WebLinks) { FPDFLink_CloseWebLinks(pagelink); FPDFText_ClosePage(textpage); - ClearFormFillEnvironment(form_handle); } diff --git a/fpdfsdk/src/fpdfview_embeddertest.cpp b/fpdfsdk/src/fpdfview_embeddertest.cpp index 76dd9a5798..4827f76839 100644 --- a/fpdfsdk/src/fpdfview_embeddertest.cpp +++ b/fpdfsdk/src/fpdfview_embeddertest.cpp @@ -27,14 +27,12 @@ TEST_F(FPDFViewEmbeddertest, Document) { TEST_F(FPDFViewEmbeddertest, Page) { EXPECT_TRUE(OpenDocument("testing/resources/about_blank.pdf")); - FPDF_FORMHANDLE form_handle = SetFormFillEnvironment(); - FPDF_PAGE page = LoadPage(0, form_handle); + FPDF_PAGE page = LoadPage(0); EXPECT_NE(nullptr, page); EXPECT_EQ(612.0, FPDF_GetPageWidth(page)); EXPECT_EQ(792.0, FPDF_GetPageHeight(page)); - UnloadPage(page, form_handle); - EXPECT_EQ(nullptr, LoadPage(1, form_handle)); - ClearFormFillEnvironment(form_handle); + UnloadPage(page); + EXPECT_EQ(nullptr, LoadPage(1)); } TEST_F(FPDFViewEmbeddertest, ViewerRef) { -- cgit v1.2.3