Cleanup some PWL_FontMap methods

Cleanup some PWL_FontMap methods

This CL makes some methods private which are only used internally, removes
unused methods and removes an unused class.

Committed: https://pdfium.googlesource.com/pdfium/+/0e3e890ba53ee7900ba44ebe50fecba21d086bfc

[dsinclair, 2016-08-24 16:02:29]

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

[commit-bot: I haz the power, 2016-08-24 16:02:32]

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

[dsinclair, 2016-08-24 16:02:44]

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

[dsinclair, 2016-08-24 16:02:44]

PTAL.

[commit-bot: I haz the power, 2016-08-24 16:12:41]

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

[commit-bot: I haz the power, 2016-08-24 16:12:42]

Dry run: This issue passed the CQ dry run.

[Wei Li, 2016-08-24 16:54:06]

https://codereview.chromium.org/2278583002/diff/1/fpdfsdk/pdfwindow/PWL_FontM...
File fpdfsdk/pdfwindow/PWL_FontMap.cpp (left):

https://codereview.chromium.org/2278583002/diff/1/fpdfsdk/pdfwindow/PWL_FontM...
fpdfsdk/pdfwindow/PWL_FontMap.cpp:279: if (m_pSystemHandler) {
No longer need the check? Ditto other places

https://codereview.chromium.org/2278583002/diff/1/fpdfsdk/pdfwindow/PWL_FontM...
File fpdfsdk/pdfwindow/PWL_FontMap.h (left):

https://codereview.chromium.org/2278583002/diff/1/fpdfsdk/pdfwindow/PWL_FontM...
fpdfsdk/pdfwindow/PWL_FontMap.h:120: CFX_SystemHandler* m_pSystemHandler;
now it is const?

[dsinclair, 2016-08-24 17:45:21]

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

[commit-bot: I haz the power, 2016-08-24 17:45:31]

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

[dsinclair, 2016-08-24 17:47:41]

https://codereview.chromium.org/2278583002/diff/1/fpdfsdk/pdfwindow/PWL_FontM...
File fpdfsdk/pdfwindow/PWL_FontMap.cpp (left):

https://codereview.chromium.org/2278583002/diff/1/fpdfsdk/pdfwindow/PWL_FontM...
fpdfsdk/pdfwindow/PWL_FontMap.cpp:279: if (m_pSystemHandler) {
On 2016/08/24 16:54:06, Wei Li wrote:
> No longer need the check? Ditto other places

Done.

https://codereview.chromium.org/2278583002/diff/1/fpdfsdk/pdfwindow/PWL_FontM...
File fpdfsdk/pdfwindow/PWL_FontMap.h (left):

https://codereview.chromium.org/2278583002/diff/1/fpdfsdk/pdfwindow/PWL_FontM...
fpdfsdk/pdfwindow/PWL_FontMap.h:120: CFX_SystemHandler* m_pSystemHandler;
On 2016/08/24 16:54:06, Wei Li wrote:
> now it is const?

Done.

[commit-bot: I haz the power, 2016-08-24 18:03:34]

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

[commit-bot: I haz the power, 2016-08-24 18:03:34]

Dry run: This issue passed the CQ dry run.

[Wei Li, 2016-08-24 18:37:00]

lgtm

[dsinclair, 2016-08-24 18:39:07]

The CQ bit was checked by dsinclair@chromium.org

[commit-bot: I haz the power, 2016-08-24 18:39:10]

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-24 18:39:26]

Description was changed from

==========
Cleanup some PWL_FontMap methods

This CL makes some methods private which are only used internally, removes
unused methods and removes an unused class.
==========

to

==========
Cleanup some PWL_FontMap methods

This CL makes some methods private which are only used internally, removes
unused methods and removes an unused class.

Committed:
https://pdfium.googlesource.com/pdfium/+/0e3e890ba53ee7900ba44ebe50fecba21d08...
==========

[commit-bot: I haz the power, 2016-08-24 18:39:27]

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