Completely de-virtualize CPDF_SyntaxParser.

Completely de-virtualize CPDF_SyntaxParser.

It isn't sub-classed.  Making GetNextChar() non-virtual
may save a virtual fn call per character if the compilers
aren't able to do so themselves.

Make members private, not protected.

BUG=
R=ochang@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/9f7b5e1b5e0dfd7310a2eb9e586bc998f8ea11fa

[Tom Sepez, 2016-03-07 22:45:01]

Description was changed from

==========
Completely de-virtualize CPDF_SyntaxParser.

It isn't sub-classed.  Making GetNextChar() non-virtual
may save a virtual fn call per character if the compilers
aren't able to do so themselves.

Make members private, not protected.

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

to

==========
Completely de-virtualize CPDF_SyntaxParser.

It isn't sub-classed.  Making GetNextChar() non-virtual
may save a virtual fn call per character if the compilers
aren't able to do so themselves.

Make members private, not protected.

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

[Tom Sepez, 2016-03-07 22:45:01]

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

[Tom Sepez, 2016-03-07 22:45:13]

Oliver, for review.

[Oliver Chang, 2016-03-07 22:52:29]

lgtm

[Tom Sepez, 2016-03-07 22:55:05]

Description was changed from

==========
Completely de-virtualize CPDF_SyntaxParser.

It isn't sub-classed.  Making GetNextChar() non-virtual
may save a virtual fn call per character if the compilers
aren't able to do so themselves.

Make members private, not protected.

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

to

==========
Completely de-virtualize CPDF_SyntaxParser.

It isn't sub-classed.  Making GetNextChar() non-virtual
may save a virtual fn call per character if the compilers
aren't able to do so themselves.

Make members private, not protected.

BUG=
R=ochang@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/9f7b5e1b5e0dfd7310a2eb9e586bc998f8ea...
==========

[Tom Sepez, 2016-03-07 22:55:06]

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