Removing dcheck in SetUploadConsentImpl.

Removing dcheck in SetUploadConsentImpl.

This DCHECK is very easy to trigger by developers, googlers specifically, without actually violating the intent of the DCHECK.

BUG=648594
Committed: https://crrev.com/6bbba65a7074b6cdbe1dada8d51d0ba90ff944b6
Cr-Commit-Position: refs/heads/master@{#419882}

[jwd, 2016-09-20 21:51:18]

jwd@chromium.org changed reviewers:
+ scottmg@chromium.org

[jwd, 2016-09-20 21:54:16]

Description was changed from

==========
Removing dcheck in SetUploadConsentImpl.

BUG=648594
==========

to

==========
Removing dcheck in SetUploadConsentImpl.

This DCHECK is very easy to trigger by developers, googlers specifically,
without actually violating the intent of the DCHECK.

BUG=648594
==========

[scottmg, 2016-09-20 22:09:05]

If you're confident in the bug's analysis, lgtm.

[jwd, 2016-09-20 22:18:35]

On 2016/09/20 22:09:05, scottmg wrote:
> If you're confident in the bug's analysis, lgtm.

For the record, I am confident in the bug's analysis.

[jwd, 2016-09-20 22:18:46]

The CQ bit was checked by jwd@chromium.org

[commit-bot: I haz the power, 2016-09-20 22:19:22]

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-09-20 22:57:30]

Description was changed from

==========
Removing dcheck in SetUploadConsentImpl.

This DCHECK is very easy to trigger by developers, googlers specifically,
without actually violating the intent of the DCHECK.

BUG=648594
==========

to

==========
Removing dcheck in SetUploadConsentImpl.

This DCHECK is very easy to trigger by developers, googlers specifically,
without actually violating the intent of the DCHECK.

BUG=648594
==========

[commit-bot: I haz the power, 2016-09-20 22:57:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-20 23:00:51]

Description was changed from

==========
Removing dcheck in SetUploadConsentImpl.

This DCHECK is very easy to trigger by developers, googlers specifically,
without actually violating the intent of the DCHECK.

BUG=648594
==========

to

==========
Removing dcheck in SetUploadConsentImpl.

This DCHECK is very easy to trigger by developers, googlers specifically,
without actually violating the intent of the DCHECK.

BUG=648594

Committed: https://crrev.com/6bbba65a7074b6cdbe1dada8d51d0ba90ff944b6
Cr-Commit-Position: refs/heads/master@{#419882}
==========

[commit-bot: I haz the power, 2016-09-20 23:00:52]

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