commit 6664ba96d8d461af53428464dad1a9ed753cc67a
author JD Pierce <brkfstmnchr@gmail.com> Fri Jun 14 16:27:44 2024 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Jun 14 16:27:44 2024 +0000
tree 1ad1d10a4248da2fee02d76007e12397e54d3c7b
parent 0f93d8e142755aed4ef1674ae2b9fc26e3d0ebb6

Allow Grayscale Colorspaces for Bug 42271133

- Update FPDFSaveEmbedderTest::42271133()
- Add CPDF_Color::IsColorSpaceGray()
- Update CPDF_ContentGenerator::WriteColorToStream() to check for
grayscale color space

Note: Regression test added in https://pdfium-review.googlesource.com/c/pdfium/+/118890

Change-Id: Ia1e33d568a05eb9a7c32243670263257d0c59930
Bug: 42271133
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/115330
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Thomas Sepez <tsepez@google.com>
Commit-Queue: Thomas Sepez <tsepez@google.com>

fpdfsdk/fpdf_save_embeddertest.cpp

diff --git a/fpdfsdk/fpdf_save_embeddertest.cpp b/fpdfsdk/fpdf_save_embeddertest.cpp
index f221d10..4f33c2a 100644
--- a/fpdfsdk/fpdf_save_embeddertest.cpp
+++ b/fpdfsdk/fpdf_save_embeddertest.cpp
@@ -123,10 +123,9 @@
   unsigned int b;
   unsigned int a;
   ASSERT_TRUE(FPDFPageObj_GetFillColor(path_obj, &r, &g, &b, &a));
-  // TODO(crbug.com/42271133): Change the assertion to EXPECT_NE().
-  EXPECT_EQ(0u, r);
-  EXPECT_EQ(0u, g);
-  EXPECT_EQ(0u, b);
+  EXPECT_EQ(180u, r);
+  EXPECT_EQ(180u, g);
+  EXPECT_EQ(180u, b);
 
   CloseSavedPage(saved_page);
   CloseSavedDocument();