Fix an assertion failure in CXFA_FWLTheme::Initialize()

Fix an assertion failure in CXFA_FWLTheme::Initialize()

The face names from font file stream may be different
with the ones from chromium. Pdfium should save face
names from chromium for font mapping when it works
in sandbox environment.

BUG=pdfium:392
BUG=pdfium:393
R=tsepez@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/c5967f9dbaee98e8ac8302a62e015e5c595aa6e0

[jun_fang, 2016-02-18 08:02:14]

Description was changed from

==========
Fix an assertion failure in CXFA_FWLTheme::Initialize()

BUG=pdfium:392,pdfium:393
==========

to

==========
Fix an assertion failure in CXFA_FWLTheme::Initialize()

The face names from font file stream may be different
with the ones from chromium. Pdfium should save face 
names from chromium for font mapping when it works 
in sandbox environment.

BUG=pdfium:392
BUG=pdfium:393
==========

[jun_fang, 2016-02-18 08:02:14]

jun_fang@foxitsoftware.com changed reviewers:
+ thestig@chromium.org, tsepez@chromium.org

[jun_fang, 2016-02-18 09:38:10]

Hi Tom and Lei,

Please help to review this CL. Thanks!

[Tom Sepez, 2016-02-18 19:46:54]

LGTM with nit.

https://codereview.chromium.org/1710753002/diff/1/xfa/src/fgas/src/font/fx_st...
File xfa/src/fgas/src/font/fx_stdfontmgr.cpp (right):

https://codereview.chromium.org/1710753002/diff/1/xfa/src/fgas/src/font/fx_st...
xfa/src/fgas/src/font/fx_stdfontmgr.cpp:622: CFX_WideString wsFaceName;
nit: can we delcare this at line 628 at the same time we assign it?

[jun_fang, 2016-02-19 00:14:04]

https://codereview.chromium.org/1710753002/diff/1/xfa/src/fgas/src/font/fx_st...
File xfa/src/fgas/src/font/fx_stdfontmgr.cpp (right):

https://codereview.chromium.org/1710753002/diff/1/xfa/src/fgas/src/font/fx_st...
xfa/src/fgas/src/font/fx_stdfontmgr.cpp:622: CFX_WideString wsFaceName;
On 2016/02/18 19:46:54, Tom Sepez wrote:
> nit: can we delcare this at line 628 at the same time we assign it?

Acknowledged.

[jun_fang, 2016-02-19 00:40:50]

Description was changed from

==========
Fix an assertion failure in CXFA_FWLTheme::Initialize()

The face names from font file stream may be different
with the ones from chromium. Pdfium should save face 
names from chromium for font mapping when it works 
in sandbox environment.

BUG=pdfium:392
BUG=pdfium:393
==========

to

==========
Fix an assertion failure in CXFA_FWLTheme::Initialize()

The face names from font file stream may be different
with the ones from chromium. Pdfium should save face
names from chromium for font mapping when it works
in sandbox environment.

BUG=pdfium:392
BUG=pdfium:393
R=tsepez@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/c5967f9dbaee98e8ac8302a62e015e5c595a...
==========

[jun_fang, 2016-02-19 00:40:54]

Committed patchset #2 (id:20001) manually as
c5967f9dbaee98e8ac8302a62e015e5c595aa6e0 (presubmit successful).