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

Side by Side Diff: tracing/tracing/extras/importer/trace_event_importer_test.html

Issue 2013553002: [Tracing] Export TraceConfig to model.metadata (Closed) Base URL: https://github.com/catapult-project/catapult@master
Patch Set: Created 4 years, 6 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/extras/importer/trace_event_importer.html ('k') | no next file » | 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 (c) 2013 The Chromium Authors. All rights reserved. 3 Copyright (c) 2013 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/core/test_utils.html"> 8 <link rel="import" href="/tracing/core/test_utils.html">
9 <link rel="import" href="/tracing/extras/importer/trace_event_importer.html"> 9 <link rel="import" href="/tracing/extras/importer/trace_event_importer.html">
10 <link rel="import" href="/tracing/extras/measure/measure.html"> 10 <link rel="import" href="/tracing/extras/measure/measure.html">
(...skipping 5282 matching lines...) Expand 10 before | Expand all | Expand 10 after
5293 metadata: metadataValue 5293 metadata: metadataValue
5294 }; 5294 };
5295 5295
5296 var m = makeModel(eventData); 5296 var m = makeModel(eventData);
5297 assert.isTrue(m.metadata instanceof Array); 5297 assert.isTrue(m.metadata instanceof Array);
5298 assert.equal(m.metadata.length, 1); 5298 assert.equal(m.metadata.length, 1);
5299 assert.equal(m.metadata[0].name, 'metadata'); 5299 assert.equal(m.metadata[0].name, 'metadata');
5300 assert.equal(m.metadata[0].value, metadataValue); 5300 assert.equal(m.metadata[0].value, metadataValue);
5301 }); 5301 });
5302 5302
5303 test('importTraceConfigAsMetadata', function() {
5304 var traceConfigData = {
5305 enable_argument_filter: false,
5306 enable_sampling: false
5307 };
5308
5309 var eventData = {
5310 traceEvents: [
5311 { name: 'TraceConfig', args: {value: traceConfigData },
fmeawad 2016/05/24 21:03:51 Can you add a real example of a current TraceConfi
nednguyen 2016/05/24 21:11:51 This is from a real example: "{"pid":16662,"tid":1
5312 pid: 2, ts: 0, tid: 1, ph: 'M' },
5313 ],
5314 };
5315
5316 var m = makeModel(eventData);
5317 assert.isTrue(m.metadata instanceof Array);
5318 assert.equal(m.metadata.length, 1);
5319 assert.equal(m.metadata[0].name, 'TraceConfig');
5320 assert.equal(m.metadata[0].value, traceConfigData);
5321 });
5322
5303 test('importMarks', function() { 5323 test('importMarks', function() {
5304 var eventData = { 5324 var eventData = {
5305 traceEvents: [ 5325 traceEvents: [
5306 { name: 'mark1', args: {}, pid: 1, tid: 2, ts: 0, tts: 0, cat: 'blink.us er_timing', ph: 'R' }, // @suppress longLineCheck 5326 { name: 'mark1', args: {}, pid: 1, tid: 2, ts: 0, tts: 0, cat: 'blink.us er_timing', ph: 'R' }, // @suppress longLineCheck
5307 { name: 'mark2', args: {}, pid: 1, tid: 2, ts: 10, tts: 10, cat: 'blink. user_timing', ph: 'R' } // @suppress longLineCheck 5327 { name: 'mark2', args: {}, pid: 1, tid: 2, ts: 10, tts: 10, cat: 'blink. user_timing', ph: 'R' } // @suppress longLineCheck
5308 ] 5328 ]
5309 }; 5329 };
5310 5330
5311 var m = makeModel(eventData); 5331 var m = makeModel(eventData);
5312 5332
(...skipping 410 matching lines...) Expand 10 before | Expand all | Expand 10 after
5723 var m = makeModel(events); 5743 var m = makeModel(events);
5724 5744
5725 assert.isTrue(m.hasImportWarnings); 5745 assert.isTrue(m.hasImportWarnings);
5726 }); 5746 });
5727 5747
5728 // TODO(nduca): one slice, two threads 5748 // TODO(nduca): one slice, two threads
5729 // TODO(nduca): one slice, two pids 5749 // TODO(nduca): one slice, two pids
5730 5750
5731 }); 5751 });
5732 </script> 5752 </script>
OLDNEW
« no previous file with comments | « tracing/tracing/extras/importer/trace_event_importer.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698