Android pylib: measurement name escaping fix.

build/android/pylib/perf_tests_helper.py

Index: build/android/pylib/perf_tests_helper.py
diff --git a/build/android/pylib/perf_tests_helper.py b/build/android/pylib/perf_tests_helper.py
index 3334b0b3943e3b3a5de0ce07ea2c1a38edb95ca1..ca9023b679393c4c6df7e5e73d39d7c259d8a9a4 100644
--- a/build/android/pylib/perf_tests_helper.py
+++ b/build/android/pylib/perf_tests_helper.py
@@ -18,8 +18,8 @@ RESULT_TYPES = {'unimportant': 'RESULT ',
 
 def _EscapePerfResult(s):
   """Escapes |s| for use in a perf result."""
-  # Colons (:) and equal signs (=) are not allowed.
-  return re.sub(':|=', '_', s)
+  # Colons (:), equal signs (=) and slashes (/) are not allowed.
+  return re.sub('[\:|=/]', '_', s)

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

nit: would this work?
r'[:=/]'

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

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.

 
 
 def GeomMeanAndStdDevFromHistogram(histogram_json):