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

Unified Diff: third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js

Issue 2678623002: DevTools: pass title when creating settings (Closed)
Patch Set: a Created 3 years, 10 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
Index: third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js b/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js
index a39ba22b83a48f18866c9cca3468e42272c75a07..ca300bd5bf1a89d2405dcd7ae755cecbbbdbd30e 100644
--- a/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js
+++ b/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js
@@ -78,12 +78,17 @@ Timeline.TimelinePanel = class extends UI.Panel {
this._viewModeSetting =
Common.settings.createSetting('timelineViewMode', Timeline.TimelinePanel.ViewMode.FlameChart);
- this._disableCaptureJSProfileSetting = Common.settings.createSetting('timelineDisableJSSampling', false);
- this._captureLayersAndPicturesSetting = Common.settings.createSetting('timelineCaptureLayersAndPictures', false);
-
- this._showScreenshotsSetting = Common.settings.createLocalSetting('timelineShowScreenshots', true);
+ this._disableCaptureJSProfileSetting = Common.settings.createSetting(
+ 'timelineDisableJSSampling', false, false, Common.UIString('Disable JavaScript Samples'));
+ this._captureLayersAndPicturesSetting = Common.settings.createSetting(
+ 'timelineCaptureLayersAndPictures', false, false,
+ Common.UIString('Enable advanced paint instrumentation (slow)'));
+
+ this._showScreenshotsSetting =
+ Common.settings.createLocalSetting('timelineShowScreenshots', true, Common.UIString('Screenshots'));
this._showScreenshotsSetting.addChangeListener(this._onModeChanged, this);
- this._showMemorySetting = Common.settings.createLocalSetting('timelineShowMemory', false);
+ this._showMemorySetting =
+ Common.settings.createLocalSetting('timelineShowMemory', false, Common.UIString('Memory'));
this._showMemorySetting.addChangeListener(this._onModeChanged, this);
this._panelToolbar = new UI.Toolbar('', this.element);
@@ -271,13 +276,12 @@ Timeline.TimelinePanel = class extends UI.Panel {
}
/**
- * @param {string} name
* @param {!Common.Setting} setting
* @param {string} tooltip
* @return {!UI.ToolbarItem}
*/
- _createSettingCheckbox(name, setting, tooltip) {
- const checkboxItem = new UI.ToolbarCheckbox(name, tooltip, setting);
+ _createSettingCheckbox(setting, tooltip) {
+ const checkboxItem = new UI.ToolbarSettingCheckbox(setting, tooltip);
this._recordingOptionUIControls.push(checkboxItem);
return checkboxItem;
}
@@ -293,12 +297,12 @@ Timeline.TimelinePanel = class extends UI.Panel {
// View
this._panelToolbar.appendSeparator();
- this._showScreenshotsToolbarCheckbox = this._createSettingCheckbox(
- Common.UIString('Screenshots'), this._showScreenshotsSetting, Common.UIString('Capture screenshots'));
+ this._showScreenshotsToolbarCheckbox =
+ this._createSettingCheckbox(this._showScreenshotsSetting, Common.UIString('Capture screenshots'));
this._panelToolbar.appendToolbarItem(this._showScreenshotsToolbarCheckbox);
- this._showMemoryToolbarCheckbox = this._createSettingCheckbox(
- Common.UIString('Memory'), this._showMemorySetting, Common.UIString('Show memory timeline.'));
+ this._showMemoryToolbarCheckbox =
+ this._createSettingCheckbox(this._showMemorySetting, Common.UIString('Show memory timeline.'));
this._panelToolbar.appendToolbarItem(this._showMemoryToolbarCheckbox);
// GC
@@ -330,10 +334,10 @@ Timeline.TimelinePanel = class extends UI.Panel {
captureToolbar.element.classList.add('flex-auto');
captureToolbar.makeVertical();
captureToolbar.appendToolbarItem(this._createSettingCheckbox(
- Common.UIString('Disable JavaScript Samples'), this._disableCaptureJSProfileSetting,
+ this._disableCaptureJSProfileSetting,
Common.UIString('Disables JavaScript sampling, reduces overhead when running against mobile devices')));
captureToolbar.appendToolbarItem(this._createSettingCheckbox(
- Common.UIString('Enable advanced paint instrumentation (slow)'), this._captureLayersAndPicturesSetting,
+ this._captureLayersAndPicturesSetting,
Common.UIString('Captures advanced paint instrumentation, introduces significant performance overhead')));
var throttlingPane = new UI.VBox();
@@ -357,7 +361,7 @@ Timeline.TimelinePanel = class extends UI.Panel {
_appendExtensionsToToolbar(event) {
var provider = /** @type {!Extensions.ExtensionTraceProvider} */ (event.data);
const setting = Timeline.TimelinePanel._settingForTraceProvider(provider);
- const checkbox = this._createSettingCheckbox(provider.shortDisplayName(), setting, provider.longDisplayName());
+ const checkbox = this._createSettingCheckbox(setting, provider.longDisplayName());
this._panelToolbar.appendToolbarItem(checkbox);
}
@@ -369,7 +373,7 @@ Timeline.TimelinePanel = class extends UI.Panel {
var setting = traceProvider[Timeline.TimelinePanel._traceProviderSettingSymbol];
if (!setting) {
var providerId = traceProvider.persistentIdentifier();
- setting = Common.settings.createSetting(providerId, false);
+ setting = Common.settings.createSetting(providerId, false, false, traceProvider.shortDisplayName());
traceProvider[Timeline.TimelinePanel._traceProviderSettingSymbol] = setting;
}
return setting;

Powered by Google App Engine
This is Rietveld 408576698