MemorySanitizer: common.gypi changes.

MemorySanitizer: common.gypi changes.


R=glider@chromium.org,timurrrr@chromium.org
BUG=178409
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=184713

[Alexander Potapenko, 2013-02-26 13:59:36]

On 2013/02/26 13:57:05, eugenis wrote:

Please add Nico

[Nico, 2013-02-26 14:02:08]

1.) when adding someone to a review, please say what you want them to do. I got
a completely empty email.

2.) if you want to set up msan bots for chromium, please have a tracking bot for
this and describe how you intend to do this on there (I suppose completely new
bots, completely new expectations, bots will eventually be owned by the memory
sheriff rotation?)

[Evgeniy Stepanov, 2013-02-26 14:30:38]

On 2013/02/26 14:02:08, Nico wrote:
> 1.) when adding someone to a review, please say what you want them to do. I
got
> a completely empty email.

Yes, sorry. I just can not deal with the codereview interface.

> 2.) if you want to set up msan bots for chromium, please have a tracking bot
for
> this and describe how you intend to do this on there (I suppose completely new
> bots, completely new expectations, bots will eventually be owned by the memory
> sheriff rotation?)

I've added a reference to a bug that Alex filed, it should answer your
questions.

[Evgeniy Stepanov, 2013-02-26 14:32:57]

On 2013/02/26 14:30:38, Evgeniy Stepanov wrote:
> On 2013/02/26 14:02:08, Nico wrote:
> > 1.) when adding someone to a review, please say what you want them to do. I
> got
> > a completely empty email.
> 
> Yes, sorry. I just can not deal with the codereview interface.
> 
> > 2.) if you want to set up msan bots for chromium, please have a tracking bot
> for
> > this and describe how you intend to do this on there (I suppose completely
new
> > bots, completely new expectations, bots will eventually be owned by the
memory
> > sheriff rotation?)
> 
> I've added a reference to a bug that Alex filed, it should answer your
> questions.

Ok, _now_ I added the bug number. "git cl" does not update the description.

[Nico, 2013-02-26 14:36:09]

lgtm

[commit-bot: I haz the power, 2013-02-26 15:41:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eugenis@chromium.org/12342013/7001

[commit-bot: I haz the power, 2013-02-26 20:07:01]

Change committed as 184713