commit 2193da94e76888232d5fb02765fc6881b7521cb7
author Lei Zhang <thestig@chromium.org> Wed Jun 30 01:03:07 2021 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jun 30 01:03:07 2021 +0000
tree 3cd66d2fac15bd6f4d1c50f27c32e6f4aad5858b
parent c8601bfbab366e56c33eb4da9d2b746f99928800

Teach FPDFPageObj_SetMatrix() to set the matrix for form objects.

Convert some embedder tests to use the new API capability, instead of
accessing the internal CPDF_FormObject directly.

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

fpdfsdk/fpdf_ppo_embeddertest.cpp

diff --git a/fpdfsdk/fpdf_ppo_embeddertest.cpp b/fpdfsdk/fpdf_ppo_embeddertest.cpp
index dc5130c..f635996 100644
--- a/fpdfsdk/fpdf_ppo_embeddertest.cpp
+++ b/fpdfsdk/fpdf_ppo_embeddertest.cpp
@@ -249,13 +249,8 @@
   ASSERT_TRUE(page_object);
   ASSERT_EQ(FPDF_PAGEOBJ_FORM, FPDFPageObj_GetType(page_object));
 
-  // Access the CPDF_FormObject underneath, as there is no public API to set
-  // the matrix for form objects. (yet)
   static constexpr FS_MATRIX kMatrix = {0.5f, 0.0f, 0.0f, 0.5f, 0.0f, 0.0f};
-  CPDF_FormObject* pFormObj =
-      CPDFPageObjectFromFPDFPageObject(page_object)->AsForm();
-  pFormObj->Transform(CFXMatrixFromFSMatrix(kMatrix));
-  pFormObj->SetDirty(true);
+  EXPECT_TRUE(FPDFPageObj_SetMatrix(page_object, &kMatrix));
 
   FPDFPage_InsertObject(page, page_object);
   EXPECT_TRUE(FPDFPage_GenerateContent(page));