Fix PerfCountNotifier email notifier used by perf_av bot.

Fix PerfCountNotifier (used by perf_av).

BUG=155124
TEST=unittests

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

[shadi, 2012-10-11 05:11:16]

cmp@ can you PTAL?

[shadi1, 2012-10-16 00:34:37]

On 2012/10/11 05:11:16, shadi wrote:
> cmp@ can you PTAL?

ping! :-) 

PS: This change only affects perf_av bots.

[cmp, 2012-10-22 18:39:16]

lgtm with nits

https://codereview.chromium.org/11092069/diff/2001/scripts/master/perf_count_...
File scripts/master/perf_count_notifier.py (right):

https://codereview.chromium.org/11092069/diff/2001/scripts/master/perf_count_...
scripts/master/perf_count_notifier.py:75: old_id.endswith(builder_name)]
Since the conditional spans the line, it would be best to wrap it in parens and
to indent line 75's old_id under line 74's old_id.

https://codereview.chromium.org/11092069/diff/2001/scripts/master/unittests/p...
File scripts/master/unittests/perf_count_notifier_test.py (right):

https://codereview.chromium.org/11092069/diff/2001/scripts/master/unittests/p...
scripts/master/unittests/perf_count_notifier_test.py:57: 
insert an empty line here

[shadi1, 2012-10-23 00:11:16]

https://codereview.chromium.org/11092069/diff/2001/scripts/master/perf_count_...
File scripts/master/perf_count_notifier.py (right):

https://codereview.chromium.org/11092069/diff/2001/scripts/master/perf_count_...
scripts/master/perf_count_notifier.py:75: old_id.endswith(builder_name)]
On 2012/10/22 18:39:16, cmp wrote:
> Since the conditional spans the line, it would be best to wrap it in parens
and
> to indent line 75's old_id under line 74's old_id.

Done.

https://codereview.chromium.org/11092069/diff/2001/scripts/master/unittests/p...
File scripts/master/unittests/perf_count_notifier_test.py (right):

https://codereview.chromium.org/11092069/diff/2001/scripts/master/unittests/p...
scripts/master/unittests/perf_count_notifier_test.py:57: 
On 2012/10/22 18:39:16, cmp wrote:
> insert an empty line here

Done.

[commit-bot: I haz the power, 2012-10-23 00:11:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shadi@chromium.org/11092069/10001

[commit-bot: I haz the power, 2012-10-23 00:13:19]

Change committed as 163450