commit b7c41498ac1f09debe0b12efe1312a22246a18b0
author Lei Zhang <thestig@chromium.org> Tue May 08 19:01:28 2018 +0000
committer Chromium commit bot <commit-bot@chromium.org> Tue May 08 19:01:28 2018 +0000
tree 695be97b644ed76bca59825d862715effe82509f
parent 4ec16d7c89f4128e00de7247ef7e36b1f5d7f267

Add some basic tests for FPDF_VIEWERREF_GetPrintPageRange().

Hand edit viewer_ref.pdf to have a page range and more pages. Fix an
invalid stream length in the file as well. Fix tests that depended on
viewer_ref.pdf having only 1 page.

Change-Id: I95d5cbf1f592d1c51f11d9ab5b26abf2f23ed598
Reviewed-on: https://pdfium-review.googlesource.com/32172
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

fpdfsdk/fpdf_ppo_embeddertest.cpp

diff --git a/fpdfsdk/fpdf_ppo_embeddertest.cpp b/fpdfsdk/fpdf_ppo_embeddertest.cpp
index 8412771..84e97dc 100644
--- a/fpdfsdk/fpdf_ppo_embeddertest.cpp
+++ b/fpdfsdk/fpdf_ppo_embeddertest.cpp
@@ -151,7 +151,7 @@
 }
 
 TEST_F(FPDFPPOEmbeddertest, BadRanges) {
-  EXPECT_TRUE(OpenDocument("viewer_ref.pdf"));
+  EXPECT_TRUE(OpenDocument("hello_world.pdf"));
 
   FPDF_PAGE page = LoadPage(0);
   EXPECT_TRUE(page);
@@ -183,8 +183,13 @@
   EXPECT_TRUE(output_doc);
   EXPECT_TRUE(FPDF_CopyViewerPreferences(output_doc, document()));
   EXPECT_TRUE(FPDF_ImportPages(output_doc, document(), "1,1,1,1", 0));
+  EXPECT_EQ(4, FPDF_GetPageCount(output_doc));
   EXPECT_TRUE(FPDF_ImportPages(output_doc, document(), "1-1", 0));
   EXPECT_EQ(5, FPDF_GetPageCount(output_doc));
+  EXPECT_TRUE(FPDF_ImportPages(output_doc, document(), "5-5", 0));
+  EXPECT_EQ(6, FPDF_GetPageCount(output_doc));
+  EXPECT_TRUE(FPDF_ImportPages(output_doc, document(), "2-4", 0));
+  EXPECT_EQ(9, FPDF_GetPageCount(output_doc));
   FPDF_CloseDocument(output_doc);
 
   UnloadPage(page);