X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Ftools%2Ftelemetry%2Ftelemetry%2Fvalue%2F__init__.py;h=c4330ecdd92c5da1a3b24217ed9d5e43b2840694;hb=3545e9f2671f595d2a2f3ee75ca0393b01e35ef6;hp=bfebc53afa87abe8dc5f483a5b4d2cf93fc32f2d;hpb=7d210d4c7e9ba36e635eabc5b5780495f8a63292;p=platform%2Fframework%2Fweb%2Fcrosswalk.git

diff --git a/src/tools/telemetry/telemetry/value/__init__.py b/src/tools/telemetry/telemetry/value/__init__.py
index bfebc53..c4330ec 100644
--- a/src/tools/telemetry/telemetry/value/__init__.py
+++ b/src/tools/telemetry/telemetry/value/__init__.py
@@ -130,9 +130,10 @@ class Value(object):
     """Returns the buildbot's equivalent value."""
     raise NotImplementedError()
 
-  def GetBuildbotMeasurementAndTraceNameForPerPageResult(self):
-    measurement, _ = _ConvertValueNameToBuildbotChartAndTraceName(self.name)
-    return measurement, self.page.display_name
+  def GetChartAndTraceNameForPerPageResult(self):
+    chart_name, _ = _ConvertValueNameToChartAndTraceName(self.name)
+    trace_name = self.page.display_name
+    return chart_name, trace_name
 
   @property
   def name_suffix(self):
@@ -142,14 +143,14 @@ class Value(object):
     else:
       return self.name
 
-  def GetBuildbotMeasurementAndTraceNameForComputedSummaryResult(
+  def GetChartAndTraceNameForComputedSummaryResult(
       self, trace_tag):
-    measurement, bb_trace_name = (
-        _ConvertValueNameToBuildbotChartAndTraceName(self.name))
+    chart_name, trace_name = (
+        _ConvertValueNameToChartAndTraceName(self.name))
     if trace_tag:
-      return measurement, bb_trace_name + trace_tag
+      return chart_name, trace_name + trace_tag
     else:
-      return measurement, bb_trace_name
+      return chart_name, trace_name
 
   def GetRepresentativeNumber(self):
     """Gets a single scalar value that best-represents this value.
@@ -179,6 +180,7 @@ class Value(object):
       'name': self.name,
       'type': self.GetJSONTypeName(),
       'units': self.units,
+      'important': self.important
     }
 
     if self.description:
@@ -290,12 +292,12 @@ def ValueNameFromTraceAndChartName(trace_name, chart_name=None):
         'empty chart_name since this is used to delimit chart_name.trace_name.')
     return trace_name
 
-def _ConvertValueNameToBuildbotChartAndTraceName(value_name):
-  """Converts a value_name into the buildbot equivalent name pair.
+def _ConvertValueNameToChartAndTraceName(value_name):
+  """Converts a value_name into the equivalent chart-trace name pair.
 
   Buildbot represents values by the measurement name and an optional trace name,
   whereas telemetry represents values with a chart_name.trace_name convention,
-  where chart_name is optional.
+  where chart_name is optional. This convention is also used by chart_json.
 
   This converts from the telemetry convention to the buildbot convention,
   returning a 2-tuple (measurement_name, trace_name).