Check author of a CL agaist the AUTHORS file.

Check author of a CL against the AUTHORS file.

The code is copied from Chromium presubmit.

BUG=

Committed: https://crrev.com/770d866823dd9240e2adfd93a3abca447bc14297
Cr-Commit-Position: refs/heads/master@{#26077}

[Michael Achenbach, 2015-01-15 12:29:21]

machenbach@chromium.org changed reviewers:
+ machenbach@chromium.org

[Michael Achenbach, 2015-01-15 12:29:22]

lgtm

https://codereview.chromium.org/837503003/diff/1/tools/presubmit.py
File tools/presubmit.py (right):

https://codereview.chromium.org/837503003/diff/1/tools/presubmit.py#newcode460
tools/presubmit.py:460: return [output_api.PresubmitPromptWarning(
I assume a warning is enough to bail out?

[marja, 2015-01-15 12:37:40]

thanks for review

https://codereview.chromium.org/837503003/diff/1/tools/presubmit.py
File tools/presubmit.py (right):

https://codereview.chromium.org/837503003/diff/1/tools/presubmit.py#newcode460
tools/presubmit.py:460: return [output_api.PresubmitPromptWarning(
On 2015/01/15 12:29:22, Michael Achenbach wrote:
> I assume a warning is enough to bail out?

It's a prompt warning, so it'll prompt the user and user has a chance to [y/n].
The CQ will not answer that and will just bail out.

(Let's keep consistent w/ chromium regarding warning level & type.)

[marja, 2015-01-15 12:41:47]

The CQ bit was checked by marja@chromium.org

[commit-bot: I haz the power, 2015-01-15 12:42:17]

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

[commit-bot: I haz the power, 2015-01-15 13:10:06]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-01-15 13:10:18]

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