tree 14d8d60b55c0fce05d672d7724277ddec4b783cf
parent 074387f08eebc377a8e79e717987bb03241caf41
author Lei Zhang <thestig@chromium.org> 1708712770 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1708712770 +0000

Fix incorrect matrix outputs in CPDF_PageContentGenerator

When CPDF_PageContentGenerator generates content stream data for a page
object, it currently does not take the object's container's current
transformation matrix into account. As a result, the generated output
effectively has the CTM applied twice for page objects whose CTM is not
the identity matrix, as shown in recently added PDFEditImgTest cases.

Change CPDF_PageContentGenerator to take the CTM into account for images
and forms. Then update the related PDFEditImgTest cases so they pass
with the expected results. Add TODOs for CPDF_PageContentGenerator
methods that process text and paths, as there are no associated tests
for those.

Along the way, use more local matrix variables to avoid repeated calls
to getters.

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