DevTools: isolate settings of custom devtoools front-end.

third_party/WebKit/Source/devtools/front_end/main/Main.js

 /*
  * Copyright (C) 2006, 2007, 2008 Apple Inc.  All rights reserved.
  * Copyright (C) 2007 Matt Lilek (pewtermoose@gmail.com).
  * Copyright (C) 2009 Joseph Pecoraro
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
@@ skipping 37 matching lines @@
     if (mainTarget && mainTarget.hasBrowserCapability())
       WebInspector.targetManager.reloadPage(hard);
   }
 
   _loaded() {
     console.timeStamp('Main._loaded');
 
     if (InspectorFrontendHost.isUnderTest())
       self.runtime.useTestBase();
     Runtime.setPlatform(WebInspector.platform());
-    InspectorFrontendHost.getPreferences(this._gotPreferences.bind(this));
+    if (this._shouldUseLocalStorage())
+      WebInspector.InspectorFrontendHostStub.prototype.getPreferences.call(null, this._gotPreferences.bind(this));
+    else
+      InspectorFrontendHost.getPreferences(this._gotPreferences.bind(this));
   }
 
   /**
+   * @return {boolean}

[dgozman, 2016/11/02 17:31:57]

I'd suggest to return InspectorFrontendHostAPI from here (either real or stub)
and use it's methods consistently.

[lushnikov, 2016/11/02 18:48:59]

Done.

+   */
+  _shouldUseLocalStorage() {
+    return window.location.toString().startsWith('chrome-devtools://devtools/custom/');
+  }
+
+  /**
    * @param {!Object<string, string>} prefs
    */
   _gotPreferences(prefs) {
     console.timeStamp('Main._gotPreferences');
     this._createSettings(prefs);
     this._createAppUI();
   }
 
   /**
    * @param {!Object<string, string>} prefs
    * Note: this function is called from testSettings in Tests.js.
    */
   _createSettings(prefs) {
     this._initializeExperiments(prefs);
-    WebInspector.settings = new WebInspector.Settings(new WebInspector.SettingsStorage(
-        prefs, InspectorFrontendHost.setPreference, InspectorFrontendHost.removePreference,
-        InspectorFrontendHost.clearPreferences));
+    if (this._shouldUseLocalStorage()) {
+      WebInspector.settings = new WebInspector.Settings(new WebInspector.SettingsStorage(
+          prefs, WebInspector.InspectorFrontendHostStub.prototype.setPreference, WebInspector.InspectorFrontendHostStub.prototype.removePreference,
+          WebInspector.InspectorFrontendHostStub.prototype.clearPreferences));
+    } else {
+      WebInspector.settings = new WebInspector.Settings(new WebInspector.SettingsStorage(
+          prefs, InspectorFrontendHost.setPreference, InspectorFrontendHost.removePreference,
+          InspectorFrontendHost.clearPreferences));
+    }
 
     if (!InspectorFrontendHost.isUnderTest())
       new WebInspector.VersionController().updateVersion();
   }
 
   /**
    * @param {!Object<string, string>} prefs
    */
   _initializeExperiments(prefs) {
     Runtime.experiments.register('accessibilityInspection', 'Accessibility Inspection');
@@ skipping 958 matching lines @@
    * @override
    * @return {?Element}
    */
   settingElement() {
     return WebInspector.SettingsUI.createSettingCheckbox(
         WebInspector.UIString('Show rulers'), WebInspector.moduleSetting('showMetricsRulers'));
   }
 };
 
 new WebInspector.Main();