diff options
| author | marja@chromium.org <marja@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-14 16:19:00 +0000 |
|---|---|---|
| committer | marja@chromium.org <marja@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-14 16:19:00 +0000 |
| commit | 9cb760a444355ff049af9268160847daf588249c (patch) | |
| tree | 0cf2e80165df96c5a6f67e8cbe6a8426bc82e107 | |
| parent | 6f34b4836f777a3c325ded3256b313c030fc3568 (diff) | |
| download | chromium_src-9cb760a444355ff049af9268160847daf588249c.zip chromium_src-9cb760a444355ff049af9268160847daf588249c.tar.gz chromium_src-9cb760a444355ff049af9268160847daf588249c.tar.bz2 | |
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
Review URL: https://chromiumcodereview.appspot.com/11565033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173167 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | build/android/pylib/perf_tests_helper.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/build/android/pylib/perf_tests_helper.py b/build/android/pylib/perf_tests_helper.py index 3334b0b..ca9023b 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) def GeomMeanAndStdDevFromHistogram(histogram_json): |