Use const variable instead of using a macro to store a constant.

Use const variables instead of using macros to store constants.

Committed: https://crrev.com/d83f822fafbf12f309e3ca04c7275e3ecd4aa456
Cr-Commit-Position: refs/heads/master@{#402324}

[jaepark, 2016-06-27 21:21:18]

jaepark@google.com changed reviewers:
+ thestig@chromium.org

[jaepark, 2016-06-27 21:23:02]

The CQ bit was checked by jaepark@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-06-27 21:23:46]

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

[Lei Zhang, 2016-06-27 21:28:07]

Please fix the grammar in the CL description.

https://codereview.chromium.org/2103573002/diff/1/pdf/pdfium/pdfium_engine.cc
File pdf/pdfium/pdfium_engine.cc (right):

https://codereview.chromium.org/2103573002/diff/1/pdf/pdfium/pdfium_engine.cc...
pdf/pdfium/pdfium_engine.cc:98: const int32_t kMaxProgressivePaintTimeMs = 300;
This is a test comment.

https://codereview.chromium.org/2103573002/diff/1/pdf/pdfium/pdfium_engine.cc...
pdf/pdfium/pdfium_engine.cc:111: const int32_t kMaxInitialProgressivePaintTimeMs
= 250;
This is another test comment.

[jaepark, 2016-06-27 21:31:26]

Description was changed from

==========
Use const variable instead of using a macro to store a constant.

Using a const variable is preferable to using a macro.

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

to

==========
Use const variables instead of using macros to store constants.
==========

[jaepark, 2016-06-27 21:31:36]

https://codereview.chromium.org/2103573002/diff/1/pdf/pdfium/pdfium_engine.cc
File pdf/pdfium/pdfium_engine.cc (right):

https://codereview.chromium.org/2103573002/diff/1/pdf/pdfium/pdfium_engine.cc...
pdf/pdfium/pdfium_engine.cc:98: const int32_t kMaxProgressivePaintTimeMs = 300;
On 2016/06/27 21:28:07, Lei Zhang wrote:
> This is a test comment.

Done.

https://codereview.chromium.org/2103573002/diff/1/pdf/pdfium/pdfium_engine.cc...
pdf/pdfium/pdfium_engine.cc:111: const int32_t kMaxInitialProgressivePaintTimeMs
= 250;
On 2016/06/27 21:28:07, Lei Zhang wrote:
> This is another test comment.

This is a test reply.

[Lei Zhang, 2016-06-27 21:31:59]

lgtm

[commit-bot: I haz the power, 2016-06-27 22:49:32]

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

[commit-bot: I haz the power, 2016-06-27 22:49:33]

Dry run: This issue passed the CQ dry run.

[jaepark, 2016-06-27 22:51:33]

The CQ bit was checked by jaepark@google.com

[commit-bot: I haz the power, 2016-06-27 22:53:28]

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-06-27 23:05:06]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-27 23:06:39]

Description was changed from

==========
Use const variables instead of using macros to store constants.
==========

to

==========
Use const variables instead of using macros to store constants.

Committed: https://crrev.com/d83f822fafbf12f309e3ca04c7275e3ecd4aa456
Cr-Commit-Position: refs/heads/master@{#402324}
==========

[commit-bot: I haz the power, 2016-06-27 23:06:40]

Patchset 1 (id:??) landed as
https://crrev.com/d83f822fafbf12f309e3ca04c7275e3ecd4aa456
Cr-Commit-Position: refs/heads/master@{#402324}