Fix clang_use_chrome_plugin for Mac and Skia builds

Fix clang_use_chrome_plugin for Mac and Skia builds

These changes are specific to Mac and Skia builds. They are
needed for these builds to compile with clang_use_chrome_plugin.

BUG=pdfium:469
Committed: https://pdfium.googlesource.com/pdfium/+/4701fcddeddfef7bccbceaf0f439dee82c3639f0

[Wei Li, 2016-06-18 23:39:05]

Description was changed from

==========
Enable clang_use_chrome_plugin on Mac and Win

These are the file changes specific to Mac and Win platforms. These
changes are needed to for compiling with clang_use_chrome_plugin
flag on these platform.

BUG=pdfium:469
==========

to

==========
Fix clang_use_chrome_plugin for Mac and Skia builds

These changes are specific to Mac and Skia builds. They are
needed for these builds to compile with clang_use_chrome_plugin.

BUG=pdfium:469
==========

[Wei Li, 2016-06-20 16:57:52]

The CQ bit was checked by weili@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-20 16:57:57]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2081523002/20001

[commit-bot: I haz the power, 2016-06-20 17:14:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-20 17:14:22]

Dry run: This issue passed the CQ dry run.

[Wei Li, 2016-06-20 17:26:52]

weili@chromium.org changed reviewers:
+ thestig@chromium.org

[Wei Li, 2016-06-20 17:26:53]

for review, thanks

[Lei Zhang, 2016-06-20 17:45:02]

https://codereview.chromium.org/2081523002/diff/20001/xfa/fgas/font/fgas_stdf...
File xfa/fgas/font/fgas_stdfontmgr.cpp (right):

https://codereview.chromium.org/2081523002/diff/20001/xfa/fgas/font/fgas_stdf...
xfa/fgas/font/fgas_stdfontmgr.cpp:477: : m_nFaceIndex(0), m_dwFontStyles(0),
m_dwUsb(), m_dwCsb() {}
Does this actually initialize |m_dwUsb| and |m_dwCsb| ?

[Wei Li, 2016-06-20 18:04:10]

https://codereview.chromium.org/2081523002/diff/20001/xfa/fgas/font/fgas_stdf...
File xfa/fgas/font/fgas_stdfontmgr.cpp (right):

https://codereview.chromium.org/2081523002/diff/20001/xfa/fgas/font/fgas_stdf...
xfa/fgas/font/fgas_stdfontmgr.cpp:477: : m_nFaceIndex(0), m_dwFontStyles(0),
m_dwUsb(), m_dwCsb() {}
On 2016/06/20 17:45:02, Lei Zhang wrote:
> Does this actually initialize |m_dwUsb| and |m_dwCsb| ?

Yes, this will zero initialize the vars, same as ={}

[Lei Zhang, 2016-06-20 18:20:25]

lgtm

[Wei Li, 2016-06-20 18:41:33]

The CQ bit was checked by weili@chromium.org

[commit-bot: I haz the power, 2016-06-20 18:41:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2081523002/20001

[commit-bot: I haz the power, 2016-06-20 18:42:01]

Description was changed from

==========
Fix clang_use_chrome_plugin for Mac and Skia builds

These changes are specific to Mac and Skia builds. They are
needed for these builds to compile with clang_use_chrome_plugin.

BUG=pdfium:469
==========

to

==========
Fix clang_use_chrome_plugin for Mac and Skia builds

These changes are specific to Mac and Skia builds. They are
needed for these builds to compile with clang_use_chrome_plugin.

BUG=pdfium:469

Committed:
https://pdfium.googlesource.com/pdfium/+/4701fcddeddfef7bccbceaf0f439dee82c36...
==========

[commit-bot: I haz the power, 2016-06-20 18:42:02]

Committed patchset #2 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/4701fcddeddfef7bccbceaf0f439dee82c36...