[Telemetry] Use telemetryFormatter in scrolling list

Use the injected telemetryFormatter in scrolling
list views, instead of replicating this functionality.
WTD-599.
This commit is contained in:
Victor Woeltjen
2014-12-24 11:23:59 -08:00
parent 74f2af1011
commit 44b16f9dd9
4 changed files with 19 additions and 19 deletions

View File

@ -16,8 +16,10 @@ define(
* @param rangeMetadata an object with the machine- and human-
* readable names for this range (in `key` and `name`
* fields, respectively.)
* @param {TelemetryFormatter} telemetryFormatter the telemetry
* formatting service, for making values human-readable.
*/
function RangeColumn(rangeMetadata) {
function RangeColumn(rangeMetadata, telemetryFormatter) {
return {
/**
* Get the title to display in this column's header.
@ -32,8 +34,9 @@ define(
* @returns {string} the text to display
*/
getValue: function (domainObject, data, index) {
var value = data.getRangeValue(index, rangeMetadata.key);
return value && value.toFixed(3);
return telemetryFormatter.formatRangeValue(
data.getRangeValue(index, rangeMetadata.key)
);
}
};
}