Remove IFX_AdditionalFontMgr and IFX_FontMgrDelegate.

Remove IFX_AdditionalFontMgr and IFX_FontMgrDelegate.

Neither are implemented. Removed.

BUG=pdfium:468
Committed: https://pdfium.googlesource.com/pdfium/+/2d4a4fc372159ac7562abea48498b6ab72c2f321

[dsinclair, 2016-04-07 14:59:49]

dsinclair@chromium.org changed reviewers:
+ tsepez@chromium.org

[dsinclair, 2016-04-07 14:59:50]

PTAL.

https://codereview.chromium.org/1872463002/diff/1/xfa/fgas/font/fgas_font.h
File xfa/fgas/font/fgas_font.h (right):

https://codereview.chromium.org/1872463002/diff/1/xfa/fgas/font/fgas_font.h#n...
xfa/fgas/font/fgas_font.h:159: static IFX_FontMgr* Create(FX_LPEnumAllFonts
pEnumerator);
As far as I can see these Create() methods are only called with 1 param. Removed
the two extra params.

[dsinclair, 2016-04-07 20:41:41]

Description was changed from

==========
Remove IFX_AdditionalFontMgr and IFX_FontMgrDelegate.

Neither are implemented. Removed.

BUG=pdfium:468
==========

to

==========
Remove IFX_AdditionalFontMgr and IFX_FontMgrDelegate.

Neither are implemented. Removed.

BUG=pdfium:468
==========

[dsinclair, 2016-04-07 20:41:42]

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

[Tom Sepez, 2016-04-08 15:34:08]

lgtm

https://codereview.chromium.org/1872463002/diff/60001/xfa/fgas/font/fgas_font.h
File xfa/fgas/font/fgas_font.h (right):

https://codereview.chromium.org/1872463002/diff/60001/xfa/fgas/font/fgas_font...
xfa/fgas/font/fgas_font.h:134: typedef FX_FONTDESCRIPTOR* FX_LPFONTDESCRIPTOR;
nit: can we kill these two typedefs while we're at it, replacing their usage
with FX_FONTDESCRIPTOR* etc as needed?

[dsinclair, 2016-04-11 13:19:38]

https://codereview.chromium.org/1872463002/diff/60001/xfa/fgas/font/fgas_font.h
File xfa/fgas/font/fgas_font.h (right):

https://codereview.chromium.org/1872463002/diff/60001/xfa/fgas/font/fgas_font...
xfa/fgas/font/fgas_font.h:134: typedef FX_FONTDESCRIPTOR* FX_LPFONTDESCRIPTOR;
On 2016/04/08 15:34:07, Tom Sepez wrote:
> nit: can we kill these two typedefs while we're at it, replacing their usage
> with FX_FONTDESCRIPTOR* etc as needed?

Done.

[dsinclair, 2016-04-11 13:19:43]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2016-04-11 13:19:43]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org
Link to the patchset: https://codereview.chromium.org/1872463002/#ps80001
(title: " ")

[commit-bot: I haz the power, 2016-04-11 13:19:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1872463002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1872463002/80001

[commit-bot: I haz the power, 2016-04-11 13:35:34]

Description was changed from

==========
Remove IFX_AdditionalFontMgr and IFX_FontMgrDelegate.

Neither are implemented. Removed.

BUG=pdfium:468
==========

to

==========
Remove IFX_AdditionalFontMgr and IFX_FontMgrDelegate.

Neither are implemented. Removed.

BUG=pdfium:468

Committed:
https://pdfium.googlesource.com/pdfium/+/2d4a4fc372159ac7562abea48498b6ab72c2...
==========

[commit-bot: I haz the power, 2016-04-11 13:35:35]

Committed patchset #5 (id:80001) as
https://pdfium.googlesource.com/pdfium/+/2d4a4fc372159ac7562abea48498b6ab72c2...