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

Unified Diff: Source/devtools/front_end/timeline/TracingTimelineModel.js

Issue 396803008: Remove some of timeline experiments (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebased 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/devtools/front_end/timeline/TimelinePanel.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/timeline/TracingTimelineModel.js
diff --git a/Source/devtools/front_end/timeline/TracingTimelineModel.js b/Source/devtools/front_end/timeline/TracingTimelineModel.js
index ba539129b15c658676489cf81855768ee29cb4bc..a36ac1da0d1105d68cdcaf46f74d9c1bfa71ec9f 100644
--- a/Source/devtools/front_end/timeline/TracingTimelineModel.js
+++ b/Source/devtools/front_end/timeline/TracingTimelineModel.js
@@ -96,8 +96,6 @@ WebInspector.TracingTimelineModel.RecordType = {
PictureSnapshot: "cc::Picture"
};
-WebInspector.TracingTimelineModel.defaultTracingCategoryFilter = "-*,disabled-by-default-cc.debug,disabled-by-default-devtools.timeline,disabled-by-default-devtools.timeline.frame";
-
WebInspector.TracingTimelineModel.prototype = {
/**
* @param {boolean} captureStacks
@@ -110,21 +108,16 @@ WebInspector.TracingTimelineModel.prototype = {
{
return "disabled-by-default-" + category;
}
- var categories;
- if (WebInspector.experimentsSettings.timelineTracingMode.isEnabled()) {
- categories = WebInspector.TracingTimelineModel.defaultTracingCategoryFilter;
- } else {
- var categoriesArray = ["-*", disabledByDefault("devtools.timeline"), disabledByDefault("devtools.timeline.frame")];
- if (captureStacks)
- categoriesArray.push(disabledByDefault("devtools.timeline.stack"));
- if (capturePictures) {
- categoriesArray = categoriesArray.concat([
- disabledByDefault("devtools.timeline.layers"),
- disabledByDefault("devtools.timeline.picture"),
- disabledByDefault("blink.graphics_context_annotations")]);
- }
- categories = categoriesArray.join(",");
+ var categoriesArray = ["-*", disabledByDefault("devtools.timeline"), disabledByDefault("devtools.timeline.frame")];
+ if (captureStacks)
+ categoriesArray.push(disabledByDefault("devtools.timeline.stack"));
+ if (capturePictures) {
+ categoriesArray = categoriesArray.concat([
+ disabledByDefault("devtools.timeline.layers"),
+ disabledByDefault("devtools.timeline.picture"),
+ disabledByDefault("blink.graphics_context_annotations")]);
}
+ var categories = categoriesArray.join(",");
this._startRecordingWithCategories(categories);
},
« no previous file with comments | « Source/devtools/front_end/timeline/TimelinePanel.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698