commit e116b67b139c89203d2d69512a041026fcb366c2
author Lei Zhang <thestig@chromium.org> Fri Jan 10 19:11:46 2025 -0800
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Jan 10 19:11:46 2025 -0800
tree 64a652100801c830fdf35b9ca955e1fa7bc4c5b6
parent 67a00b1671226395883438489110db981b28d6b5

Fix bad refactoring in CXFA_TextParser::GetFont()

CL [1] made a mistake and replaced an italic font style constant with
the bold font style. Undo that so GetFont() makes more sense.

[1] https://pdfium-review.googlesource.com/c/pdfium/+/15071/5/xfa/fxfa/cxfa_textparser.cpp

Change-Id: I5aac4358bcf5fe6951a76496fd67e08454858d98
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/127030
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: dan sinclair <dsinclair@chromium.org>
Reviewed-by: Thomas Sepez <tsepez@google.com>

xfa/fxfa/cxfa_textparser.cpp

diff --git a/xfa/fxfa/cxfa_textparser.cpp b/xfa/fxfa/cxfa_textparser.cpp
index 4ab7d3b..363ac63 100644
--- a/xfa/fxfa/cxfa_textparser.cpp
+++ b/xfa/fxfa/cxfa_textparser.cpp
@@ -348,7 +348,7 @@
       dwStyle |= pdfium::kFontStyleForceBold;
     }
     if (font->IsItalic()) {
-      dwStyle |= pdfium::kFontStyleForceBold;
+      dwStyle |= pdfium::kFontStyleItalic;
     }
   }