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

Side by Side Diff: tracing/tracing/value/merge_histograms_cmdline.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/xhr.html"> 8 <link rel="import" href="/tracing/base/xhr.html">
9 <link rel="import" href="/tracing/value/histogram_set.html"> 9 <link rel="import" href="/tracing/value/histogram_set.html">
10 10
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 merged.push(hist.clone()); 53 merged.push(hist.clone());
54 } 54 }
55 for (const hist of merged) { 55 for (const hist of merged) {
56 mergedHistograms.addHistogram(hist); 56 mergedHistograms.addHistogram(hist);
57 } 57 }
58 } 58 }
59 } 59 }
60 60
61 function stripInternalDiagnostics(mergedHistograms) { 61 function stripInternalDiagnostics(mergedHistograms) {
62 for (const hist of mergedHistograms) { 62 for (const hist of mergedHistograms) {
63 hist.diagnostics.delete(tr.v.d.RESERVED_NAMES.MERGED_FROM);
64 hist.diagnostics.delete(tr.v.d.RESERVED_NAMES.MERGED_TO);
65 hist.diagnostics.delete(tr.v.d.RESERVED_NAMES.GROUPING_PATH); 63 hist.diagnostics.delete(tr.v.d.RESERVED_NAMES.GROUPING_PATH);
66 } 64 }
67 } 65 }
68 66
69 if (tr.isHeadless) { 67 if (tr.isHeadless) {
70 const histograms = new tr.v.HistogramSet(); 68 const histograms = new tr.v.HistogramSet();
71 histograms.importDicts(JSON.parse(tr.b.getSync('file://' + sys.argv[1]))); 69 histograms.importDicts(JSON.parse(tr.b.getSync('file://' + sys.argv[1])));
72 histograms.resolveRelatedHistograms(); 70 histograms.resolveRelatedHistograms();
73 histograms.buildGroupingsFromTags([tr.v.d.RESERVED_NAMES.STORY_TAGS]); 71 histograms.buildGroupingsFromTags([tr.v.d.RESERVED_NAMES.STORY_TAGS]);
74 const groupingKeys = sys.argv.slice(2); 72 const groupingKeys = sys.argv.slice(2);
75 const groupings = groupingKeys.map(findGrouping); 73 const groupings = groupingKeys.map(findGrouping);
76 const groupedHistograms = histograms.groupHistogramsRecursively(groupings); 74 const groupedHistograms = histograms.groupHistogramsRecursively(groupings);
77 const mergedHistograms = new tr.v.HistogramSet(); 75 const mergedHistograms = new tr.v.HistogramSet();
78 mergeLeafHistograms(groupedHistograms, mergedHistograms); 76 mergeLeafHistograms(groupedHistograms, mergedHistograms);
79 mergedHistograms.deduplicateDiagnostics(); 77 mergedHistograms.deduplicateDiagnostics();
80 mergedHistograms.mergeRelationships(); 78 mergedHistograms.mergeRelationships();
81 stripInternalDiagnostics(mergedHistograms); 79 stripInternalDiagnostics(mergedHistograms);
82 console.log(JSON.stringify(mergedHistograms.asDicts())); 80 console.log(JSON.stringify(mergedHistograms.asDicts()));
83 } 81 }
84 </script> 82 </script>
OLDNEW
« no previous file with comments | « tracing/tracing/value/histogram_unittest.py ('k') | tracing/tracing/value/ui/breakdown_span.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698