Remove a commented out DCHECK

Remove a commented out DCHECK

This DCHECK contradicts the DCHECK above it, so it's obsolete.

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

[falken, 2016-08-29 05:34:25]

Description was changed from

==========
rm dead code

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

to

==========
Remove a commented out DCHECK

This DCHECK contradicts the DCHECK above it, so it's obsolete.

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

[falken, 2016-08-29 05:34:37]

Description was changed from

==========
Remove a commented out DCHECK

This DCHECK contradicts the DCHECK above it, so it's obsolete.

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

to

==========
Remove a commented out DCHECK

This DCHECK contradicts the DCHECK above it, so it's obsolete.
==========

[falken, 2016-08-29 05:39:12]

falken@chromium.org changed reviewers:
+ mmenke@chromium.org

[falken, 2016-08-29 05:39:13]

It looks like the request_->NotifyReadCompleted() call at the end will set the
status.

[falken, 2016-08-29 05:46:21]

The referenced bug is internal b/1004233 which I'll close if this patch is ok.

[mmenke, 2016-08-29 14:45:23]

On 2016/08/29 05:46:21, falken wrote:
> The referenced bug is internal b/1004233 which I'll close if this patch is ok.

LGTM, yea, getting rid of this makes sense.  And note that it's now the opposite
of the DCHECK at the start of the method, as we've be trying to clean up
URLRequest/URLRequestJob's state machinery.

[mmenke, 2016-08-29 14:45:42]

On 2016/08/29 14:45:23, mmenke wrote:
> On 2016/08/29 05:46:21, falken wrote:
> > The referenced bug is internal b/1004233 which I'll close if this patch is
ok.
> 
> LGTM, yea, getting rid of this makes sense.  And note that it's now the
opposite
> of the DCHECK at the start of the method, as we've be trying to clean up
> URLRequest/URLRequestJob's state machinery.

Oops, you already noticed that, sorry, missed the description.

[falken, 2016-08-29 22:18:01]

The CQ bit was checked by falken@chromium.org

[commit-bot: I haz the power, 2016-08-29 22:57:05]

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-08-30 05:57:43]

Description was changed from

==========
Remove a commented out DCHECK

This DCHECK contradicts the DCHECK above it, so it's obsolete.
==========

to

==========
Remove a commented out DCHECK

This DCHECK contradicts the DCHECK above it, so it's obsolete.
==========

[commit-bot: I haz the power, 2016-08-30 05:57:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 05:59:04]

Description was changed from

==========
Remove a commented out DCHECK

This DCHECK contradicts the DCHECK above it, so it's obsolete.
==========

to

==========
Remove a commented out DCHECK

This DCHECK contradicts the DCHECK above it, so it's obsolete.

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

[commit-bot: I haz the power, 2016-08-30 05:59:05]

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