Fix font rendering issues because of an incorrect transform.
authorJohn Abd-El-Malek <jam@chromium.org>
Wed, 10 Jun 2015 22:44:42 +0000 (15:44 -0700)
committerJohn Abd-El-Malek <jam@chromium.org>
Wed, 10 Jun 2015 22:44:42 +0000 (15:44 -0700)
commite4503ea9947d2f9c61704da20271b413a364a9c0
tree717335d094e1b35de7496d52d9ab17883e7bad9d
parenta8a718c974210a919f2af0b099d540143cf330c6
Fix font rendering issues because of an incorrect transform.

This regressed in https://pdfium.googlesource.com/pdfium/+/71c24b839498fb89184002ed30fcff353e1e402c. The code would reach into FreeType internals and reset transform_flags. This would effectively set the font's transform matrix to the identity (since a transform is only used if the flag is set). I removed it because I assumed this is only a cache, and any other place that would call FT_Load_Glyph would have set a transform first. Apparently that's not the case (verified through adding some additional code). The fix is to reset the transform matrix after changing it. This is functionally equivalent to the previous behavior, since if the flag was 0 but there was a transform, it would be ignored until another transform is set.

BUG=479434
R=tsepez@chromium.org

Review URL: https://codereview.chromium.org/1163423005.
core/src/fxge/ge/fx_ge_text.cpp