Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(50)

Issue 2097523002: Load font maps in XFA. (Closed)

Created:
4 years, 6 months ago by dsinclair
Modified:
4 years, 6 months ago
Reviewers:
Lei Zhang, Tom Sepez, Wei Li
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Target Ref:
refs/heads/master
Project:
pdfium
Visibility:
Public.

Description

Load font maps in XFA. When XFA is enabled documents using some unicode fonts will render incorrectly as seen in the example attached to chromium:617490. This CL enables the loading of the font map files in XFA so we can correctly identify the character sets as unicode and map the glyphs correctly. BUG=chromium:617490 Committed: https://pdfium.googlesource.com/pdfium/+/ab5b60edba7e8e80d7ee964d58e58b384fcfe5ee

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -5 lines) Patch
M fpdfsdk/fpdfview.cpp View 1 chunk +4 lines, -5 lines 0 comments Download

Messages

Total messages: 8 (4 generated)
dsinclair
PTAL.
4 years, 6 months ago (2016-06-23 13:31:17 UTC) #3
Lei Zhang
lgtm
4 years, 6 months ago (2016-06-23 15:18:18 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2097523002/20001
4 years, 6 months ago (2016-06-23 15:18:25 UTC) #6
commit-bot: I haz the power
4 years, 6 months ago (2016-06-23 15:18:39 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/ab5b60edba7e8e80d7ee964d58e58b384fcf...

Powered by Google App Engine
This is Rietveld 408576698