Backed out 5 changesets (bug 1604539) for xpc failures complanining about test_encryption.js CLOSED TREE

Backed out changeset 8455e50002a9 (bug 1604539)
Backed out changeset fb410a1bf9aa (bug 1604539)
Backed out changeset da94f7d503a1 (bug 1604539)
Backed out changeset b4b54335ee2d (bug 1604539)
Backed out changeset 866d6794bbba (bug 1604539)

--HG--
rename : devtools/client/fronts/accessibility.js => devtools/shared/fronts/accessibility.js
rename : devtools/client/fronts/addon/addons.js => devtools/shared/fronts/addon/addons.js
rename : devtools/client/fronts/addon/moz.build => devtools/shared/fronts/addon/moz.build
rename : devtools/client/fronts/addon/webextension-inspected-window.js => devtools/shared/fronts/addon/webextension-inspected-window.js
rename : devtools/client/fronts/animation.js => devtools/shared/fronts/animation.js
rename : devtools/client/fronts/array-buffer.js => devtools/shared/fronts/array-buffer.js
rename : devtools/client/fronts/changes.js => devtools/shared/fronts/changes.js
rename : devtools/client/fronts/content-viewer.js => devtools/shared/fronts/content-viewer.js
rename : devtools/client/fronts/css-properties.js => devtools/shared/fronts/css-properties.js
rename : devtools/client/fronts/descriptors/frame.js => devtools/shared/fronts/descriptors/frame.js
rename : devtools/client/fronts/descriptors/moz.build => devtools/shared/fronts/descriptors/moz.build
rename : devtools/client/fronts/descriptors/process.js => devtools/shared/fronts/descriptors/process.js
rename : devtools/client/fronts/descriptors/tab.js => devtools/shared/fronts/descriptors/tab.js
rename : devtools/client/fronts/descriptors/webextension.js => devtools/shared/fronts/descriptors/webextension.js
rename : devtools/client/fronts/device.js => devtools/shared/fronts/device.js
rename : devtools/client/fronts/environment.js => devtools/shared/fronts/environment.js
rename : devtools/client/fronts/frame.js => devtools/shared/fronts/frame.js
rename : devtools/client/fronts/framerate.js => devtools/shared/fronts/framerate.js
rename : devtools/client/fronts/highlighters.js => devtools/shared/fronts/highlighters.js
rename : devtools/client/fronts/inspector.js => devtools/shared/fronts/inspector.js
rename : devtools/client/fronts/inspector/moz.build => devtools/shared/fronts/inspector/moz.build
rename : devtools/client/fronts/inspector/rule-rewriter.js => devtools/shared/fronts/inspector/rule-rewriter.js
rename : devtools/client/fronts/layout.js => devtools/shared/fronts/layout.js
rename : devtools/client/fronts/manifest.js => devtools/shared/fronts/manifest.js
rename : devtools/client/fronts/memory.js => devtools/shared/fronts/memory.js
rename : devtools/client/fronts/moz.build => devtools/shared/fronts/moz.build
rename : devtools/client/fronts/node.js => devtools/shared/fronts/node.js
rename : devtools/client/fronts/object.js => devtools/shared/fronts/object.js
rename : devtools/client/fronts/perf.js => devtools/shared/fronts/perf.js
rename : devtools/client/fronts/performance-recording.js => devtools/shared/fronts/performance-recording.js
rename : devtools/client/fronts/performance.js => devtools/shared/fronts/performance.js
rename : devtools/client/fronts/preference.js => devtools/shared/fronts/preference.js
rename : devtools/client/fronts/property-iterator.js => devtools/shared/fronts/property-iterator.js
rename : devtools/client/fronts/reflow.js => devtools/shared/fronts/reflow.js
rename : devtools/client/fronts/responsive.js => devtools/shared/fronts/responsive.js
rename : devtools/client/fronts/root.js => devtools/shared/fronts/root.js
rename : devtools/client/fronts/screenshot.js => devtools/shared/fronts/screenshot.js
rename : devtools/client/fronts/source.js => devtools/shared/fronts/source.js
rename : devtools/client/fronts/storage.js => devtools/shared/fronts/storage.js
rename : devtools/client/fronts/string.js => devtools/shared/fronts/string.js
rename : devtools/client/fronts/styles.js => devtools/shared/fronts/styles.js
rename : devtools/client/fronts/stylesheets.js => devtools/shared/fronts/stylesheets.js
rename : devtools/client/fronts/symbol-iterator.js => devtools/shared/fronts/symbol-iterator.js
rename : devtools/client/fronts/targets/browsing-context.js => devtools/shared/fronts/targets/browsing-context.js
rename : devtools/client/fronts/targets/content-process.js => devtools/shared/fronts/targets/content-process.js
rename : devtools/client/fronts/targets/local-tab.js => devtools/shared/fronts/targets/local-tab.js
rename : devtools/client/fronts/targets/moz.build => devtools/shared/fronts/targets/moz.build
rename : devtools/client/fronts/targets/target-mixin.js => devtools/shared/fronts/targets/target-mixin.js
rename : devtools/client/fronts/targets/worker.js => devtools/shared/fronts/targets/worker.js
rename : devtools/client/fronts/thread.js => devtools/shared/fronts/thread.js
rename : devtools/client/fronts/walker.js => devtools/shared/fronts/walker.js
rename : devtools/client/fronts/webconsole.js => devtools/shared/fronts/webconsole.js
rename : devtools/client/fronts/websocket.js => devtools/shared/fronts/websocket.js
rename : devtools/client/fronts/worker/moz.build => devtools/shared/fronts/worker/moz.build
rename : devtools/client/fronts/worker/push-subscription.js => devtools/shared/fronts/worker/push-subscription.js
rename : devtools/client/fronts/worker/service-worker-registration.js => devtools/shared/fronts/worker/service-worker-registration.js
rename : devtools/client/fronts/worker/service-worker.js => devtools/shared/fronts/worker/service-worker.js

