Patch by antomn@chromium.org

Description:
Massage test names into allowed set of characteres.

Luckily currently massaged names are still unique.

Review URL: http://codereview.chromium.org/269096

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29158 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 13 insertions, 8 deletions

diff --git a/chrome/test/ui/dromaeo_benchmark_uitest.cc b/chrome/test/ui/dromaeo_benchmark_uitest.cc
index 2467493..e1d060e 100644
--- a/chrome/test/ui/dromaeo_benchmark_uitest.cc
+++ b/chrome/test/ui/dromaeo_benchmark_uitest.cc
@@ -108,8 +108,13 @@ class DromaeoTest : public UITest {
     PrintResult("score", "", trace_name, score, unit_name, true);
 
     ResultsMap::const_iterator it = results.begin();
-    for (; it != results.end(); ++it)
-      PrintResult(it->first, "", trace_name, it->second, unit_name, false);
+    for (; it != results.end(); ++it) {
+      std::string test_name = it->first;
+      for (size_t i = 0; i < test_name.length(); i++)
+        if (!isalnum(test_name[i]))
+          test_name[i] = '_';
+      PrintResult(test_name, "", trace_name, it->second, unit_name, false);
+    }
   }
 
   DISALLOW_COPY_AND_ASSIGN(DromaeoTest);
@@ -140,17 +145,17 @@ class DromaeoReferenceTest : public DromaeoTest {
 };
 
 TEST_F(DromaeoTest, Perf) {
-  //if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
-  //  return;
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
 
-  //RunTest();
+  RunTest();
 }
 
 TEST_F(DromaeoReferenceTest, Perf) {
-  //if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
-  //  return;
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(kRunDromaeo))
+    return;
 
-  //RunTest();
+  RunTest();
 }
 
 }  // namespace