tree 282918c1b135e4d443faf428f74668f5e80cc34d
parent 2b2e1584ef6ddc352d829848a1433cdd036e965e
author Lei Zhang <thestig@chromium.org> 1600469944 +0000
committer Chromium commit bot <commit-bot@chromium.org> 1600469944 +0000

Fix FPDFText_GetCharBox() returning excessively tall char boxes.

In CPDF_TextPage::ProcessTextObject(), when the initial char box
calculation produces too small of a char box, the method compensates by
usng the font size for the height, and the glyph width for the width.
The height compensation code uses the font size in the wrong unit, so it
is 1000 times bigger than it needs to be. Fix this and update the
affected test case to show FPDFText_GetCharBox() no longer returns
excessively tall char boxes.

Also update a FPDFText_GetRect() test case that suffers from the same
calculation issue. Now, the second line of hello_world.pdf, which uses a
font of size 16, now has a rectangle height of ~15 instead of ~19.

Bug: pdfium:1591
Change-Id: I1641b882c11c2f06b8a610c9d1ae11c850cd5edf
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/73772
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Hui Yingst <nigi@chromium.org>