Potentially do less work in PDFiumEngine::CreateSinglePageRasterPdf().

Potentially do less work in PDFiumEngine::CreateSinglePageRasterPdf().

In r479694, the method started always doing some calculations even
though the results will only be used under certain conditions.

Review-Url: https://codereview.chromium.org/2947533002
Cr-Commit-Position: refs/heads/master@{#480539}
Committed: https://chromium.googlesource.com/chromium/src/+/59c41df5c64b6bc8641412eb5c3b3d0681d9c8b5

[Lei Zhang, 2017-06-17 03:03:41]

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

[commit-bot: I haz the power, 2017-06-17 03:03:54]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Lei Zhang, 2017-06-17 03:04:34]

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

[commit-bot: I haz the power, 2017-06-17 03:05:22]

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

[commit-bot: I haz the power, 2017-06-17 03:05:23]

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[msarett1, 2017-06-19 11:59:07]

lgtm

[Lei Zhang, 2017-06-19 17:54:04]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2017-06-19 17:54:35]

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, 2017-06-19 19:41:47]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497894844588110, "parent_rev":
"077532a552de32b5062002674884b550325486ea", "commit_rev":
"59c41df5c64b6bc8641412eb5c3b3d0681d9c8b5"}

[commit-bot: I haz the power, 2017-06-19 19:42:00]

Description was changed from

==========
Potentially do less work in PDFiumEngine::CreateSinglePageRasterPdf().

In r479694, the method started always doing some calculations even
though the results will only be used under certain conditions.
==========

to

==========
Potentially do less work in PDFiumEngine::CreateSinglePageRasterPdf().

In r479694, the method started always doing some calculations even
though the results will only be used under certain conditions.

Review-Url: https://codereview.chromium.org/2947533002
Cr-Commit-Position: refs/heads/master@{#480539}
Committed:
https://chromium.googlesource.com/chromium/src/+/59c41df5c64b6bc8641412eb5c3b...
==========

[commit-bot: I haz the power, 2017-06-19 19:42:01]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/59c41df5c64b6bc8641412eb5c3b...