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

Unified Diff: third_party/WebKit/Source/devtools/front_end/layer_viewer/Layers3DView.js

Issue 2678623002: DevTools: pass title when creating settings (Closed)
Patch Set: ac 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/layer_viewer/Layers3DView.js
diff --git a/third_party/WebKit/Source/devtools/front_end/layer_viewer/Layers3DView.js b/third_party/WebKit/Source/devtools/front_end/layer_viewer/Layers3DView.js
index 52f1fe91877c8271ea7c8dd77101e04e5f0e91b7..949a6fb49c2669038f16f9caf6613117a13497da 100644
--- a/third_party/WebKit/Source/devtools/front_end/layer_viewer/Layers3DView.js
+++ b/third_party/WebKit/Source/devtools/front_end/layer_viewer/Layers3DView.js
@@ -666,11 +666,10 @@ LayerViewer.Layers3DView = class extends UI.VBox {
* @return {!Common.Setting}
*/
_createVisibilitySetting(caption, name, value, toolbar) {
- var checkbox = new UI.ToolbarCheckbox(Common.UIString(caption));
- toolbar.appendToolbarItem(checkbox);
var setting = Common.settings.createSetting(name, value);
- UI.SettingsUI.bindCheckbox(checkbox.inputElement, setting);
+ setting.setTitle(Common.UIString(caption));
setting.addChangeListener(this._update, this);
+ toolbar.appendToolbarItem(new UI.ToolbarSettingCheckbox(setting));
return setting;
}

Powered by Google App Engine
This is Rietveld 408576698