Fix name/path separator for telemetry_unittests test names.

scripts/slave/recipe_modules/json/api.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import functools
 import collections
 import contextlib
 import json
 
 from cStringIO import StringIO
@@ skipping 64 matching lines @@
 
     if self.add_json_log:
       key = self.name + ('' if valid else ' (invalid)')
       with contextlib.closing(StringListIO()) as listio:
         json.dump(ret, listio, indent=2, sort_keys=True)
       presentation.logs[key] = listio.lines
 
     return ret
 
 
-# TODO(phajdan.jr): Rename to LayoutTestResultsOutputPlaceholder.

[Dirk Pranke, 2014/10/24 00:15:22]

I removed this and the other TODOs since this class is being used by the other
PythonBased tests now. We can certainly rename it to something if need be to
disambiguate it from the gtest-based stuff, but LayoutTestResults would be the
wrong thing to rename it to.

 class TestResultsOutputPlaceholder(JsonOutputPlaceholder):
   def result(self, presentation, test):
     ret = super(TestResultsOutputPlaceholder, self).result(presentation, test)
     return TestResults(ret)
 
 
 # TODO(martiniss) replace this with step.AggregateResults once
 # aggregate steps lands
 class GTestResultsOutputPlaceholder(JsonOutputPlaceholder):
   def result(self, presentation, test):
@@ skipping 15 matching lines @@
   @recipe_util.returns_placeholder
   def input(self, data):
     """A placeholder which will expand to a file path containing <data>."""
     return self.m.raw_io.input(self.dumps(data), '.json')
 
   @recipe_util.returns_placeholder
   def output(self, add_json_log=True):
     """A placeholder which will expand to '/tmp/file'."""
     return JsonOutputPlaceholder(self, add_json_log)
 
-  # TODO(phajdan.jr): Rename to layout_test_results.
   @recipe_util.returns_placeholder
   def test_results(self, add_json_log=True):
     """A placeholder which will expand to '/tmp/file'.
 
     The recipe must provide the expected --json-test-results flag.
 
     The test_results will be an instance of the TestResults class.
     """
     return TestResultsOutputPlaceholder(self, add_json_log)
 
@@ skipping 31 matching lines @@
 
     It's vastly preferable to have your recipe only pass the bare minimum
     of arguments to steps. Passing property objects obscures the data that
     the script actually consumes from the property object.
     """
     prop_str = self.dumps(dict(self.m.properties))
     return [
       '--factory-properties', prop_str,
       '--build-properties', prop_str
     ]