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

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: tracing/tracing/value/diagnostics/composition_test.html

Issue 2293533002: Refactor NumericBuilder to HistogramBinBoundaries. (Closed) Base URL: https://github.com/catapult-project/catapult.git@master
Patch Set: rebase 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
« no previous file with comments | « tracing/tracing/value/diagnostics/composition.html ('k') | tracing/tracing/value/histogram.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/extras/chrome/chrome_user_friendly_category_dr iver.html"> 8 <link rel="import" href="/tracing/extras/chrome/chrome_user_friendly_category_dr iver.html">
9 <link rel="import" href="/tracing/model/event_set.html"> 9 <link rel="import" href="/tracing/model/event_set.html">
10 <link rel="import" href="/tracing/value/diagnostics/composition.html"> 10 <link rel="import" href="/tracing/value/diagnostics/composition.html">
11 <link rel="import" href="/tracing/value/histogram.html"> 11 <link rel="import" href="/tracing/value/histogram.html">
12 <link rel="import" href="/tracing/value/value.html"> 12 <link rel="import" href="/tracing/value/value.html">
13 <link rel="import" href="/tracing/value/value_set.html"> 13 <link rel="import" href="/tracing/value/value_set.html">
14 14
15 <script> 15 <script>
16 'use strict'; 16 'use strict';
17 17
18 tr.b.unittest.testSuite(function() { 18 tr.b.unittest.testSuite(function() {
19 var TEST_NUMERIC_BUILDER = tr.v.NumericBuilder.createLinear( 19 var TEST_BOUNDARIES = tr.v.HistogramBinBoundaries.createLinear(0, 10, 10);
20 tr.v.Unit.byName.timeDurationInMs, tr.b.Range.fromExplicitRange(0, 10),
21 10);
22 20
23 test('build', function() { 21 test('build', function() {
24 var values = new tr.v.ValueSet(); 22 var values = new tr.v.ValueSet();
25 var events = new tr.model.EventSet([ 23 var events = new tr.model.EventSet([
26 { 24 {
27 guid: 9, 25 guid: 9,
28 title: 'V8.Execute', 26 title: 'V8.Execute',
29 cpuSelfTime: 1, 27 cpuSelfTime: 1,
30 stableId: '11.11' 28 stableId: '11.11'
31 }, 29 },
(...skipping 10 matching lines...) Expand all
42 stableId: '33.33' 40 stableId: '33.33'
43 }, 41 },
44 { 42 {
45 guid: 12, 43 guid: 12,
46 title: 'UpdateLayoutTree', 44 title: 'UpdateLayoutTree',
47 cpuSelfTime: undefined, 45 cpuSelfTime: undefined,
48 stableId: '44.44' 46 stableId: '44.44'
49 } 47 }
50 ]); 48 ]);
51 49
50 var sampleForEvent = undefined;
52 var composition = tr.v.d.Composition.buildFromEvents( 51 var composition = tr.v.d.Composition.buildFromEvents(
53 values, 'foo-', events, TEST_NUMERIC_BUILDER, 52 values, 'foo-', events,
54 tr.e.chrome.ChromeUserFriendlyCategoryDriver.fromEvent); 53 tr.e.chrome.ChromeUserFriendlyCategoryDriver.fromEvent,
54 tr.v.Unit.byName.timeDurationInMs, sampleForEvent, TEST_BOUNDARIES);
55 55
56 assert.lengthOf(composition, 2); 56 assert.lengthOf(composition, 2);
57 var scriptValue = values.getValuesWithName('foo-script')[0]; 57 var scriptValue = values.getValuesWithName('foo-script')[0];
58 assert.instanceOf(scriptValue, tr.v.NumericValue); 58 assert.instanceOf(scriptValue, tr.v.NumericValue);
59 assert.strictEqual(scriptValue, composition.get('script_execute')); 59 assert.strictEqual(scriptValue, composition.get('script_execute'));
60 assert.lengthOf(scriptValue.numeric.centralBins[1].diagnosticMaps, 1); 60 assert.lengthOf(scriptValue.numeric.centralBins[1].diagnosticMaps, 1);
61 events = [...events]; 61 events = [...events];
62 var relatedEvents = scriptValue.numeric.centralBins[1].diagnosticMaps[0] 62 var relatedEvents = scriptValue.numeric.centralBins[1].diagnosticMaps[0]
63 .get('relatedEvents'); 63 .get('relatedEvents');
64 assert.lengthOf(relatedEvents, 1); 64 assert.lengthOf(relatedEvents, 1);
(...skipping 10 matching lines...) Expand all
75 assert.instanceOf(styleValue, tr.v.NumericValue); 75 assert.instanceOf(styleValue, tr.v.NumericValue);
76 assert.strictEqual(styleValue, composition.get('style')); 76 assert.strictEqual(styleValue, composition.get('style'));
77 assert.lengthOf(styleValue.numeric.centralBins[3].diagnosticMaps, 1); 77 assert.lengthOf(styleValue.numeric.centralBins[3].diagnosticMaps, 1);
78 relatedEvents = styleValue.numeric.centralBins[3].diagnosticMaps[0] 78 relatedEvents = styleValue.numeric.centralBins[3].diagnosticMaps[0]
79 .get('relatedEvents'); 79 .get('relatedEvents');
80 assert.lengthOf(relatedEvents, 1); 80 assert.lengthOf(relatedEvents, 1);
81 assert.strictEqual(events[2], [...relatedEvents][0]); 81 assert.strictEqual(events[2], [...relatedEvents][0]);
82 }); 82 });
83 }); 83 });
84 </script> 84 </script>
OLDNEW
« no previous file with comments | « tracing/tracing/value/diagnostics/composition.html ('k') | tracing/tracing/value/histogram.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698