Cleanup includes, comments, and unused code

Cleanup includes, comments, and unused code

R=thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/6664977832350e4a9717bd67b09e3d01196791ba

[Wei Li, 2016-01-05 23:59:02]

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

[Wei Li, 2016-01-05 23:59:03]

PTAL, thanks.

[Lei Zhang, 2016-01-06 00:05:34]

https://codereview.chromium.org/1563673002/diff/1/core/src/fpdfapi/fpdf_parse...
File core/src/fpdfapi/fpdf_parser/fpdf_parser_encrypt.cpp (left):

https://codereview.chromium.org/1563673002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/fpdf_parser_encrypt.cpp:7: #include
"core/include/fpdfapi/fpdf_parser.h"
I moved this up here because it's the "related header" that defines
CPDF_StandardCryptoHandler.

https://codereview.chromium.org/1563673002/diff/1/core/src/fpdfapi/fpdf_parse...
File core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp (left):

https://codereview.chromium.org/1563673002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp:7: #include "parser_int.h"
This one was wrong. I should have put core/include/fpdfapi/fpdf_parser.h here.

[Wei Li, 2016-01-06 04:46:24]

https://codereview.chromium.org/1563673002/diff/1/core/src/fpdfapi/fpdf_parse...
File core/src/fpdfapi/fpdf_parser/fpdf_parser_encrypt.cpp (left):

https://codereview.chromium.org/1563673002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/fpdf_parser_encrypt.cpp:7: #include
"core/include/fpdfapi/fpdf_parser.h"
On 2016/01/06 00:05:33, Lei Zhang wrote:
> I moved this up here because it's the "related header" that defines
> CPDF_StandardCryptoHandler.

Got it. I apply this to other files as well.

https://codereview.chromium.org/1563673002/diff/1/core/src/fpdfapi/fpdf_parse...
File core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp (left):

https://codereview.chromium.org/1563673002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp:7: #include "parser_int.h"
On 2016/01/06 00:05:33, Lei Zhang wrote:
> This one was wrong. I should have put core/include/fpdfapi/fpdf_parser.h here.

Done.

[Lei Zhang, 2016-01-06 09:41:40]

lgtm

[Wei Li, 2016-01-07 18:30:37]

Description was changed from

==========
Cleanup includes, comments, and unused code
==========

to

==========
Cleanup includes, comments, and unused code

R=thestig@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/6664977832350e4a9717bd67b09e3d011967...
==========

[Wei Li, 2016-01-07 18:30:38]

Committed patchset #2 (id:20001) manually as
6664977832350e4a9717bd67b09e3d01196791ba (presubmit successful).