Android pylib: measurement name escaping fix.

Android pylib: measurement name escaping fix.

The original re.sub wasn't doing at all what wanted. Also, added / to be
replaced, because we now have URLs in measurement names, and measurement names
are also used for constructing directory names.

BUG=NONE
NOTRY=true


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

[marja, 2012-12-14 13:59:41]

bulach: ptal
michaelbai: fyi

[bulach, 2012-12-14 14:23:34]

lgtm

https://chromiumcodereview.appspot.com/11565033/diff/1/build/android/pylib/pe...
File build/android/pylib/perf_tests_helper.py (right):

https://chromiumcodereview.appspot.com/11565033/diff/1/build/android/pylib/pe...
build/android/pylib/perf_tests_helper.py:22: return re.sub('[\:|=/]', '_', s)
nit: would this work?
r'[:=/]'

[marja, 2012-12-14 16:17:13]

Thanks!

https://chromiumcodereview.appspot.com/11565033/diff/1/build/android/pylib/pe...
File build/android/pylib/perf_tests_helper.py (right):

https://chromiumcodereview.appspot.com/11565033/diff/1/build/android/pylib/pe...
build/android/pylib/perf_tests_helper.py:22: return re.sub('[\:|=/]', '_', s)
On 2012/12/14 14:23:34, bulach wrote:
> nit: would this work?
> r'[:=/]'

Yeah, but imo it's not any clearer than this version (to understand it, you need
to know that "aha, : is a special character inside [] *unless* it's in the
beginning"), and it's more error prone in case somebody feels like adding other
characters to the beginning instead of the end... -> left this like it was.

[commit-bot: I haz the power, 2012-12-14 16:17:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/marja@chromium.org/11565033/1

[commit-bot: I haz the power, 2012-12-14 16:19:02]

Change committed as 173167