Remove trivially-true check in parsed_cookie.cc

Remove trivially-true check in parsed_cookie.cc

IsControlCharacter() was checking an unsigned char to
make sure it's >= 0, which is trivially true. This throws
a warning on the default CrOS toolchain, so remove it even though it's harmless

BUG=424334
TEST=net_unittests
R=rsleevi@chromium.org

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

[Chris Masone, 2014-10-17 18:07:08]

cmasone@chromium.org changed reviewers:
+ jww@chromium.org

[Chris Masone, 2014-10-17 18:07:09]

Joel landed this code, but Ryan can you OWNERS review?

[jww, 2014-10-17 18:08:54]

lgtm

[Ryan Sleevi, 2014-10-17 19:17:10]

lgtm % nit

https://codereview.chromium.org/640303008/diff/1/net/cookies/parsed_cookie.cc
File net/cookies/parsed_cookie.cc (right):

https://codereview.chromium.org/640303008/diff/1/net/cookies/parsed_cookie.cc...
net/cookies/parsed_cookie.cc:140: return c <= 31;  // unsigned char is always >=
0.
just nuke this comment

[Chris Masone, 2014-10-17 20:14:50]

The CQ bit was checked by cmasone@chromium.org

[commit-bot: I haz the power, 2014-10-17 20:15:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/640303008/20001

[commit-bot: I haz the power, 2014-10-17 22:03:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-17 22:05:10]

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