From b38a900497ab85c5badd5f1cd1f8c00d0c6f580d Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Fri, 30 Jan 2015 14:31:58 -0800 Subject: Track formhandles in EmbedderTest class itself This removes some duplicated code from each individual test. R=thestig@chromium.org Review URL: https://codereview.chromium.org/885403002 --- fpdfsdk/src/fpdftext_embeddertest.cpp | 16 ++++------------ fpdfsdk/src/fpdfview_embeddertest.cpp | 8 +++----- 2 files changed, 7 insertions(+), 17 deletions(-) (limited to 'fpdfsdk/src') 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 14a6532a58..d0fc40d417 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