Correct return values in CPDF_HintTables::CheckPage()

Correct return values in CPDF_HintTables::CheckPage()

BUG=pdfium:329
R=thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/178086485892c9bb6aa0bf57ee449b33705e71c0

[jun_fang, 2015-12-18 04:06:25]

jun_fang@foxitsoftware.com changed reviewers:
+ thestig@chromium.org, tsepez@chromium.org

[jun_fang, 2015-12-18 04:19:05]

Description was changed from

==========
Correct the return value in CPDF_HintTables::CheckPage()

BUG=pdfium:329
==========

to

==========
Correct return values in CPDF_HintTables::CheckPage()

BUG=pdfium:329
==========

[Lei Zhang, 2015-12-18 07:59:05]

lgtm

https://codereview.chromium.org/1529173005/diff/20001/core/src/fpdfapi/fpdf_p...
File core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp (right):

https://codereview.chromium.org/1529173005/diff/20001/core/src/fpdfapi/fpdf_p...
core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp:4940: pHints))
Please add the braces back for multi-line if statements.

[jun_fang, 2015-12-18 08:05:20]

https://codereview.chromium.org/1529173005/diff/20001/core/src/fpdfapi/fpdf_p...
File core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp (right):

https://codereview.chromium.org/1529173005/diff/20001/core/src/fpdfapi/fpdf_p...
core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp:4940: pHints))
On 2015/12/18 07:59:05, Lei Zhang wrote:
> Please add the braces back for multi-line if statements.

Acknowledged.

[jun_fang, 2015-12-18 08:06:12]

Description was changed from

==========
Correct return values in CPDF_HintTables::CheckPage()

BUG=pdfium:329
==========

to

==========
Correct return values in CPDF_HintTables::CheckPage()

BUG=pdfium:329
R=thestig@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/178086485892c9bb6aa0bf57ee449b33705e...
==========

[jun_fang, 2015-12-18 08:06:16]

Committed patchset #3 (id:40001) manually as
178086485892c9bb6aa0bf57ee449b33705e71c0 (presubmit successful).