Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(244)

Side by Side Diff: tracing/tracing/metrics/system_health/scroll_power_metric.html

Issue 2334233003: Merge NumericValue into Histogram (Closed)
Patch Set: fix rail_power_metric Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <!-- 2 <!--
3 Copyright 2016 The Chromium Authors. All rights reserved. 3 Copyright 2016 The Chromium Authors. All rights reserved.
4 Use of this source code is governed by a BSD-style license that can be 4 Use of this source code is governed by a BSD-style license that can be
5 found in the LICENSE file. 5 found in the LICENSE file.
6 --> 6 -->
7 7
8 <link rel="import" href="/tracing/base/statistics.html"> 8 <link rel="import" href="/tracing/base/statistics.html">
9 <link rel="import" href="/tracing/metrics/metric_registry.html"> 9 <link rel="import" href="/tracing/metrics/metric_registry.html">
10 <link rel="import" href="/tracing/value/histogram.html"> 10 <link rel="import" href="/tracing/value/histogram.html">
11 <link rel="import" href="/tracing/value/value.html">
12 11
13 <script> 12 <script>
14 'use strict'; 13 'use strict';
15 14
16 tr.exportTo('tr.metrics.sh', function() { 15 tr.exportTo('tr.metrics.sh', function() {
17 function scrollPowerMetric(values, model) { 16 function scrollPowerMetric(values, model) {
18 if (!model.device.powerSeries) 17 if (!model.device.powerSeries)
19 return; 18 return;
20 var timeHist = new tr.v.Histogram( 19 var timeHist = new tr.v.Histogram('time:scroll',
21 tr.b.Unit.byName.timeDurationInMs_smallerIsBetter) 20 tr.b.Unit.byName.timeDurationInMs_smallerIsBetter)
22 var energyHist = new tr.v.Histogram( 21 timeHist.description = 'Time spent in scrolling';
22 var energyHist = new tr.v.Histogram('energy:scroll',
23 tr.b.Unit.byName.energyInJoules_smallerIsBetter) 23 tr.b.Unit.byName.energyInJoules_smallerIsBetter)
24 var powerHist = new tr.v.Histogram( 24 energyHist.description = 'Energy consumed by scrolling';
25 var powerHist = new tr.v.Histogram('power:scroll',
25 tr.b.Unit.byName.powerInWatts_smallerIsBetter) 26 tr.b.Unit.byName.powerInWatts_smallerIsBetter)
27 powerHist.description = 'Energy consumption rate in scrolling';
26 28
27 timeHist.customizeSummaryOptions({ 29 timeHist.customizeSummaryOptions({
28 avg: false, 30 avg: false,
29 count: false, 31 count: false,
30 max: true, 32 max: true,
31 min: true, 33 min: true,
32 std: false, 34 std: false,
33 sum: true, 35 sum: true,
34 }); 36 });
35 37
(...skipping 23 matching lines...) Expand all
59 var energyInJ = model.device.powerSeries.getEnergyConsumedInJ( 61 var energyInJ = model.device.powerSeries.getEnergyConsumedInJ(
60 exp.start, exp.end); 62 exp.start, exp.end);
61 var powerInW = energyInJ / durationInS; 63 var powerInW = energyInJ / durationInS;
62 64
63 timeHist.addSample(durationInMs); 65 timeHist.addSample(durationInMs);
64 energyHist.addSample(energyInJ); 66 energyHist.addSample(energyInJ);
65 powerHist.addSample(powerInW); 67 powerHist.addSample(powerInW);
66 } 68 }
67 } 69 }
68 70
69 values.addValue(new tr.v.NumericValue('time:scroll', 71 values.addHistogram(timeHist);
70 timeHist, {description: 'Time spent in scrolling'})); 72 values.addHistogram(energyHist);
71 73 values.addHistogram(powerHist);
72 values.addValue(new tr.v.NumericValue('energy:scroll',
73 energyHist, {description: 'Energy consumed by scrolling'}));
74
75 values.addValue(new tr.v.NumericValue('power:scroll',
76 powerHist, {description: 'Energy consumption rate in scrolling'}));
77 } 74 }
78 75
79 tr.metrics.MetricRegistry.register(scrollPowerMetric); 76 tr.metrics.MetricRegistry.register(scrollPowerMetric);
80 77
81 return { 78 return {
82 scrollPowerMetric: scrollPowerMetric 79 scrollPowerMetric: scrollPowerMetric
83 }; 80 };
84 }); 81 });
85 </script> 82 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698