Remove redundant DCHECKs

Remove redundant DCHECKs

BUG=

Review-Url: https://codereview.chromium.org/2768923004
Cr-Commit-Position: refs/heads/master@{#459659}
Committed: https://chromium.googlesource.com/chromium/src/+/c654a79ce1533215e2dcad41d5e623d8220d2dd9

[matthewtff.asm, 2017-03-24 16:09:44]

Description was changed from

==========
Remove redundant DCHECKs

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

to

==========
Remove redundant DCHECKs

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

[matthewtff.asm, 2017-03-24 16:09:44]

matthewtff@yandex-team.ru changed reviewers:
+ haraken@chromium.org, maxbogue@chromium.org, zhongyi@chromium.org

[matthewtff.asm, 2017-03-24 16:11:47]

On 2017/03/24 16:09:44, matthewtff.asm wrote:
> mailto:matthewtff@yandex-team.ru changed reviewers:
> + mailto:haraken@chromium.org, mailto:maxbogue@chromium.org,
mailto:zhongyi@chromium.org

These DCHECKS had no sense as compared unsigned number agains zero.
I've tries to fix/remove them as clang hits [-Werror,-Wtautological-compare] if
source code for these translation units was first preprocessed and then
compiled.
This could be an issue for distributed build systems.
Take a look, please.

[haraken, 2017-03-24 16:18:05]

WebKit LGTM

[maxbogue, 2017-03-24 17:40:55]

components/sync lgtm

[Zhongyi Shi, 2017-03-24 23:30:58]

net/ lgtm

[matthewtff.asm, 2017-03-25 12:55:49]

The CQ bit was checked by matthewtff@yandex-team.ru

[commit-bot: I haz the power, 2017-03-25 12:56:02]

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-25 15:07:34]

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

[commit-bot: I haz the power, 2017-03-25 15:07:36]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[matthewtff.asm, 2017-03-25 21:58:14]

The CQ bit was checked by matthewtff@yandex-team.ru

[commit-bot: I haz the power, 2017-03-25 21:58: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, 2017-03-25 22:29:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490479094308120, "parent_rev":
"ce1f5b6ed9f4ceb9170ecb79b178a1b03055b34b", "commit_rev":
"c654a79ce1533215e2dcad41d5e623d8220d2dd9"}

[commit-bot: I haz the power, 2017-03-25 22:29:51]

Description was changed from

==========
Remove redundant DCHECKs

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

to

==========
Remove redundant DCHECKs

BUG=

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

[commit-bot: I haz the power, 2017-03-25 22:29:52]

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