Add reviewers to optional outputs for changes

Add reviewers to optional outputs for changes

This adds a comma-separated list of reviewers to the optional outputs
for changes. This can be added to the output using the custom formatter
option.  E.g. --output-format-changes="{url} {title} <{reviewers}>"

Also, for webkit changes, use CommitterList to map reviewer names to
emails so that they are similar format to other changes.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=187351

[cjhopman, 2013-03-11 18:12:21]

enne: check if the webkit change is reasonable
cmp: OWNERS

[Agrawal, 2013-03-11 18:27:57]

On 2013/03/11 18:12:21, cjhopman wrote:
> enne: check if the webkit change is reasonable
> cmp: OWNERS

LGTM

[Agrawal, 2013-03-11 18:28:26]

On 2013/03/11 18:27:57, Agrawal wrote:
> On 2013/03/11 18:12:21, cjhopman wrote:
> > enne: check if the webkit change is reasonable
> > cmp: OWNERS
> 
> LGTM

I think you should add yourself as a per-file OWNER for this

[cmp, 2013-03-11 18:37:53]

lgtm

[enne (OOO), 2013-03-11 19:14:16]

WebKit change lgtm (although there's only ever one WebKit reviewer for any
patch).

https://codereview.chromium.org/12446008/diff/1/my_activity.py
File my_activity.py (right):

https://codereview.chromium.org/12446008/diff/1/my_activity.py#newcode741
my_activity.py:741: def print_review(self, review):
What about passing reviewers as an optional value here too?

[commit-bot: I haz the power, 2013-03-11 19:59:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/cjhopman@chromium.org/12446008/6003

[commit-bot: I haz the power, 2013-03-11 20:02:45]

Change committed as 187351