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

Unified Diff: third_party/WebKit/Source/devtools/front_end/resources/ServiceWorkersView.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/resources/ServiceWorkersView.js
diff --git a/third_party/WebKit/Source/devtools/front_end/resources/ServiceWorkersView.js b/third_party/WebKit/Source/devtools/front_end/resources/ServiceWorkersView.js
index bebb91a3e8ca832f058b4596b84d2a12cb18d0b7..af55aa88b121b837bbc5807555967cc5345dba60 100644
--- a/third_party/WebKit/Source/devtools/front_end/resources/ServiceWorkersView.js
+++ b/third_party/WebKit/Source/devtools/front_end/resources/ServiceWorkersView.js
@@ -18,14 +18,15 @@ Resources.ServiceWorkersView = class extends UI.VBox {
this._sections = new Map();
this._toolbar.appendToolbarItem(NetworkConditions.NetworkConditionsSelector.createOfflineToolbarCheckbox());
- var forceUpdate = new UI.ToolbarCheckbox(
- Common.UIString('Update on reload'), Common.UIString('Force update Service Worker on page reload'),
- Common.settings.createSetting('serviceWorkerUpdateOnReload', false));
+ var updateOnReloadSetting = Common.settings.createSetting('serviceWorkerUpdateOnReload', false);
+ updateOnReloadSetting.setTitle(Common.UIString('Update on reload'));
+ var forceUpdate = new UI.ToolbarSettingCheckbox(
+ updateOnReloadSetting, Common.UIString('Force update Service Worker on page reload'));
this._toolbar.appendToolbarItem(forceUpdate);
- var fallbackToNetwork = new UI.ToolbarCheckbox(
- Common.UIString('Bypass for network'),
- Common.UIString('Bypass Service Worker and load resources from the network'),
- Common.settings.createSetting('bypassServiceWorker', false));
+ var bypassServiceWorkerSetting = Common.settings.createSetting('bypassServiceWorker', false);
+ bypassServiceWorkerSetting.setTitle(Common.UIString('Bypass for network'));
+ var fallbackToNetwork = new UI.ToolbarSettingCheckbox(
+ bypassServiceWorkerSetting, Common.UIString('Bypass Service Worker and load resources from the network'));
this._toolbar.appendToolbarItem(fallbackToNetwork);
this._toolbar.appendSpacer();
this._showAllCheckbox = new UI.ToolbarCheckbox(

Powered by Google App Engine
This is Rietveld 408576698