remove redundant DCHECK that a size_t variable >= 0

remove redundant DCHECK that a size_t variable >= 0

SampleVector::GetCountAtIndex has a DCHECK which checks that a size_t
variable is >= 0 and then some other condition.  But size_t is an
unsigned type so the first part of the check is useless (and it
gives an error because of this with clang).


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=190776

[Mostyn Bramley-Moore, 2013-03-25 21:29:37]

This was originally uploaded and reviewed here (uploaded again with a different
account since I couldn't figure out how to automatically move issues over):
https://codereview.chromium.org/13041004/

[commit-bot: I haz the power, 2013-03-26 03:41:21]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Ilya Sherman, 2013-03-26 04:37:06]

lgtm

[commit-bot: I haz the power, 2013-03-26 04:39:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/13006014/1

[commit-bot: I haz the power, 2013-03-26 05:46:56]

Retried try job too often on linux_rel for step(s) content_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-03-26 05:59:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/13006014/1

[commit-bot: I haz the power, 2013-03-26 07:10:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/13006014/1

[commit-bot: I haz the power, 2013-03-26 09:37:19]

Retried try job too often on linux_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[Mostyn Bramley-Moore, 2013-03-26 10:09:53]

On 2013/03/26 09:37:19, I haz the power (commit-bot) wrote:
> Retried try job too often on linux_rel for step(s) browser_tests
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

Looks like the window manager crashes in the test environment, which probably
causes the focus test to fail:

/usr/bin/python ../../../scripts/slave/runisolatedtest.py --test_name
browser_tests --builder_name linux_rel --tool_dir
/mnt/scratch0/b_used/build/slave/linux/build/src/out/../tools
/mnt/scratch0/b_used/build/slave/linux/build/src/out/Release/browser_tests --
/usr/bin/python
/mnt/scratch0/b_used/build/slave/linux/build/src/out/../tools/sharding_supervisor/sharding_supervisor.py
--no-color --retry-failed
/mnt/scratch0/b_used/build/slave/linux/build/src/out/Release/browser_tests
--lib=browser_tests --gtest_print_time
*** buffer overflow detected ***: icewm terminated

Who should I ping to look into this?

[commit-bot: I haz the power, 2013-03-26 10:20:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/13006014/1

[commit-bot: I haz the power, 2013-03-26 12:40:33]

Retried try job too often on linux_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-03-26 21:01:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/13006014/1

[commit-bot: I haz the power, 2013-03-26 22:02:03]

Change committed as 190776