commit 1f4904f680cd7a63d96f072a371ef20ce939cd7a
author Lei Zhang <thestig@chromium.org> Wed Mar 29 17:58:17 2023 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Mar 29 17:58:17 2023 +0000
tree 1511ef771a714d808e35a83ded33fb4c9a2a9953
parent 5018240ebf009a67fc9b2613a6f3f8bda1e80c03

Test trailer generation in FPDFSaveEmbedderTest.SaveLinearizedDoc

Make sure the /Size in the trailer is correct. Also check for the
existence of objects that influence the /Size.

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

fpdfsdk/fpdf_save_embeddertest.cpp

diff --git a/fpdfsdk/fpdf_save_embeddertest.cpp b/fpdfsdk/fpdf_save_embeddertest.cpp
index 18868a8..1289a1f 100644
--- a/fpdfsdk/fpdf_save_embeddertest.cpp
+++ b/fpdfsdk/fpdf_save_embeddertest.cpp
@@ -113,6 +113,11 @@
   EXPECT_THAT(GetString(), StartsWith("%PDF-1.6\r\n"));
   EXPECT_THAT(GetString(), HasSubstr("/Root "));
   EXPECT_THAT(GetString(), HasSubstr("/Info "));
+  EXPECT_THAT(GetString(), HasSubstr("/Size 38"));
+  EXPECT_THAT(GetString(), HasSubstr("35 0 obj"));
+  EXPECT_THAT(GetString(), HasSubstr("36 0 obj"));
+  EXPECT_THAT(GetString(), HasSubstr("37 0 obj"));
+  EXPECT_THAT(GetString(), Not(HasSubstr("38 0 obj")));
   EXPECT_EQ(8219u, GetString().size());
 
   // Make sure new document renders the same as the old one.