Commit ba3e74de authored by eakuefner's avatar eakuefner Committed by Commit bot

[Telemetry] Add 'important' to JSON output.

We have a plan in progress to upgrade the 'important' field to a combination
of two fields 'primary' and 'improvement_direction'. This will be a two-sided
patch that will require reworking the dashboard's handling of chart JSON. Since
the dashboard has been modified to accept chart JSON with 'important' values,
we want to update Telemetry to continue serializing values with this field for
now.

R=nednguyen

Review URL: https://codereview.chromium.org/541253002

Cr-Commit-Position: refs/heads/master@{#293468}
parent 731b3521
......@@ -179,6 +179,7 @@ class Value(object):
'name': self.name,
'type': self.GetJSONTypeName(),
'units': self.units,
'important': self.important
}
if self.description:
......
......@@ -97,6 +97,7 @@ class ValueTest(TestBase):
'name': 'x',
'type': 'baz',
'units': 'unit',
'important': True
})
def testAsDictWithPage(self):
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment