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

Side by Side Diff: tracing/tracing/value/histogram.html

Issue 3009553002: Refactor Histogram relationship diagnostics. (Closed)
Patch Set: Created 3 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/math/range.html"> 8 <link rel="import" href="/tracing/base/math/range.html">
9 <link rel="import" href="/tracing/base/math/running_statistics.html"> 9 <link rel="import" href="/tracing/base/math/running_statistics.html">
10 <link rel="import" href="/tracing/base/math/statistics.html"> 10 <link rel="import" href="/tracing/base/math/statistics.html">
(...skipping 555 matching lines...) Expand 10 before | Expand all | Expand 10 after
566 566
567 for (let i = 0; i < this.allBins.length; ++i) { 567 for (let i = 0; i < this.allBins.length; ++i) {
568 let bin = this.allBins[i]; 568 let bin = this.allBins[i];
569 if (bin.count === 0) { 569 if (bin.count === 0) {
570 bin = new HistogramBin(bin.range); 570 bin = new HistogramBin(bin.range);
571 this.allBins[i] = bin; 571 this.allBins[i] = bin;
572 } 572 }
573 bin.addBin(other.allBins[i]); 573 bin.addBin(other.allBins[i]);
574 } 574 }
575 575
576 let mergedFrom = this.diagnostics.get(tr.v.d.RESERVED_NAMES.MERGED_FROM);
577 if (!mergedFrom) {
578 mergedFrom = new tr.v.d.RelatedHistogramMap();
579 this.diagnostics.set(tr.v.d.RESERVED_NAMES.MERGED_FROM, mergedFrom);
580 }
581 mergedFrom.set(mergedFrom.length, other);
582
583 let mergedTo = other.diagnostics.get(tr.v.d.RESERVED_NAMES.MERGED_TO);
584 if (!mergedTo) {
585 mergedTo = new tr.v.d.RelatedHistogramMap();
586 other.diagnostics.set(tr.v.d.RESERVED_NAMES.MERGED_TO, mergedTo);
587 }
588 mergedTo.set(mergedTo.length, this);
589
590 this.diagnostics.addDiagnostics(other.diagnostics); 576 this.diagnostics.addDiagnostics(other.diagnostics);
591 577
592 for (const [stat, option] of other.summaryOptions) { 578 for (const [stat, option] of other.summaryOptions) {
593 if (stat === 'percentile') { 579 if (stat === 'percentile') {
594 const percentiles = this.summaryOptions.get(stat); 580 const percentiles = this.summaryOptions.get(stat);
595 for (const percent of option) { 581 for (const percent of option) {
596 if (!percentiles.includes(percent)) percentiles.push(percent); 582 if (!percentiles.includes(percent)) percentiles.push(percent);
597 } 583 }
598 } else if (stat === 'iprs') { 584 } else if (stat === 'iprs') {
599 const thisIprs = this.summaryOptions.get(stat); 585 const thisIprs = this.summaryOptions.get(stat);
(...skipping 824 matching lines...) Expand 10 before | Expand all | Expand 10 after
1424 Histogram, 1410 Histogram,
1425 HistogramBinBoundaries, 1411 HistogramBinBoundaries,
1426 P_VALUE_NAME, 1412 P_VALUE_NAME,
1427 U_STATISTIC_NAME, 1413 U_STATISTIC_NAME,
1428 Z_SCORE_NAME, 1414 Z_SCORE_NAME,
1429 percentFromString, 1415 percentFromString,
1430 percentToString, 1416 percentToString,
1431 }; 1417 };
1432 }); 1418 });
1433 </script> 1419 </script>
OLDNEW
« no previous file with comments | « tracing/tracing/value/diagnostics/unmergeable_diagnostic_set.html ('k') | tracing/tracing/value/histogram.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698