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

Unified Diff: third_party/WebKit/Source/devtools/front_end/settings/SettingsScreen.js

Issue 2839413002: DevTools: add rendering overlay options to command menu (Closed)
Patch Set: ac Created 3 years, 7 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/settings/SettingsScreen.js
diff --git a/third_party/WebKit/Source/devtools/front_end/settings/SettingsScreen.js b/third_party/WebKit/Source/devtools/front_end/settings/SettingsScreen.js
index 098edceb612134dd39948eb338508908e9c5cc47..75c9e9a00679b47cd5d4af9bdbf16939928351aa 100644
--- a/third_party/WebKit/Source/devtools/front_end/settings/SettingsScreen.js
+++ b/third_party/WebKit/Source/devtools/front_end/settings/SettingsScreen.js
@@ -146,8 +146,6 @@ Settings.GenericSettingsTab = class extends Settings.SettingsTab {
['', 'Appearance', 'Elements', 'Sources', 'Network', 'Profiler', 'Console', 'Extensions'];
/** @type {!Map<string, !Element>} */
this._nameToSection = new Map();
- /** @type {!Map<string, !Element>} */
- this._nameToSettingElement = new Map();
for (var sectionName of explicitSectionOrder)
this._sectionElement(sectionName);
self.runtime.extensions('setting').forEach(this._addSetting.bind(this));
@@ -181,33 +179,11 @@ Settings.GenericSettingsTab = class extends Settings.SettingsTab {
_addSetting(extension) {
if (!Settings.GenericSettingsTab.isSettingVisible(extension))
return;
- var descriptor = extension.descriptor();
- var sectionName = descriptor['category'];
- var settingName = descriptor['settingName'];
- var setting = Common.moduleSetting(settingName);
- var uiTitle = Common.UIString(extension.title());
-
- var sectionElement = this._sectionElement(sectionName);
- var settingControl;
-
- switch (descriptor['settingType']) {
- case 'boolean':
- settingControl = UI.SettingsUI.createSettingCheckbox(uiTitle, setting);
- break;
- case 'enum':
- if (Array.isArray(descriptor['options']))
- settingControl = UI.SettingsUI.createSettingSelect(uiTitle, descriptor['options'], setting);
- else
- console.error('Enum setting defined without options');
- break;
- default:
- console.error('Invalid setting type: ' + descriptor['settingType']);
- return;
- }
- if (settingControl) {
- this._nameToSettingElement.set(settingName, settingControl);
+ var sectionElement = this._sectionElement(extension.descriptor()['category']);
+ var setting = Common.moduleSetting(extension.descriptor()['settingName']);
+ var settingControl = UI.SettingsUI.createControlForSetting(setting);
+ if (settingControl)
sectionElement.appendChild(settingControl);
- }
}
/**

Powered by Google App Engine
This is Rietveld 408576698