Add a broken test for flattening AcroForms [pdfium : main]

0 views
Skip to first unread message

April Kallmeyer (Gerrit)

unread,
Apr 7, 2026, 4:52:49 PM (4 days ago) Apr 7
to Lei Zhang, pdfium-...@googlegroups.com
Attention needed from Lei Zhang

April Kallmeyer voted Commit-Queue+1

Commit-Queue+1
Open in Gerrit

Related details

Attention is currently required from:
  • Lei Zhang
Submit Requirements:
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not 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: I26a79ce441aca58087345eff2823b05d43f2530b
Gerrit-Change-Number: 145690
Gerrit-PatchSet: 1
Gerrit-Owner: April Kallmeyer <a...@chromium.org>
Gerrit-Reviewer: April Kallmeyer <a...@chromium.org>
Gerrit-Reviewer: Lei Zhang <the...@chromium.org>
Gerrit-Attention: Lei Zhang <the...@chromium.org>
Gerrit-Comment-Date: Tue, 07 Apr 2026 20:52:46 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
unsatisfied_requirement
open
diffy

Lei Zhang (Gerrit)

unread,
Apr 7, 2026, 4:59:04 PM (4 days ago) Apr 7
to April Kallmeyer, Pdfium LUCI CQ, Lei Zhang, pdfium-...@googlegroups.com
Attention needed from April Kallmeyer

Lei Zhang added 3 comments

Commit Message
Line 7, Patchset 1 (Latest):Add a broken test for flattening AcroForms
Lei Zhang . unresolved

Maybe write "Add a test to demonstrate a PDF flattening bug"

And then write a brief CL description to mention it is intentionally broken.

Line 10, Patchset 1 (Latest):Bug: 498010830
Lei Zhang . unresolved

Flip with line 9.

File fpdfsdk/fpdf_flatten_embeddertest.cpp
Line 128, Patchset 1 (Latest):TEST_F(FPDFFlattenEmbedderTest, AcroForm) {
Lei Zhang . unresolved

Maybe "FlattenShouldRemoveAcroForm"

Open in Gerrit

Related details

Attention is currently required from:
  • April Kallmeyer
Submit Requirements:
    • requirement is not satisfiedCode-Owners
    • requirement is not satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement is not 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: I26a79ce441aca58087345eff2823b05d43f2530b
    Gerrit-Change-Number: 145690
    Gerrit-PatchSet: 1
    Gerrit-Owner: April Kallmeyer <a...@chromium.org>
    Gerrit-Reviewer: April Kallmeyer <a...@chromium.org>
    Gerrit-Reviewer: Lei Zhang <the...@chromium.org>
    Gerrit-Attention: April Kallmeyer <a...@chromium.org>
    Gerrit-Comment-Date: Tue, 07 Apr 2026 20:59:01 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    unsatisfied_requirement
    open
    diffy
    Reply all
    Reply to author
    Forward
    0 new messages