chromium/src
0
Fork 0
Code Activity

Switch to FPDFPageObj_TransformF() in pdfium_searchify.cc

Browse Source 
Replace FPDFPageObj_Transform() calls with FPDFPageObj_TransformF(). Now
the matrix is passes as a single struct, rather than as 6 arguments.
Also CHECK() FPDFPageObj_TransformF() succeeds.

Bug: 352379279
Change-Id: I944b1fc7f6fd0a01ca544d268177ce668ea6bcbf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5698190
Reviewed-by: Chu-Hsuan Yang <chuhsuan@chromium.org>
Code-Coverage: findit-for-me@appspot.gserviceaccount.com <findit-for-me@appspot.gserviceaccount.com>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1329344}
This commit is contained in:
Lei Zhang
2024-07-18 06:40:46 +00:00
committed by Chromium LUCI CQ
parent b3ead75e2d
commit 08324d4bf6
1 changed files with 2 additions and 5 deletions

7
pdf/pdfium/pdfium_searchify.cc

@ -170,13 +170,10 @@ void AddWordOnImage(FPDF_DOCUMENT document,
const FS_MATRIX text_scale_matrix(
word->bounding_box.width() / text_object_size.width(), 0, 0,
word->bounding_box.height() / text_object_size.height(), 0, 0);
FPDFPageObj_Transform(text.get(), text_scale_matrix.a, text_scale_matrix.b,
text_scale_matrix.c, text_scale_matrix.d,
text_scale_matrix.e, text_scale_matrix.f);
CHECK(FPDFPageObj_TransformF(text.get(), &text_scale_matrix));
for (const auto& matrix : transform_matrices) {
FPDFPageObj_Transform(text.get(), matrix.a, matrix.b, matrix.c, matrix.d,
matrix.e, matrix.f);
FPDFPageObj_TransformF(text.get(), &matrix);
}
FPDFPage_InsertObject(page, text.release());