Delete m_ExtHandle and RetainFont

Delete m_ExtHandle and RetainFont because they are not being used.

CFX_SubstFont::m_ExtHandle is never assigned a value different from nullptr.
Similarly, IFX_SystemFontInfo::RetainFont has two implementations.
Both of these return nullptr, so this method is not needed.

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

[npm, 2016-08-16 21:10:35]

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

[commit-bot: I haz the power, 2016-08-16 21:10:40]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-16 22:13:20]

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

[commit-bot: I haz the power, 2016-08-16 22:13:22]

Dry run: This issue passed the CQ dry run.

[npm, 2016-08-16 22:20:10]

Description was changed from

==========
Delete m_ExtHandle and RetainFont

BUG=
==========

to

==========
Delete m_ExtHandle and RetainFont because they are not being used.

CFX_SubstFont::m_ExtHandle is never assigned a value different from nullptr.
Similarly, IFX_SystemFontInfo::RetainFont has two implementations.
Both of these return nullptr, so this method is not needed.
==========

[npm, 2016-08-16 22:20:11]

npm@chromium.org changed reviewers:
+ dsinclair@chromium.org, thestig@chromium.org, tsepez@chromium.org,
weili@chromium.org

[npm, 2016-08-16 22:20:56]

PTAL

[Tom Sepez, 2016-08-16 22:27:48]

lgtm

[npm, 2016-08-16 22:36:59]

The CQ bit was checked by npm@chromium.org

[commit-bot: I haz the power, 2016-08-16 22:37:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-16 22:37:23]

Description was changed from

==========
Delete m_ExtHandle and RetainFont because they are not being used.

CFX_SubstFont::m_ExtHandle is never assigned a value different from nullptr.
Similarly, IFX_SystemFontInfo::RetainFont has two implementations.
Both of these return nullptr, so this method is not needed.
==========

to

==========
Delete m_ExtHandle and RetainFont because they are not being used.

CFX_SubstFont::m_ExtHandle is never assigned a value different from nullptr.
Similarly, IFX_SystemFontInfo::RetainFont has two implementations.
Both of these return nullptr, so this method is not needed.

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

[commit-bot: I haz the power, 2016-08-16 22:37:24]

Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/b88b8e47fd7adc14c7aff048c0def0579bdd...