Add an embedder test for inserted text not saving correctly [pdfium : main]

0 views
Skip to first unread message

Andy Phan (Gerrit)

unread,
Mar 13, 2026, 1:00:57 PM (4 days ago) Mar 13
to Lei Zhang, Pdfium LUCI CQ, pdfium-...@googlegroups.com
Attention needed from Lei Zhang

Andy Phan voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Lei Zhang
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: pdfium
Gerrit-Branch: main
Gerrit-Change-Id: Ib3a7ecf913d0c6693e5f28bc25e8a8a33d4b88a8
Gerrit-Change-Number: 144790
Gerrit-PatchSet: 5
Gerrit-Owner: Lei Zhang <the...@chromium.org>
Gerrit-Reviewer: Andy Phan <andy...@chromium.org>
Gerrit-Reviewer: Lei Zhang <the...@chromium.org>
Gerrit-Attention: Lei Zhang <the...@chromium.org>
Gerrit-Comment-Date: Fri, 13 Mar 2026 17:00:53 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Lei Zhang (Gerrit)

unread,
Mar 13, 2026, 1:46:42 PM (4 days ago) Mar 13
to Lei Zhang, Andy Phan, Pdfium LUCI CQ, pdfium-...@googlegroups.com

Lei Zhang voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: pdfium
Gerrit-Branch: main
Gerrit-Change-Id: Ib3a7ecf913d0c6693e5f28bc25e8a8a33d4b88a8
Gerrit-Change-Number: 144790
Gerrit-PatchSet: 5
Gerrit-Owner: Lei Zhang <the...@chromium.org>
Gerrit-Reviewer: Andy Phan <andy...@chromium.org>
Gerrit-Reviewer: Lei Zhang <the...@chromium.org>
Gerrit-Comment-Date: Fri, 13 Mar 2026 17:46:38 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Pdfium LUCI CQ (Gerrit)

unread,
Mar 13, 2026, 1:47:25 PM (4 days ago) Mar 13
to Lei Zhang, Andy Phan, pdfium-...@googlegroups.com

Pdfium LUCI CQ submitted the change

Change information

Commit message:
Add an embedder test for inserted text not saving correctly

Add a test to show the editing steps to add text to a particular page
renders correctly in memory, but the text is missing when saved. Thanks
to prior investigations on the bug, there was already some embedder test
code and the problematic portion of the sample PDF has already been
identified.
Bug: 461845674
Change-Id: Ib3a7ecf913d0c6693e5f28bc25e8a8a33d4b88a8
Commit-Queue: Lei Zhang <the...@chromium.org>
Reviewed-by: Andy Phan <andy...@chromium.org>
Files:
  • M fpdfsdk/fpdf_edit_embeddertest.cpp
  • A testing/resources/bug_461845674.in
  • A testing/resources/bug_461845674.pdf
  • A testing/resources/embedder_tests/bug_461845674_agg.png
  • A testing/resources/embedder_tests/bug_461845674_agg_mac.png
  • A testing/resources/embedder_tests/bug_461845674_skia.png
  • A testing/resources/embedder_tests/bug_461845674_skia_mac.png
  • A testing/resources/embedder_tests/bug_461845674_skia_win.png
Change size: M
Delta: 8 files changed, 105 insertions(+), 0 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Andy Phan
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: pdfium
Gerrit-Branch: main
Gerrit-Change-Id: Ib3a7ecf913d0c6693e5f28bc25e8a8a33d4b88a8
Gerrit-Change-Number: 144790
Gerrit-PatchSet: 6
Gerrit-Owner: Lei Zhang <the...@chromium.org>
Gerrit-Reviewer: Andy Phan <andy...@chromium.org>
Gerrit-Reviewer: Lei Zhang <the...@chromium.org>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages