Revert of Change CHECK into DCHECK.

Revert of Change CHECK into DCHECK. (patchset #1 id:1 of https://codereview.chromium.org/2721393002/ )

Reason for revert:
(per discussion, I was using this to get a signal. I'll send a cl to fix)

Original issue's description:
> Change CHECK into DCHECK.
> 
> Code in the browser process should not be doing a CHECK based on data
> coming from the renderer process, as this allows a compromised renderer
> to trivially kill the whole browser.
> 
> BUG=688425
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2721393002
> Cr-Commit-Position: refs/heads/master@{#454047}
> Committed: https://chromium.googlesource.com/chromium/src/+/43f7ab4d121be7ad05f31728ccc130d500232031

TBR=creis@chromium.org,nasko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=688425
Review-Url: https://codereview.chromium.org/2729613003
Cr-Commit-Position: refs/heads/master@{#454058}
Committed: https://chromium.googlesource.com/chromium/src/+/494a3b618ecf62e96f14c195a0d2234b20db785c

[jam, 2017-03-01 22:03:20]

The CQ bit was checked by jam@chromium.org

[jam, 2017-03-01 22:03:21]

Created Revert of Change CHECK into DCHECK.

[commit-bot: I haz the power, 2017-03-01 22:04:06]

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-03-01 22:07:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488405800904500, "parent_rev":
"3944b9334aa1e6bd10f2627294f82d3e6b92c549", "commit_rev":
"494a3b618ecf62e96f14c195a0d2234b20db785c"}

[commit-bot: I haz the power, 2017-03-01 22:07:59]

Description was changed from

==========
Revert of Change CHECK into DCHECK. (patchset #1 id:1 of
https://codereview.chromium.org/2721393002/ )

Reason for revert:
(per discussion, I was using this to get a signal. I'll send a cl to fix)

Original issue's description:
> Change CHECK into DCHECK.
> 
> Code in the browser process should not be doing a CHECK based on data
> coming from the renderer process, as this allows a compromised renderer
> to trivially kill the whole browser.
> 
> BUG=688425
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2721393002
> Cr-Commit-Position: refs/heads/master@{#454047}
> Committed:
https://chromium.googlesource.com/chromium/src/+/43f7ab4d121be7ad05f31728ccc1...

TBR=creis@chromium.org,nasko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=688425
==========

to

==========
Revert of Change CHECK into DCHECK. (patchset #1 id:1 of
https://codereview.chromium.org/2721393002/ )

Reason for revert:
(per discussion, I was using this to get a signal. I'll send a cl to fix)

Original issue's description:
> Change CHECK into DCHECK.
> 
> Code in the browser process should not be doing a CHECK based on data
> coming from the renderer process, as this allows a compromised renderer
> to trivially kill the whole browser.
> 
> BUG=688425
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> 
> Review-Url: https://codereview.chromium.org/2721393002
> Cr-Commit-Position: refs/heads/master@{#454047}
> Committed:
https://chromium.googlesource.com/chromium/src/+/43f7ab4d121be7ad05f31728ccc1...

TBR=creis@chromium.org,nasko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=688425

Review-Url: https://codereview.chromium.org/2729613003
Cr-Commit-Position: refs/heads/master@{#454058}
Committed:
https://chromium.googlesource.com/chromium/src/+/494a3b618ecf62e96f14c195a0d2...
==========

[commit-bot: I haz the power, 2017-03-01 22:08:00]

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