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

Side by Side Diff: Source/devtools/front_end/timeline/TracingTimelineModel.js

Issue 365103002: DevTools: [Timeline] do not collect "devtools" category because we do not use it any more. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | 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 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @constructor 6 * @constructor
7 * @param {!WebInspector.TracingModel} tracingModel 7 * @param {!WebInspector.TracingModel} tracingModel
8 * @param {!WebInspector.TimelineModel.Filter} recordFilter 8 * @param {!WebInspector.TimelineModel.Filter} recordFilter
9 * @extends {WebInspector.TimelineModel} 9 * @extends {WebInspector.TimelineModel}
10 */ 10 */
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 * @param {boolean} captureStacks 103 * @param {boolean} captureStacks
104 * @param {boolean} captureMemory 104 * @param {boolean} captureMemory
105 * @param {boolean} capturePictures 105 * @param {boolean} capturePictures
106 */ 106 */
107 startRecording: function(captureStacks, captureMemory, capturePictures) 107 startRecording: function(captureStacks, captureMemory, capturePictures)
108 { 108 {
109 var categories; 109 var categories;
110 if (WebInspector.experimentsSettings.timelineTracingMode.isEnabled()) { 110 if (WebInspector.experimentsSettings.timelineTracingMode.isEnabled()) {
111 categories = WebInspector.TracingTimelineModel.defaultTracingCategor yFilter; 111 categories = WebInspector.TracingTimelineModel.defaultTracingCategor yFilter;
112 } else { 112 } else {
113 var categoriesArray = ["disabled-by-default-devtools.timeline", "dis abled-by-default-devtools.timeline.frame", "devtools"]; 113 var categoriesArray = ["disabled-by-default-devtools.timeline", "dis abled-by-default-devtools.timeline.frame"];
114 if (captureStacks) 114 if (captureStacks)
115 categoriesArray.push("disabled-by-default-devtools.timeline.stac k"); 115 categoriesArray.push("disabled-by-default-devtools.timeline.stac k");
116 if (capturePictures) 116 if (capturePictures)
117 categoriesArray.push("disabled-by-default-devtools.timeline.laye rs", "disabled-by-default-devtools.timeline.picture"); 117 categoriesArray.push("disabled-by-default-devtools.timeline.laye rs", "disabled-by-default-devtools.timeline.picture");
118 categories = categoriesArray.join(","); 118 categories = categoriesArray.join(",");
119 } 119 }
120 this._startRecordingWithCategories(categories); 120 this._startRecordingWithCategories(categories);
121 }, 121 },
122 122
123 stopRecording: function() 123 stopRecording: function()
(...skipping 528 matching lines...) Expand 10 before | Expand all | Expand 10 after
652 * @param {!WebInspector.TracingTimelineModel.TraceEventRecord} child 652 * @param {!WebInspector.TracingTimelineModel.TraceEventRecord} child
653 */ 653 */
654 _addChild: function(child) 654 _addChild: function(child)
655 { 655 {
656 this._children.push(child); 656 this._children.push(child);
657 child.parent = this; 657 child.parent = this;
658 }, 658 },
659 659
660 660
661 } 661 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698