Add some CHECKs to PepperPDFHost to figure out a crash.

Add some CHECKs to PepperPDFHost to figure out a crash.

BUG=627814
Committed: https://crrev.com/a0cec3faf14da6bd8f78b8bfaa6147af8926a3e8
Cr-Commit-Position: refs/heads/master@{#405338}

[Lei Zhang, 2016-07-13 21:10:07]

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

[Lei Zhang, 2016-07-13 21:27:13]

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

[commit-bot: I haz the power, 2016-07-13 21:29:06]

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

[tommycli, 2016-07-13 21:51:02]

LGTM sans:

https://codereview.chromium.org/2148063002/diff/20001/components/pdf/renderer...
File components/pdf/renderer/pepper_pdf_host.cc (right):

https://codereview.chromium.org/2148063002/diff/20001/components/pdf/renderer...
components/pdf/renderer/pepper_pdf_host.cc:145: // TODO(thestig): Turn CHECKs
into the proper if statement after figuring ut
nit: "figuring ut" typo

[Lei Zhang, 2016-07-13 21:52:45]

https://codereview.chromium.org/2148063002/diff/20001/components/pdf/renderer...
File components/pdf/renderer/pepper_pdf_host.cc (right):

https://codereview.chromium.org/2148063002/diff/20001/components/pdf/renderer...
components/pdf/renderer/pepper_pdf_host.cc:145: // TODO(thestig): Turn CHECKs
into the proper if statement after figuring ut
On 2016/07/13 21:51:02, tommycli wrote:
> nit: "figuring ut" typo

Clearly a case of packet loss between chair and keyboard.

[Lei Zhang, 2016-07-13 21:52:55]

The CQ bit was checked by thestig@chromium.org

[Lei Zhang, 2016-07-13 21:52:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
tommycli@chromium.org
Link to the patchset: https://codereview.chromium.org/2148063002/#ps40001
(title: "typo")

[commit-bot: I haz the power, 2016-07-13 21:53:51]

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-07-13 23:06:43]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-13 23:10:01]

Description was changed from

==========
Add some CHECKs to PepperPDFHost to figure out a crash.

BUG=627814
==========

to

==========
Add some CHECKs to PepperPDFHost to figure out a crash.

BUG=627814

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

[commit-bot: I haz the power, 2016-07-13 23:10:01]

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