devtools/.eslintrc.js

@@ -86,12 +86,16 @@ module.exports = {
       },
     },
     {
-      // For all server and shared files, prevent requiring devtools/client
-      // modules.
-      files: ["server/**", "shared/**"],
-      // devtools-client.js is effectively a client file, but it is still loaded
-      // by test suites in devtools/shared and devtools/server.
-      excludedFiles: ["shared/client/devtools-client.js"],
+      // For all server files, prevent requiring devtools/client modules.
+      files: [
+        "server/**",
+        // The rule also applies to all `shared` files, with the exception of
+        // shared/fronts.
+        // This first pattern matches files in a shared subfolder other than "fronts".
+        "shared/!(fronts)/**",
+        // This second pattern matches files directly under shared.
+        "shared/*.**",
+      ],
       rules: {
         "mozilla/reject-some-requires": [
           "error",

devtools/client/debugger/configs/mozilla-central-mappings.js

@@ -12,7 +12,7 @@ const EXCLUDED_FILES = {
 };
 
 const whatwgUrl = `
-(() => {
+(() => { 
   importScripts("resource://devtools/client/shared/vendor/whatwg-url.js");
   return { URL }
 })()
@@ -23,7 +23,7 @@ module.exports = Object.assign(
     "./source-editor": "devtools/client/sourceeditor/editor",
     "../editor/source-editor": "devtools/client/sourceeditor/editor",
     "./test-flag": "devtools/shared/flags",
-    "./fronts-device": "devtools/client/fronts/device",
+    "./fronts-device": "devtools/shared/fronts/device",
     immutable: "devtools/client/shared/vendor/immutable",
     lodash: "devtools/client/shared/vendor/lodash",
     react: "devtools/client/shared/vendor/react",

devtools/client/framework/toolbox.js

@@ -193,7 +193,7 @@ loader.lazyRequireGetter(
 loader.lazyRequireGetter(
   this,
   "NodeFront",
-  "devtools/client/fronts/node",
+  "devtools/shared/fronts/node",
   true
 );
 

devtools/client/inspector/extensions/components/ExpressionResultView.js

@@ -25,14 +25,14 @@ const {
 loader.lazyRequireGetter(
   this,
   "LongStringFront",
-  "devtools/client/fronts/string",
+  "devtools/shared/fronts/string",
   true
 );
 
 loader.lazyRequireGetter(
   this,
   "ObjectFront",
-  "devtools/client/fronts/object",
+  "devtools/shared/fronts/object",
   true
 );
 

devtools/client/inspector/inspector.js

@@ -19,7 +19,7 @@ const Promise = require("Promise");
 loader.lazyRequireGetter(
   this,
   "initCssProperties",
-  "devtools/client/fronts/css-properties",
+  "devtools/shared/fronts/css-properties",
   true
 );
 loader.lazyRequireGetter(

devtools/client/inspector/rules/models/element-style.js

@@ -37,7 +37,7 @@ loader.lazyRequireGetter(
 loader.lazyRequireGetter(
   this,
   "isCssVariable",
-  "devtools/client/fronts/css-properties",
+  "devtools/shared/fronts/css-properties",
   true
 );
 

devtools/client/inspector/shared/test/browser_styleinspector_output-parser.js

@@ -13,7 +13,7 @@ const { CSS_PROPERTIES_DB } = require("devtools/shared/css/properties-db");
 const {
   initCssProperties,
   getCssProperties,
-} = require("devtools/client/fronts/css-properties");
+} = require("devtools/shared/fronts/css-properties");
 
 const COLOR_CLASS = "color-class";
 const URL_CLASS = "url-class";

devtools/client/memory/models.js

@@ -8,7 +8,7 @@
 "use strict";
 
 const { assert } = require("devtools/shared/DevToolsUtils");
-const { MemoryFront } = require("devtools/client/fronts/memory");
+const { MemoryFront } = require("devtools/shared/fronts/memory");
 const HeapAnalysesClient = require("devtools/shared/heapsnapshot/HeapAnalysesClient");
 const PropTypes = require("devtools/client/shared/vendor/react-prop-types");
 const {

devtools/client/moz.build

@@ -13,7 +13,6 @@ DIRS += [
     'debugger',
     'dom',
     'framework',
-    'fronts',
     'inspector',
     'jsonview',
     'locales',

devtools/client/shared/build/build-debugger.js

@@ -19,7 +19,7 @@ const mappings = {
   "./source-editor": "devtools/client/shared/sourceeditor/editor",
   "../editor/source-editor": "devtools/client/shared/sourceeditor/editor",
   "./test-flag": "devtools/shared/flags",
-  "./fronts-device": "devtools/client/fronts/device",
+  "./fronts-device": "devtools/shared/fronts/device",
   immutable: "devtools/client/shared/vendor/immutable",
   lodash: "devtools/client/shared/vendor/lodash",
   react: "devtools/client/shared/vendor/react",

devtools/client/shared/sourceeditor/css-autocompleter.js

@@ -10,7 +10,7 @@ const {
 } = require("devtools/shared/css/parsing-utils");
 const {
   getClientCssProperties,
-} = require("devtools/client/fronts/css-properties");
+} = require("devtools/shared/fronts/css-properties");
 
 /**
  * Here is what this file (+ css-parsing-utils.js) do.

devtools/client/shared/sourceeditor/editor.js

@@ -150,7 +150,7 @@ function Editor(config) {
     themeSwitching: true,
     autocomplete: false,
     autocompleteOpts: {},
-    // Expect a CssProperties object (see devtools/client/fronts/css-properties.js)
+    // Expect a CssProperties object (see devtools/shared/fronts/css-properties.js)
     cssProperties: null,
     // Set to true to prevent the search addon to be activated.
     disableSearchAddon: false,

devtools/client/shared/sourceeditor/test/head.js

@@ -15,7 +15,7 @@ Services.scriptloader.loadSubScript(
 const Editor = require("devtools/client/shared/sourceeditor/editor");
 const {
   getClientCssProperties,
-} = require("devtools/client/fronts/css-properties");
+} = require("devtools/shared/fronts/css-properties");
 
 function promiseWaitForFocus(el) {
   return new Promise(resolve => waitForFocus(resolve, el));

devtools/client/shared/test/browser_outputparser.js

@@ -6,7 +6,7 @@
 const {
   initCssProperties,
   getCssProperties,
-} = require("devtools/client/fronts/css-properties");
+} = require("devtools/shared/fronts/css-properties");
 const { CSS_PROPERTIES_DB } = require("devtools/shared/css/properties-db");
 
 add_task(async function() {

devtools/client/shared/test/xpcshell/test_rewriteDeclarations.js

@@ -4,7 +4,7 @@
 "use strict";
 
 const { require } = ChromeUtils.import("resource://devtools/shared/Loader.jsm");
-const RuleRewriter = require("devtools/client/fronts/inspector/rule-rewriter");
+const RuleRewriter = require("devtools/shared/fronts/inspector/rule-rewriter");
 const { isCssPropertyKnown } = require("devtools/server/actors/css-properties");
 
 const TEST_DATA = [

devtools/client/styleeditor/panel.js

@@ -14,7 +14,7 @@ var {
 var {
   getString,
 } = require("resource://devtools/client/styleeditor/StyleEditorUtil.jsm");
-var { initCssProperties } = require("devtools/client/fronts/css-properties");
+var { initCssProperties } = require("devtools/shared/fronts/css-properties");
 
 var StyleEditorPanel = function StyleEditorPanel(panelWin, toolbox) {
   EventEmitter.decorate(this);

devtools/client/styleeditor/test/browser_styleeditor_autocomplete.js

@@ -7,7 +7,7 @@
 const TESTCASE_URI = TEST_BASE_HTTP + "autocomplete.html";
 const MAX_SUGGESTIONS = 15;
 
-const { initCssProperties } = require("devtools/client/fronts/css-properties");
+const { initCssProperties } = require("devtools/shared/fronts/css-properties");
 
 // Test cases to test that autocompletion works correctly when enabled.
 // Format:

devtools/client/webconsole/test/browser/stub-generator-helpers.js

@@ -8,7 +8,7 @@ const { OS } = ChromeUtils.import("resource://gre/modules/osfile.jsm");
 
 const {
   getAdHocFrontOrPrimitiveGrip,
-} = require("devtools/client/fronts/object");
+} = require("devtools/shared/fronts/object");
 
 const CHROME_PREFIX = "chrome://mochitests/content/browser/";
 const STUBS_FOLDER = "devtools/client/webconsole/test/node/fixtures/stubs/";

devtools/client/webconsole/utils/object-inspector.js

@@ -29,14 +29,14 @@ loader.lazyRequireGetter(
 loader.lazyRequireGetter(
   this,
   "LongStringFront",
-  "devtools/client/fronts/string",
+  "devtools/shared/fronts/string",
   true
 );
 
 loader.lazyRequireGetter(
   this,
   "ObjectFront",
-  "devtools/client/fronts/object",
+  "devtools/shared/fronts/object",
   true
 );
 

devtools/client/webconsole/webconsole-ui.js

@@ -18,7 +18,7 @@ const { BrowserLoader } = ChromeUtils.import(
 );
 const {
   getAdHocFrontOrPrimitiveGrip,
-} = require("devtools/client/fronts/object");
+} = require("devtools/shared/fronts/object");
 
 loader.lazyRequireGetter(
   this,

devtools/client/webconsole/webconsole.js

@@ -467,7 +467,7 @@ class WebConsole {
    *
    * @param {String} string: The code you want to evaluate.
    * @param {Object} options: Options for evaluation. See evaluateJSAsync method on
-   *                          devtools/client/fronts/webconsole.js
+   *                          devtools/shared/fronts/webconsole.js
    */
   evaluateJSAsync(expression, options = {}) {
     return this.ui._commands.evaluateJSAsync(expression, options);

devtools/docs/backend/protocol.js.md

@@ -67,7 +67,7 @@ A request/reply will look like this:
     <- { from: <actorID>, greeting: "hello" }
 
 Now we can create a client side object.  We call these *front* objects and
-they typically go in `devtools/client/fronts/`.
+they typically go in `devtools/shared/fronts/`.
 
 Here's the front for the HelloActor:
 
@@ -479,7 +479,7 @@ Now you can listen to events on a front:
 If you want to modify the argument that will be passed to event listeners callbacks, you
 can use `before(eventName, fn)` in the front definition. This can only be used once for a
 given `eventName`. The `fn` function will be called before emitting the event via
-the EventEmitter API on the Front, and its return value will be passed to the event
+the EventEmitter API on the Front, and its return value will be passed to the event 
 listener callbacks. If `fn` is async, the event will only be emitted after `fn` call resolves.
 
     // In front file, most probably in the constructor:

devtools/server/tests/chrome/test_css-properties.html

@@ -14,7 +14,7 @@ Bug 1265798 - Replace inIDOMUtils.cssPropertyIsShorthand
 
 window.onload = function() {
   const { initCssProperties, getCssProperties } =
-    require("devtools/client/fronts/css-properties");
+    require("devtools/shared/fronts/css-properties");
 
   function toSortedString(array) {
     return JSON.stringify(array.sort());

devtools/server/tests/xpcshell/head_dbg.js

@@ -43,8 +43,8 @@ const { DevToolsServer: WorkerDevToolsServer } = worker.require(
   "devtools/server/devtools-server"
 );
 const { DevToolsClient } = require("devtools/shared/client/devtools-client");
-const { ObjectFront } = require("devtools/client/fronts/object");
-const { LongStringFront } = require("devtools/client/fronts/string");
+const { ObjectFront } = require("devtools/shared/fronts/object");
+const { LongStringFront } = require("devtools/shared/fronts/string");
 const { TargetFactory } = require("devtools/client/framework/target");
 
 const { addDebuggerToGlobal } = ChromeUtils.import(

devtools/server/tests/xpcshell/test_attach.js

@@ -3,10 +3,10 @@
 
 "use strict";
 
-const { ThreadFront } = require("devtools/client/fronts/thread");
+const { ThreadFront } = require("devtools/shared/fronts/thread");
 const {
   BrowsingContextTargetFront,
-} = require("devtools/client/fronts/targets/browsing-context");
+} = require("devtools/shared/fronts/targets/browsing-context");
 
 /**
  * Very naive test that checks threadClearTest helper.

devtools/server/tests/xpcshell/test_framebindings-07.js

@@ -4,7 +4,7 @@
 
 "use strict";
 
-const { EnvironmentFront } = require("devtools/client/fronts/environment");
+const { EnvironmentFront } = require("devtools/shared/fronts/environment");
 
 add_task(
   threadFrontTest(async ({ threadFront, debuggee, client }) => {

devtools/shared/client/devtools-client.js

@@ -37,7 +37,7 @@ loader.lazyRequireGetter(
 loader.lazyRequireGetter(
   this,
   "ObjectFront",
-  "devtools/client/fronts/object",
+  "devtools/shared/fronts/object",
   true
 );
 loader.lazyRequireGetter(this, "Front", "devtools/shared/protocol", true);

devtools/shared/fronts/addon/webextension-inspected-window.js

@@ -15,7 +15,7 @@ const {
 
 const {
   getAdHocFrontOrPrimitiveGrip,
-} = require("devtools/client/fronts/object");
+} = require("devtools/shared/fronts/object");
 
 /**
  * The corresponding Front object for the WebExtensionInspectedWindowActor.

devtools/shared/fronts/descriptors/frame.js

@@ -10,7 +10,7 @@ const {
 loader.lazyRequireGetter(
   this,
   "BrowsingContextTargetFront",
-  "devtools/client/fronts/targets/browsing-context",
+  "devtools/shared/fronts/targets/browsing-context",
   true
 );
 

devtools/shared/fronts/descriptors/process.js

@@ -8,10 +8,10 @@ const {
 } = require("devtools/shared/specs/descriptors/process");
 const {
   BrowsingContextTargetFront,
-} = require("devtools/client/fronts/targets/browsing-context");
+} = require("devtools/shared/fronts/targets/browsing-context");
 const {
   ContentProcessTargetFront,
-} = require("devtools/client/fronts/targets/content-process");
+} = require("devtools/shared/fronts/targets/content-process");
 const {
   FrontClassWithSpec,
   registerFront,

devtools/shared/fronts/descriptors/tab.js

@@ -8,13 +8,13 @@ const { tabDescriptorSpec } = require("devtools/shared/specs/descriptors/tab");
 loader.lazyRequireGetter(
   this,
   "BrowsingContextTargetFront",
-  "devtools/client/fronts/targets/browsing-context",
+  "devtools/shared/fronts/targets/browsing-context",
   true
 );
 loader.lazyRequireGetter(
   this,
   "LocalTabTargetFront",
-  "devtools/client/fronts/targets/local-tab",
+  "devtools/shared/fronts/targets/local-tab",
   true
 );
 const {

devtools/shared/fronts/descriptors/webextension.js

@@ -13,7 +13,7 @@ const {
 loader.lazyRequireGetter(
   this,
   "BrowsingContextTargetFront",
-  "devtools/client/fronts/targets/browsing-context",
+  "devtools/shared/fronts/targets/browsing-context",
   true
 );
 

devtools/shared/fronts/node.js

@@ -11,7 +11,7 @@ const {
   registerFront,
 } = require("devtools/shared/protocol.js");
 const { nodeSpec, nodeListSpec } = require("devtools/shared/specs/node");
-const { SimpleStringFront } = require("devtools/client/fronts/string");
+const { SimpleStringFront } = require("devtools/shared/fronts/string");
 const Services = require("Services");
 
 loader.lazyRequireGetter(

devtools/shared/fronts/object.js

@@ -9,7 +9,7 @@ const {
   FrontClassWithSpec,
   registerFront,
 } = require("devtools/shared/protocol");
-const { LongStringFront } = require("devtools/client/fronts/string");
+const { LongStringFront } = require("devtools/shared/fronts/string");
 
 /**
  * A ObjectFront is used as a front end for the ObjectActor that is

devtools/shared/fronts/performance.js

@@ -11,7 +11,7 @@ const {
 } = require("devtools/shared/protocol");
 const {
   PerformanceRecordingFront,
-} = require("devtools/client/fronts/performance-recording");
+} = require("devtools/shared/fronts/performance-recording");
 const { performanceSpec } = require("devtools/shared/specs/performance");
 
 loader.lazyRequireGetter(

devtools/shared/fronts/property-iterator.js

@@ -13,7 +13,7 @@ const {
 } = require("devtools/shared/specs/property-iterator");
 const {
   getAdHocFrontOrPrimitiveGrip,
-} = require("devtools/client/fronts/object");
+} = require("devtools/shared/fronts/object");
 
 /**
  * A PropertyIteratorFront provides a way to access to property names and

devtools/shared/fronts/root.js

@@ -14,19 +14,19 @@ loader.lazyRequireGetter(this, "getFront", "devtools/shared/protocol", true);
 loader.lazyRequireGetter(
   this,
   "TabDescriptorFront",
-  "devtools/client/fronts/descriptors/tab",
+  "devtools/shared/fronts/descriptors/tab",
   true
 );
 loader.lazyRequireGetter(
   this,
   "BrowsingContextTargetFront",
-  "devtools/client/fronts/targets/browsing-context",
+  "devtools/shared/fronts/targets/browsing-context",
   true
 );
 loader.lazyRequireGetter(
   this,
   "LocalTabTargetFront",
-  "devtools/client/fronts/targets/local-tab",
+  "devtools/shared/fronts/targets/local-tab",
   true
 );
 

devtools/shared/fronts/source.js

@@ -9,7 +9,7 @@ const {
   FrontClassWithSpec,
   registerFront,
 } = require("devtools/shared/protocol");
-const { ArrayBufferFront } = require("devtools/client/fronts/array-buffer");
+const { ArrayBufferFront } = require("devtools/shared/fronts/array-buffer");
 
 /**
  * A SourceFront provides a way to access the source text of a script.

devtools/shared/fronts/styles.js

@@ -17,7 +17,7 @@ const promise = require("promise");
 loader.lazyRequireGetter(
   this,
   "RuleRewriter",
-  "devtools/client/fronts/inspector/rule-rewriter"
+  "devtools/shared/fronts/inspector/rule-rewriter"
 );
 
 /**

devtools/shared/fronts/symbol-iterator.js

@@ -11,7 +11,7 @@ const {
 const { symbolIteratorSpec } = require("devtools/shared/specs/symbol-iterator");
 const {
   getAdHocFrontOrPrimitiveGrip,
-} = require("devtools/client/fronts/object");
+} = require("devtools/shared/fronts/object");
 
 /**
  * A SymbolIteratorFront is used as a front end for the SymbolIterator that is

devtools/shared/fronts/targets/browsing-context.js

@@ -10,7 +10,7 @@ const {
   FrontClassWithSpec,
   registerFront,
 } = require("devtools/shared/protocol");
-const { TargetMixin } = require("devtools/client/fronts/targets/target-mixin");
+const { TargetMixin } = require("devtools/shared/fronts/targets/target-mixin");
 
 class BrowsingContextTargetFront extends TargetMixin(
   FrontClassWithSpec(browsingContextTargetSpec)

devtools/shared/fronts/targets/content-process.js

@@ -10,7 +10,7 @@ const {
   FrontClassWithSpec,
   registerFront,
 } = require("devtools/shared/protocol");
-const { TargetMixin } = require("devtools/client/fronts/targets/target-mixin");
+const { TargetMixin } = require("devtools/shared/fronts/targets/target-mixin");
 
 class ContentProcessTargetFront extends TargetMixin(
   FrontClassWithSpec(contentProcessTargetSpec)

devtools/shared/fronts/targets/local-tab.js

@@ -19,7 +19,7 @@ loader.lazyRequireGetter(
 loader.lazyRequireGetter(
   this,
   "BrowsingContextTargetFront",
-  "devtools/client/fronts/targets/browsing-context",
+  "devtools/shared/fronts/targets/browsing-context",
   true
 );
 

devtools/shared/fronts/targets/target-mixin.js

@@ -165,7 +165,7 @@ function TargetMixin(parentClass) {
 
     /**
      * The following getters: isLocalTab, localTab, ... will be overriden for
-     * local tabs by some code in devtools/client/fronts/targets/local-tab.js.
+     * local tabs by some code in devtools/shared/fronts/targets/local-tab.js.
      * They are all specific to local tabs, i.e. when you are debugging a tab of
      * the current Firefox instance.
      */

devtools/shared/fronts/targets/worker.js

@@ -9,7 +9,7 @@ const {
   FrontClassWithSpec,
   registerFront,
 } = require("devtools/shared/protocol");
-const { TargetMixin } = require("devtools/client/fronts/targets/target-mixin");
+const { TargetMixin } = require("devtools/shared/fronts/targets/target-mixin");
 
 class WorkerTargetFront extends TargetMixin(
   FrontClassWithSpec(workerTargetSpec)

devtools/shared/fronts/thread.js

@@ -15,14 +15,14 @@ const { threadSpec } = require("devtools/shared/specs/thread");
 loader.lazyRequireGetter(
   this,
   "ObjectFront",
-  "devtools/client/fronts/object",
+  "devtools/shared/fronts/object",
   true
 );
-loader.lazyRequireGetter(this, "FrameFront", "devtools/client/fronts/frame");
+loader.lazyRequireGetter(this, "FrameFront", "devtools/shared/fronts/frame");
 loader.lazyRequireGetter(
   this,
   "SourceFront",
-  "devtools/client/fronts/source",
+  "devtools/shared/fronts/source",
   true
 );
 

devtools/shared/fronts/webconsole.js

@@ -5,7 +5,7 @@
 "use strict";
 
 const DevToolsUtils = require("devtools/shared/DevToolsUtils");
-const { LongStringFront } = require("devtools/client/fronts/string");
+const { LongStringFront } = require("devtools/shared/fronts/string");
 const {
   FrontClassWithSpec,
   registerFront,
@@ -13,7 +13,7 @@ const {
 const { webconsoleSpec } = require("devtools/shared/specs/webconsole");
 const {
   getAdHocFrontOrPrimitiveGrip,
-} = require("devtools/client/fronts/object");
+} = require("devtools/shared/fronts/object");
 
 /**
  * A WebConsoleFront is used as a front end for the WebConsoleActor that is

devtools/shared/moz.build

@@ -12,6 +12,7 @@ DIRS += [
     'client',
     'css',
     'discovery',
+    'fronts',
     'heapsnapshot',
     'inspector',
     'jsbeautify',

devtools/shared/protocol/tests/xpcshell/test_protocol_longstring.js

@@ -13,7 +13,7 @@ var EventEmitter = require("devtools/shared/event-emitter");
 var { LongStringActor } = require("devtools/server/actors/string");
 
 // The test implicitly relies on this.
-require("devtools/client/fronts/string");
+require("devtools/shared/fronts/string");
 
 function simpleHello() {
   return {

devtools/shared/specs/index.js

@@ -20,82 +20,82 @@ const Types = (exports.__TypesForTests = [
       "parentaccessibility",
     ],
     spec: "devtools/shared/specs/accessibility",
-    front: "devtools/client/fronts/accessibility",
+    front: "devtools/shared/fronts/accessibility",
   },
   {
     types: ["addons"],
     spec: "devtools/shared/specs/addon/addons",
-    front: "devtools/client/fronts/addon/addons",
+    front: "devtools/shared/fronts/addon/addons",
   },
   {
     types: ["webExtensionInspectedWindow"],
     spec: "devtools/shared/specs/addon/webextension-inspected-window",
-    front: "devtools/client/fronts/addon/webextension-inspected-window",
+    front: "devtools/shared/fronts/addon/webextension-inspected-window",
   },
   {
     types: ["animationplayer", "animations"],
     spec: "devtools/shared/specs/animation",
-    front: "devtools/client/fronts/animation",
+    front: "devtools/shared/fronts/animation",
   },
   {
     types: ["arraybuffer"],
     spec: "devtools/shared/specs/array-buffer",
-    front: "devtools/client/fronts/array-buffer",
+    front: "devtools/shared/fronts/array-buffer",
   },
   {
     types: ["changes"],
     spec: "devtools/shared/specs/changes",
-    front: "devtools/client/fronts/changes",
+    front: "devtools/shared/fronts/changes",
   },
   {
     types: ["contentViewer"],
     spec: "devtools/shared/specs/content-viewer",
-    front: "devtools/client/fronts/content-viewer",
+    front: "devtools/shared/fronts/content-viewer",
   },
   {
     types: ["cssProperties"],
     spec: "devtools/shared/specs/css-properties",
-    front: "devtools/client/fronts/css-properties",
+    front: "devtools/shared/fronts/css-properties",
   },
   {
     types: ["frameDescriptor"],
     spec: "devtools/shared/specs/descriptors/frame",
-    front: "devtools/client/fronts/descriptors/frame",
+    front: "devtools/shared/fronts/descriptors/frame",
   },
   {
     types: ["processDescriptor"],
     spec: "devtools/shared/specs/descriptors/process",
-    front: "devtools/client/fronts/descriptors/process",
+    front: "devtools/shared/fronts/descriptors/process",
   },
   {
     types: ["tabDescriptor"],
     spec: "devtools/shared/specs/descriptors/tab",
-    front: "devtools/client/fronts/descriptors/tab",
+    front: "devtools/shared/fronts/descriptors/tab",
   },
   {
     types: ["webExtensionDescriptor"],
     spec: "devtools/shared/specs/descriptors/webextension",
-    front: "devtools/client/fronts/descriptors/webextension",
+    front: "devtools/shared/fronts/descriptors/webextension",
   },
   {
     types: ["device"],
     spec: "devtools/shared/specs/device",
-    front: "devtools/client/fronts/device",
+    front: "devtools/shared/fronts/device",
   },
   {
     types: ["environment"],
     spec: "devtools/shared/specs/environment",
-    front: "devtools/client/fronts/environment",
+    front: "devtools/shared/fronts/environment",
   },
   {
     types: ["frame"],
     spec: "devtools/shared/specs/frame",
-    front: "devtools/client/fronts/frame",
+    front: "devtools/shared/fronts/frame",
   },
   {
     types: ["framerate"],
     spec: "devtools/shared/specs/framerate",
-    front: "devtools/client/fronts/framerate",
+    front: "devtools/shared/fronts/framerate",
   },
   /* heap snapshot has old fashion client and no front */
   {
@@ -106,27 +106,27 @@ const Types = (exports.__TypesForTests = [
   {
     types: ["highlighter", "customhighlighter"],
     spec: "devtools/shared/specs/highlighters",
-    front: "devtools/client/fronts/highlighters",
+    front: "devtools/shared/fronts/highlighters",
   },
   {
     types: ["inspector"],
     spec: "devtools/shared/specs/inspector",
-    front: "devtools/client/fronts/inspector",
+    front: "devtools/shared/fronts/inspector",
   },
   {
     types: ["flexbox", "grid", "layout"],
     spec: "devtools/shared/specs/layout",
-    front: "devtools/client/fronts/layout",
+    front: "devtools/shared/fronts/layout",
   },
   {
     types: ["manifest"],
     spec: "devtools/shared/specs/manifest",
-    front: "devtools/client/fronts/manifest",
+    front: "devtools/shared/fronts/manifest",
   },
   {
     types: ["memory"],
     spec: "devtools/shared/specs/memory",
-    front: "devtools/client/fronts/memory",
+    front: "devtools/shared/fronts/memory",
   },
   {
     types: ["netEvent"],
@@ -142,7 +142,7 @@ const Types = (exports.__TypesForTests = [
   {
     types: ["domnode", "domnodelist"],
     spec: "devtools/shared/specs/node",
-    front: "devtools/client/fronts/node",
+    front: "devtools/shared/fronts/node",
   },
   {
     types: ["obj", "object.descriptor"],
@@ -152,52 +152,52 @@ const Types = (exports.__TypesForTests = [
   {
     types: ["perf"],
     spec: "devtools/shared/specs/perf",
-    front: "devtools/client/fronts/perf",
+    front: "devtools/shared/fronts/perf",
   },
   {
     types: ["performance"],
     spec: "devtools/shared/specs/performance",
-    front: "devtools/client/fronts/performance",
+    front: "devtools/shared/fronts/performance",
   },
   {
     types: ["performance-recording"],
     spec: "devtools/shared/specs/performance-recording",
-    front: "devtools/client/fronts/performance-recording",
+    front: "devtools/shared/fronts/performance-recording",
   },
   {
     types: ["preference"],
     spec: "devtools/shared/specs/preference",
-    front: "devtools/client/fronts/preference",
+    front: "devtools/shared/fronts/preference",
   },
   {
     types: ["propertyIterator"],
     spec: "devtools/shared/specs/property-iterator",
-    front: "devtools/client/fronts/property-iterator",
+    front: "devtools/shared/fronts/property-iterator",
   },
   {
     types: ["reflow"],
     spec: "devtools/shared/specs/reflow",
-    front: "devtools/client/fronts/reflow",
+    front: "devtools/shared/fronts/reflow",
   },
   {
     types: ["responsive"],
     spec: "devtools/shared/specs/responsive",
-    front: "devtools/client/fronts/responsive",
+    front: "devtools/shared/fronts/responsive",
   },
   {
     types: ["root"],
     spec: "devtools/shared/specs/root",
-    front: "devtools/client/fronts/root",
+    front: "devtools/shared/fronts/root",
   },
   {
     types: ["screenshot"],
     spec: "devtools/shared/specs/screenshot",
-    front: "devtools/client/fronts/screenshot",
+    front: "devtools/shared/fronts/screenshot",
   },
   {
     types: ["source"],
     spec: "devtools/shared/specs/source",
-    front: "devtools/client/fronts/source",
+    front: "devtools/shared/fronts/source",
   },
   {
     types: [
@@ -209,7 +209,7 @@ const Types = (exports.__TypesForTests = [
       "storage",
     ],
     spec: "devtools/shared/specs/storage",
-    front: "devtools/client/fronts/storage",
+    front: "devtools/shared/fronts/storage",
   },
   /* longstring is special, it has a wrapper type. See its spec module */
   {
@@ -220,17 +220,17 @@ const Types = (exports.__TypesForTests = [
   {
     types: ["longstractor"],
     spec: "devtools/shared/specs/string",
-    front: "devtools/client/fronts/string",
+    front: "devtools/shared/fronts/string",
   },
   {
     types: ["pagestyle", "domstylerule"],
     spec: "devtools/shared/specs/styles",
-    front: "devtools/client/fronts/styles",
+    front: "devtools/shared/fronts/styles",
   },
   {
     types: ["mediarule", "stylesheet", "stylesheets"],
     spec: "devtools/shared/specs/stylesheets",
-    front: "devtools/client/fronts/stylesheets",
+    front: "devtools/shared/fronts/stylesheets",
   },
   {
     types: ["symbol"],
@@ -240,12 +240,12 @@ const Types = (exports.__TypesForTests = [
   {
     types: ["symbolIterator"],
     spec: "devtools/shared/specs/symbol-iterator",
-    front: "devtools/client/fronts/symbol-iterator",
+    front: "devtools/shared/fronts/symbol-iterator",
   },
   {
     types: ["browsingContextTarget"],
     spec: "devtools/shared/specs/targets/browsing-context",
-    front: "devtools/client/fronts/targets/browsing-context",
+    front: "devtools/shared/fronts/targets/browsing-context",
   },
   {
     types: ["chromeWindowTarget"],
@@ -275,42 +275,42 @@ const Types = (exports.__TypesForTests = [
   {
     types: ["workerTarget"],
     spec: "devtools/shared/specs/targets/worker",
-    front: "devtools/client/fronts/targets/worker",
+    front: "devtools/shared/fronts/targets/worker",
   },
   {
     types: ["thread"],
     spec: "devtools/shared/specs/thread",
-    front: "devtools/client/fronts/thread",
+    front: "devtools/shared/fronts/thread",
   },
   {
     types: ["domwalker"],
     spec: "devtools/shared/specs/walker",
-    front: "devtools/client/fronts/walker",
+    front: "devtools/shared/fronts/walker",
   },
   {
     types: ["console"],
     spec: "devtools/shared/specs/webconsole",
-    front: "devtools/client/fronts/webconsole",
+    front: "devtools/shared/fronts/webconsole",
   },
   {
     types: ["webSocket"],
     spec: "devtools/shared/specs/websocket",
-    front: "devtools/client/fronts/websocket",
+    front: "devtools/shared/fronts/websocket",
   },
   {
     types: ["pushSubscription"],
     spec: "devtools/shared/specs/worker/push-subscription",
-    front: "devtools/client/fronts/worker/push-subscription",
+    front: "devtools/shared/fronts/worker/push-subscription",
   },
   {
     types: ["serviceWorker"],
     spec: "devtools/shared/specs/worker/service-worker",
-    front: "devtools/client/fronts/worker/service-worker",
+    front: "devtools/shared/fronts/worker/service-worker",
   },
   {
     types: ["serviceWorkerRegistration"],
     spec: "devtools/shared/specs/worker/service-worker-registration",
-    front: "devtools/client/fronts/worker/service-worker-registration",
+    front: "devtools/shared/fronts/worker/service-worker-registration",
   },
 ]);
 

devtools/shared/webconsole/test/chrome/common.js

@@ -4,14 +4,14 @@
 
 "use strict";
 
-/* exported attachConsole, attachConsoleToTab, attachConsoleToWorker,
+/* exported ObjectFront, attachConsole, attachConsoleToTab, attachConsoleToWorker,
    closeDebugger, checkConsoleAPICalls, checkRawHeaders, runTests, nextTest, Ci, Cc,
    withActiveServiceWorker, Services, consoleAPICall */
 
 const { require } = ChromeUtils.import("resource://devtools/shared/Loader.jsm");
 const { DevToolsServer } = require("devtools/server/devtools-server");
 const { DevToolsClient } = require("devtools/shared/client/devtools-client");
-
+const { ObjectFront } = require("devtools/shared/fronts/object");
 const Services = require("Services");
 
 function initCommon() {