Print Preview: Return or crash on failures

Print Preview: Return or crash on failures

Ensure we always return after rejecting JS callback or CHECK if the
condition should always be met.

BUG=717296
Review-Url: https://codereview.chromium.org/2980603002
Cr-Commit-Position: refs/heads/master@{#485707}
Committed: https://chromium.googlesource.com/chromium/src/+/cae285de3cfae6a0b24adefcdbb56d5b67b46739

[rbpotter, 2017-07-11 16:33:45]

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

[commit-bot: I haz the power, 2017-07-11 16:33:55]

Dry run: 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-07-11 17:41:31]

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

[commit-bot: I haz the power, 2017-07-11 17:41:32]

Dry run: This issue passed the CQ dry run.

[rbpotter, 2017-07-11 20:31:09]

Description was changed from

==========
Print Preview: Return or crash on failures

Ensure we always return after rejecting JS callback or CHECK if the
condition should always be met.

BUG=717296
==========

to

==========
Print Preview: Return or crash on failures

Ensure we always return after rejecting JS callback or CHECK if the
condition should always be met.

BUG=717296
==========

[rbpotter, 2017-07-11 20:31:09]

rbpotter@chromium.org changed reviewers:
+ dpapad@chromium.org, thestig@chromium.org

[dpapad, 2017-07-11 21:19:09]

lgtm

[Lei Zhang, 2017-07-11 21:23:01]

lgtm

[rbpotter, 2017-07-11 21:29:01]

The CQ bit was checked by rbpotter@chromium.org

[commit-bot: I haz the power, 2017-07-11 21:29:30]

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-07-11 22:33:38]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499808541464380, "parent_rev":
"731db9d2f7dca2fe98df656ed9b139e494409793", "commit_rev":
"0d35e3c58a3c2c83dffb858b9103b12c0b303b0c"}

[commit-bot: I haz the power, 2017-07-11 22:33:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499808541464380, "parent_rev":
"9a45f2c60ef1c80dde586d0864e509bfef88d7ff", "commit_rev":
"cae285de3cfae6a0b24adefcdbb56d5b67b46739"}

[commit-bot: I haz the power, 2017-07-11 22:34:10]

Description was changed from

==========
Print Preview: Return or crash on failures

Ensure we always return after rejecting JS callback or CHECK if the
condition should always be met.

BUG=717296
==========

to

==========
Print Preview: Return or crash on failures

Ensure we always return after rejecting JS callback or CHECK if the
condition should always be met.

BUG=717296

Review-Url: https://codereview.chromium.org/2980603002
Cr-Commit-Position: refs/heads/master@{#485707}
Committed:
https://chromium.googlesource.com/chromium/src/+/cae285de3cfae6a0b24adefcdbb5...
==========

[commit-bot: I haz the power, 2017-07-11 22:34:11]

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