Make presumbit warnings ignore symbols that start with UNIT_TEST

Make presumbit warnings ignore symbols that start with UNIT_TEST

UNIT_TEST_MODE was being flagged as use of UNIT_TEST on a cc file.

NOTRY=true
R=joi@chromium.org
BUG=

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

[Jói, 2013-11-18 06:41:18]

https://codereview.chromium.org/60573010/diff/1/PRESUBMIT.py
File PRESUBMIT.py (right):

https://codereview.chromium.org/60573010/diff/1/PRESUBMIT.py#newcode299
PRESUBMIT.py:299: if 'UNIT_TEST ' in line:
Suggest combining these conditionals since the action is the same for both:

if 'UNIT_TEST ' in line or line.endswith('UNIT_TEST'):
  ...

[rvargas (doing something else), 2013-11-18 19:07:37]

thanks

https://codereview.chromium.org/60573010/diff/1/PRESUBMIT.py
File PRESUBMIT.py (right):

https://codereview.chromium.org/60573010/diff/1/PRESUBMIT.py#newcode299
PRESUBMIT.py:299: if 'UNIT_TEST ' in line:
On 2013/11/18 06:41:19, Jói wrote:
> Suggest combining these conditionals since the action is the same for both:
> 
> if 'UNIT_TEST ' in line or line.endswith('UNIT_TEST'):
>   ...

:) done

[Jói, 2013-11-19 12:30:21]

LGTM

[commit-bot: I haz the power, 2013-11-19 12:30:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rvargas@chromium.org/60573010/60001

[commit-bot: I haz the power, 2013-11-19 12:57:40]

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

[commit-bot: I haz the power, 2013-11-19 12:59:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rvargas@chromium.org/60573010/60001

[commit-bot: I haz the power, 2013-11-19 13:00:07]

Change committed as 235987