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

Unified Diff: Source/devtools/front_end/TimelinePanel.js

Issue 216723003: Revert r170330 "DevTools: Start/Stop overview recording for available mode" (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 9 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/TimelinePanel.js
diff --git a/Source/devtools/front_end/TimelinePanel.js b/Source/devtools/front_end/TimelinePanel.js
index e9cab3de78c39cd1fd4bdd9d5d22ffe22dd3f474..1f6457db3eb8691756510c60dc493542ea1b4fdc 100644
--- a/Source/devtools/front_end/TimelinePanel.js
+++ b/Source/devtools/front_end/TimelinePanel.js
@@ -232,29 +232,6 @@ WebInspector.TimelinePanel.prototype = {
},
/**
- * @return {!WebInspector.TimelinePanel.Mode}
- */
- _availableModes: function()
- {
- if (this._modes)
- return this._modes;
-
- this._modes = {
- Events: WebInspector.TimelinePanel.Mode.Events,
- Frames: WebInspector.TimelinePanel.Mode.Frames,
- Memory: WebInspector.TimelinePanel.Mode.Memory,
- };
-
- if (WebInspector.experimentsSettings.timelineFlameChart.isEnabled())
- this._modes.FlameChart = WebInspector.TimelinePanel.Mode.FlameChart;
-
- if (Capabilities.canProfilePower)
- this._modes.Power = WebInspector.TimelinePanel.Mode.Power;
-
- return this._modes;
- },
-
- /**
* @return {!WebInspector.TimelineView}
*/
_timelineView: function()
@@ -317,7 +294,10 @@ WebInspector.TimelinePanel.prototype = {
var topPaneSidebarTree = new TreeOutline(overviewTreeElement);
this._overviewItems = {};
- for (var mode in this._availableModes()) {
+ for (var mode in WebInspector.TimelinePanel.Mode) {
+ if (mode === WebInspector.TimelinePanel.Mode.FlameChart && !WebInspector.experimentsSettings.timelineFlameChart.isEnabled() ||
+ mode === WebInspector.TimelinePanel.Mode.Power && !Capabilities.canProfilePower)
+ continue;
this._overviewItems[mode] = new WebInspector.SidebarTreeElement("timeline-overview-sidebar-" + mode.toLowerCase(), WebInspector.UIString(mode));
var item = this._overviewItems[mode];
item.onselect = this._onModeChanged.bind(this, mode);
@@ -578,7 +558,7 @@ WebInspector.TimelinePanel.prototype = {
{
this._userInitiatedRecording = userInitiated;
this._model.startRecording();
- for (var mode in this._availableModes())
+ for (var mode in WebInspector.TimelinePanel.Mode)
this._viewsForMode(mode).overviewView.timelineStarted();
if (userInitiated)
@@ -589,7 +569,7 @@ WebInspector.TimelinePanel.prototype = {
{
this._userInitiatedRecording = false;
this._model.stopRecording();
- for (var mode in this._availableModes())
+ for (var mode in WebInspector.TimelinePanel.Mode)
this._viewsForMode(mode).overviewView.timelineStopped();
},
« 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