Bug 1429421 - remove gcli code; r=jdescottes,julienw,ochameau
authoryulia <ystartsev@mozilla.com>
Thu, 06 Sep 2018 11:42:57 +0000
changeset 490811 61970ba394500a42b1602ba7717574fcecc01ea1
parent 490810 204f4f6c6fbde2a739f0371f805cd236eb746bb8
child 490812 6615b44439f9ae068611efca66cb09bb94cd3920
push id9984
push userffxbld-merge
push dateMon, 15 Oct 2018 21:07:35 +0000
treeherdermozilla-beta@183d27ea8570 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjdescottes, julienw, ochameau
bugs1429421
milestone64.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1429421 - remove gcli code; r=jdescottes,julienw,ochameau Remove remaining GCLI code Differential Revision: https://phabricator.services.mozilla.com/D3618
.eslintignore
browser/base/content/test/static/browser_all_files_referenced.js
devtools/client/debugger/test/mochitest/head.js
devtools/client/definitions.js
devtools/client/framework/devtools-browser.js
devtools/client/inspector/inspector.js
devtools/client/netmonitor/README.md
devtools/client/responsive.html/index.js
devtools/client/responsive.html/manager.js
devtools/client/responsive.html/utils/notification.js
devtools/client/shared/developer-toolbar.js
devtools/client/shared/keycodes.js
devtools/client/shared/moz.build
devtools/client/shared/telemetry.js
devtools/client/shared/test/browser.ini
devtools/client/shared/test/browser_templater_basic.js
devtools/client/themes/images/gcli_sec_bad.svg
devtools/client/themes/images/gcli_sec_good.svg
devtools/client/themes/images/gcli_sec_moderate.svg
devtools/docs/files/adding-files.md
devtools/docs/frontend/telemetry.md
devtools/server/actors/gcli.js
devtools/server/actors/highlighters.js
devtools/server/actors/moz.build
devtools/server/actors/webconsole/commands.js
devtools/server/main.js
devtools/shared/Loader.jsm
devtools/shared/fronts/gcli.js
devtools/shared/fronts/moz.build
devtools/shared/gcli/command-state.js
devtools/shared/gcli/commands/index.js
devtools/shared/gcli/commands/measure.js
devtools/shared/gcli/commands/moz.build
devtools/shared/gcli/commands/paintflashing.js
devtools/shared/gcli/commands/rulers.js
devtools/shared/gcli/commands/screenshot.js
devtools/shared/gcli/moz.build
devtools/shared/gcli/source/LICENSE
devtools/shared/gcli/source/docs/design.md
devtools/shared/gcli/source/docs/developing-gcli.md
devtools/shared/gcli/source/docs/index.md
devtools/shared/gcli/source/docs/running-tests.md
devtools/shared/gcli/source/docs/writing-commands.md
devtools/shared/gcli/source/docs/writing-tests.md
devtools/shared/gcli/source/docs/writing-types.md
devtools/shared/gcli/source/lib/gcli/cli.js
devtools/shared/gcli/source/lib/gcli/commands/clear.js
devtools/shared/gcli/source/lib/gcli/commands/commands.js
devtools/shared/gcli/source/lib/gcli/commands/context.js
devtools/shared/gcli/source/lib/gcli/commands/help.js
devtools/shared/gcli/source/lib/gcli/commands/mocks.js
devtools/shared/gcli/source/lib/gcli/commands/moz.build
devtools/shared/gcli/source/lib/gcli/commands/pref.js
devtools/shared/gcli/source/lib/gcli/commands/preflist.js
devtools/shared/gcli/source/lib/gcli/commands/test.js
devtools/shared/gcli/source/lib/gcli/connectors/connectors.js
devtools/shared/gcli/source/lib/gcli/connectors/moz.build
devtools/shared/gcli/source/lib/gcli/converters/basic.js
devtools/shared/gcli/source/lib/gcli/converters/converters.js
devtools/shared/gcli/source/lib/gcli/converters/html.js
devtools/shared/gcli/source/lib/gcli/converters/moz.build
devtools/shared/gcli/source/lib/gcli/converters/terminal.js
devtools/shared/gcli/source/lib/gcli/fields/delegate.js
devtools/shared/gcli/source/lib/gcli/fields/fields.js
devtools/shared/gcli/source/lib/gcli/fields/moz.build
devtools/shared/gcli/source/lib/gcli/fields/selection.js
devtools/shared/gcli/source/lib/gcli/index.js
devtools/shared/gcli/source/lib/gcli/l10n.js
devtools/shared/gcli/source/lib/gcli/languages/command.html
devtools/shared/gcli/source/lib/gcli/languages/command.js
devtools/shared/gcli/source/lib/gcli/languages/javascript.js
devtools/shared/gcli/source/lib/gcli/languages/languages.js
devtools/shared/gcli/source/lib/gcli/languages/moz.build
devtools/shared/gcli/source/lib/gcli/moz.build
devtools/shared/gcli/source/lib/gcli/mozui/completer.js
devtools/shared/gcli/source/lib/gcli/mozui/inputter.js
devtools/shared/gcli/source/lib/gcli/mozui/moz.build
devtools/shared/gcli/source/lib/gcli/mozui/tooltip.js
devtools/shared/gcli/source/lib/gcli/settings.js
devtools/shared/gcli/source/lib/gcli/system.js
devtools/shared/gcli/source/lib/gcli/types/array.js
devtools/shared/gcli/source/lib/gcli/types/boolean.js
devtools/shared/gcli/source/lib/gcli/types/command.js
devtools/shared/gcli/source/lib/gcli/types/date.js
devtools/shared/gcli/source/lib/gcli/types/delegate.js
devtools/shared/gcli/source/lib/gcli/types/file.js
devtools/shared/gcli/source/lib/gcli/types/fileparser.js
devtools/shared/gcli/source/lib/gcli/types/javascript.js
devtools/shared/gcli/source/lib/gcli/types/moz.build
devtools/shared/gcli/source/lib/gcli/types/node.js
devtools/shared/gcli/source/lib/gcli/types/number.js
devtools/shared/gcli/source/lib/gcli/types/resource.js
devtools/shared/gcli/source/lib/gcli/types/selection.js
devtools/shared/gcli/source/lib/gcli/types/setting.js
devtools/shared/gcli/source/lib/gcli/types/string.js
devtools/shared/gcli/source/lib/gcli/types/types.js
devtools/shared/gcli/source/lib/gcli/types/union.js
devtools/shared/gcli/source/lib/gcli/types/url.js
devtools/shared/gcli/source/lib/gcli/ui/focus.js
devtools/shared/gcli/source/lib/gcli/ui/history.js
devtools/shared/gcli/source/lib/gcli/ui/intro.js
devtools/shared/gcli/source/lib/gcli/ui/menu.css
devtools/shared/gcli/source/lib/gcli/ui/menu.html
devtools/shared/gcli/source/lib/gcli/ui/menu.js
devtools/shared/gcli/source/lib/gcli/ui/moz.build
devtools/shared/gcli/source/lib/gcli/ui/view.js
devtools/shared/gcli/source/lib/gcli/util/domtemplate.js
devtools/shared/gcli/source/lib/gcli/util/fileparser.js
devtools/shared/gcli/source/lib/gcli/util/filesystem.js
devtools/shared/gcli/source/lib/gcli/util/host.js
devtools/shared/gcli/source/lib/gcli/util/l10n.js
devtools/shared/gcli/source/lib/gcli/util/legacy.js
devtools/shared/gcli/source/lib/gcli/util/moz.build
devtools/shared/gcli/source/lib/gcli/util/prism.js
devtools/shared/gcli/source/lib/gcli/util/spell.js
devtools/shared/gcli/source/lib/gcli/util/util.js
devtools/shared/gcli/templater.js
devtools/shared/locales/en-US/gcli.properties
devtools/shared/locales/en-US/gclicommands.properties
devtools/shared/moz.build
devtools/shared/preferences/devtools-shared.js
devtools/shared/screenshot/capture.js
devtools/shared/specs/gcli.js
devtools/shared/specs/index.js
devtools/shared/specs/moz.build
toolkit/components/telemetry/Events.yaml
toolkit/components/telemetry/Histograms.json
toolkit/components/telemetry/histogram-whitelists.json
--- a/.eslintignore
+++ b/.eslintignore
@@ -130,17 +130,16 @@ devtools/shared/preferences/**
 devtools/startup/preferences/devtools-startup.js
 
 # Ignore devtools generated code
 devtools/shared/css/generated/properties-db.js
 
 # Ignore devtools third-party libs
 devtools/shared/jsbeautify/*
 devtools/shared/acorn/*
-devtools/shared/gcli/source/*
 devtools/shared/node-properties/*
 devtools/shared/pretty-fast/*
 devtools/shared/sourcemap/*
 devtools/shared/sprintfjs/*
 devtools/shared/qrcode/decoder/*
 devtools/shared/qrcode/encoder/*
 devtools/client/inspector/markup/test/lib_*
 devtools/client/jsonview/lib/require.js
--- a/browser/base/content/test/static/browser_all_files_referenced.js
+++ b/browser/base/content/test/static/browser_all_files_referenced.js
@@ -404,24 +404,23 @@ function parseCodeFile(fileUri) {
         let match = line.match("(?:src|href)=[\"']([^$&\"']+)");
         if (match && match[1]) {
           let url = Services.io.newURI(match[1], null, fileUri).spec;
           addCodeReference(convertToCodeURI(url), fileUri);
         }
 
         if (isDevtools) {
           let rules = [
-            ["gcli", "resource://devtools/shared/gcli/source/lib/gcli"],
             ["devtools/client/locales", "chrome://devtools/locale"],
             ["devtools/shared/locales", "chrome://devtools-shared/locale"],
             ["devtools/shared/platform", "resource://devtools/shared/platform/chrome"],
             ["devtools", "resource://devtools"],
           ];
 
-          match = line.match(/["']((?:devtools|gcli)\/[^\\#"']+)["']/);
+          match = line.match(/["']((?:devtools)\/[^\\#"']+)["']/);
           if (match && match[1]) {
             let path = match[1];
             for (let rule of rules) {
               if (path.startsWith(rule[0] + "/")) {
                 path = path.replace(rule[0], rule[1]);
                 if (!/\.(properties|js|jsm|json|css)$/.test(path))
                   path += ".js";
                 addCodeReference(path, fileUri);
--- a/devtools/client/debugger/test/mochitest/head.js
+++ b/devtools/client/debugger/test/mochitest/head.js
@@ -57,17 +57,16 @@ registerCleanupFunction(async function()
   info("Forcing GC/CC after debugger test.");
   await new Promise(resolve => {
     Cu.forceGC();
     Cu.forceCC();
     Cu.schedulePreciseGC(resolve);
   });
 });
 
-// Import the GCLI test helper
 var testDir = gTestPath.substr(0, gTestPath.lastIndexOf("/"));
 testDir = testDir.replace(/\/\//g, "/");
 testDir = testDir.replace("chrome:/mochitest", "chrome://mochitest");
 
 function addWindow(aUrl) {
   info("Adding window: " + aUrl);
   return promise.resolve(getChromeWindow(window.open(aUrl)));
 }
--- a/devtools/client/definitions.js
+++ b/devtools/client/definitions.js
@@ -542,18 +542,17 @@ exports.ToolboxButtons = [
   },
   { id: "command-button-responsive",
     description: l10n("toolbox.buttons.responsive",
                       osString == "Darwin" ? "Cmd+Opt+M" : "Ctrl+Shift+M"),
     isTargetSupported: target => target.isLocalTab,
     onClick(event, toolbox) {
       const tab = toolbox.target.tab;
       const browserWindow = tab.ownerDocument.defaultView;
-      ResponsiveUIManager.handleGcliCommand(browserWindow, tab,
-        "resize toggle", null);
+      ResponsiveUIManager.toggle(browserWindow, tab, { trigger: "toolbox" });
     },
     isChecked(toolbox) {
       if (!toolbox.target.tab) {
         return false;
       }
       return ResponsiveUIManager.isActiveForTab(toolbox.target.tab);
     },
     setup(toolbox, onChange) {
--- a/devtools/client/framework/devtools-browser.js
+++ b/devtools/client/framework/devtools-browser.js
@@ -122,28 +122,28 @@ var gDevToolsBrowser = exports.gDevTools
       toggleMenuItem("menu_devtools_replayExecution", recordReplayEnabled);
     } catch (e) {
       // devtools.recordreplay.enabled only exists on certain platforms.
     }
   },
 
   /**
    * This function makes sure that the "devtoolstheme" attribute is set on the browser
-   * window to make it possible to change colors on elements in the browser (like gcli,
-   * or the splitter between the toolbox and web content).
+   * window to make it possible to change colors on elements in the browser (like the
+   * splitter between the toolbox and web content).
    */
   updateDevtoolsThemeAttribute(win) {
     // Set an attribute on root element of each window to make it possible
     // to change colors based on the selected devtools theme.
     let devtoolsTheme = Services.prefs.getCharPref("devtools.theme");
     if (devtoolsTheme != "dark") {
       devtoolsTheme = "light";
     }
 
-    // Style gcli and the splitter between the toolbox and page content.  This used to
+    // Style the splitter between the toolbox and page content.  This used to
     // set the attribute on the browser's root node but that regressed tpaint:
     // bug 1331449.
     win.document.getElementById("browser-bottombox")
        .setAttribute("devtoolstheme", devtoolsTheme);
     win.document.getElementById("appcontent")
        .setAttribute("devtoolstheme", devtoolsTheme);
   },
 
--- a/devtools/client/inspector/inspector.js
+++ b/devtools/client/inspector/inspector.js
@@ -2317,17 +2317,17 @@ Inspector.prototype = {
 
     this.telemetry.scalarSet("devtools.copy.xpath.opened", 1);
     this.selection.nodeFront.getXPath().then(path => {
       clipboardHelper.copyString(path);
     }).catch(console.error);
   },
 
   /**
-   * Initiate gcli screenshot command on selected node.
+   * Initiate screenshot command on selected node.
    */
   async screenshotNode() {
     // Bug 1332936 - it's possible to call `screenshotNode` while the BoxModel highlighter
     // is still visible, therefore showing it in the picture.
     // To avoid that, we have to hide it before taking the screenshot. The `hideBoxModel`
     // will do that, calling `hide` for the highlighter only if previously shown.
     await this.highlighter.hideBoxModel();
 
--- a/devtools/client/netmonitor/README.md
+++ b/devtools/client/netmonitor/README.md
@@ -1,91 +1,16 @@
 # Network Monitor
 
 The Network Monitor (netmonitor) shows you all the network requests Firefox makes (for example, when a page is loaded or when an XMLHttpRequest is performed) , how long each request takes, and details of each request. You can edit the method, query, header and resend the request as well. Read [MDN](https://developer.mozilla.org/en-US/docs/Tools/Network_Monitor) to learn all the features and how to use the tool.
 
 ## Prerequisite
 
 If you want to build the Network Monitor inside of the DevTools toolbox (Firefox Devtools Panels), follow the [simple Firefox build](https://developer.mozilla.org/docs/Mozilla/Developer_guide/Build_Instructions/Simple_Firefox_build) document in MDN. Start your compiled firefox and open the Firefox developer tool, you can see the Network Monitor inside.
 
-If you would like to run the Network Monitor in the browser tab (experimental), you need following packages:
-
-* [node](https://nodejs.org/) >= 6.9.x JavaScript runtime.
-* [yarn](https://yarnpkg.com/docs/install) >= 0.21.x the package dependency management tool.
-* [Firefox](https://www.mozilla.org/firefox/new/) any version or build from the source code.
-
-## Quick Setup
-
-Navigate to the `mozilla-central/devtools/client/netmonitor` folder with your terminal.
-Note that this folder is available after `mozilla-central` was cloned in order to get a local copy of the repository. Then run the following commands:
-
-```bash
-# Install packages
-yarn install
-
-# Create a dev server instance for hosting netmonitor on browser
-yarn start
-
-# Run firefox
-firefox http://localhost:8000 --start-debugger-server 6080
-```
-
-Then open `localhost:8000` in any browser to see all tabs in Firefox.
-
-### More detailed setup
-
-Instead of running command to open a new Firefox window like
-
-```
-firefox http://localhost:8000 --start-debugger-server 6080
-```
-
-If you have an opened Firefox browser, you can manually configure Firefox via type `about:config` in Firefox URL field, grant the warning to access preferences. And set these two preferences:
-
-* disable `devtools.debugger.prompt-connection` to remove the connection prompt.
-* enable `devtools.debugger.remote-enabled` to allow remote debugging a browser tab via the Mozilla remote debugging protocol (RDP)
-
-Go to the Web Developer menu in Firefox and select [Developer Toolbar](https://developer.mozilla.org/docs/Tools/GCLI). Run the command
-
-`listen 6080 mozilla-rdp`
-
-The command will make Firefox act as a remote debugging server.
-
-Run the command
-
-`yarn start`
-
-Then open `localhost:8000` in any browser to see all tabs in Firefox.
-
-### How it works
-
-The Network Monitor uses [webpack](https://webpack.js.org/) and several packages from [devtools-core](https://github.com/devtools-html/devtools-core) to run the Network Monitor as a normal web page. The Network Monitor uses [Mozilla remote debugging protocol](http://searchfox.org/mozilla-central/source/devtools/docs/backend/protocol.md) to fetch result and execute commands from Firefox browser.
-
-![](https://hacks.mozilla.org/files/2017/06/image4.png)
-
-Open `localhost:8000` in any browser to see the [launchpad](https://github.com/devtools-html/devtools-core/tree/master/packages/devtools-launchpad) interface. Devtools Launchpad will communicate with Firefox (the remote debugging server) and list all opened tabs from Firefox. Click one of the browser tab entry, now you can see the Network Monitor runs in a browser tab.
-
-### Develop with related modules
-
-When working on make the Network Monitor running in the browser tab, you may need to work on external modules. Besides the third party modules, here are modules required for the Network Monitor and is hosted under `devtools-html` (modules shared across Devtools):
-
-* [devtools-config](https://github.com/devtools-html/devtools-core/blob/master/packages/devtools-config/#readme) config used in dev server
-* [devtools-launchpad](https://github.com/devtools-html/devtools-core/blob/master/packages/devtools-launchpad/#readme) provide the dev server, landing page and the bootstrap functions to run devtools in the browser tab.
-* [devtools-modules](https://github.com/devtools-html/devtools-core/blob/master/packages/devtools-modules/#readme) Devtools shared and shim modules.
-* [devtools-source-editor](https://github.com/devtools-html/devtools-core/blob/master/packages/devtools-source-editor/#readme) Source Editor component.
-* [devtools-reps](https://github.com/devtools-html/debugger.html/blob/master/packages/devtools-reps/#readme) remote object formatter for variables representation.
-
-Do `yarn link` modules in related module directory, then do `yarn link [module-name]` after `yarn install` modules.
-
-## Code Structure
-
-Top level files are used to launch the Network Monitor inside of the DevTools toolbox or run in the browser tab (experimental). The Network Monitor source is mainly located in the `src/` folder, the same code base is used to run in both environments.
-
-We prefer use web standard API instead of FIrefox specific API, to make the Network Monitor can be opened in any browser tab.
-
 ### Run inside of the DevTools toolbox
 
 Files used to run the Network Monitor inside of the DevTools toolbox.
 
 * `panel.js` called by devtools toolbox to launch the Network Monitor panel.
 * `index.html` panel UI and launch scripts.
 * `src/connector/` wrap function call for Browser specific API. Current support Firefox and Chrome(experimental).
 
--- a/devtools/client/responsive.html/index.js
+++ b/devtools/client/responsive.html/index.js
@@ -132,17 +132,17 @@ window.getViewportSize = () => {
     return null;
   }
 
   const { width, height } = viewports[0];
   return { width, height };
 };
 
 /**
- * Called by manager.js to set viewport size from tests, GCLI, etc.
+ * Called by manager.js to set viewport size from tests, etc.
  */
 window.setViewportSize = ({ width, height }) => {
   try {
     bootstrap.dispatch(resizeViewport(0, width, height));
   } catch (e) {
     console.error(e);
   }
 };
--- a/devtools/client/responsive.html/manager.js
+++ b/devtools/client/responsive.html/manager.js
@@ -48,17 +48,17 @@ const ResponsiveUIManager = exports.Resp
    *
    * @param window
    *        The main browser chrome window.
    * @param tab
    *        The browser tab.
    * @param options
    *        Other options associated with toggling.  Currently includes:
    *        - `trigger`: String denoting the UI entry point, such as:
-   *          - `command`:  GCLI command bar or toolbox button
+   *          - `toolbox`:  Toolbox Button
    *          - `menu`:     Web Developer menu item
    *          - `shortcut`: Keyboard shortcut
    * @return Promise
    *         Resolved when the toggling has completed.  If the UI has opened,
    *         it is resolved to the ResponsiveUI instance for this tab.  If the
    *         the UI has closed, there is no resolution value.
    */
   toggle(window, tab, options = {}) {
@@ -73,17 +73,17 @@ const ResponsiveUIManager = exports.Resp
    *
    * @param window
    *        The main browser chrome window.
    * @param tab
    *        The browser tab.
    * @param options
    *        Other options associated with opening.  Currently includes:
    *        - `trigger`: String denoting the UI entry point, such as:
-   *          - `command`:  GCLI command bar or toolbox button
+   *          - `toolbox`:  Toolbox Button
    *          - `menu`:     Web Developer menu item
    *          - `shortcut`: Keyboard shortcut
    * @return Promise
    *         Resolved to the ResponsiveUI instance for this tab when opening is
    *         complete.
    */
   async openIfNeeded(window, tab, options = {}) {
     if (!tab.linkedBrowser.isRemoteBrowser) {
@@ -130,17 +130,17 @@ const ResponsiveUIManager = exports.Resp
    *
    * @param window
    *        The main browser chrome window.
    * @param tab
    *        The browser tab.
    * @param options
    *        Other options associated with closing.  Currently includes:
    *        - `trigger`: String denoting the UI entry point, such as:
-   *          - `command`:  GCLI command bar or toolbox button
+   *          - `toolbox`:  Toolbox Button
    *          - `menu`:     Web Developer menu item
    *          - `shortcut`: Keyboard shortcut
    *        - `reason`: String detailing the specific cause for closing
    * @return Promise
    *         Resolved (with no value) when closing is complete.
    */
   async closeIfNeeded(window, tab, options = {}) {
     if (this.isActiveForTab(tab)) {
@@ -207,49 +207,16 @@ const ResponsiveUIManager = exports.Resp
    *        The browser tab.
    * @return ResponsiveUI
    *         The UI instance for this tab.
    */
   getResponsiveUIForTab(tab) {
     return this.activeTabs.get(tab);
   },
 
-  /**
-   * Handle GCLI commands.
-   *
-   * @param window
-   *        The main browser chrome window.
-   * @param tab
-   *        The browser tab.
-   * @param command
-   *        The GCLI command name.
-   * @param args
-   *        The GCLI command arguments.
-   */
-  handleGcliCommand(window, tab, command, args) {
-    let completed;
-    switch (command) {
-      case "resize to":
-        completed = this.openIfNeeded(window, tab, { trigger: "command" });
-        this.activeTabs.get(tab).setViewportSize(args);
-        break;
-      case "resize on":
-        completed = this.openIfNeeded(window, tab, { trigger: "command" });
-        break;
-      case "resize off":
-        completed = this.closeIfNeeded(window, tab, { trigger: "command" });
-        break;
-      case "resize toggle":
-        completed = this.toggle(window, tab, { trigger: "command" });
-        break;
-      default:
-    }
-    completed.catch(console.error);
-  },
-
   handleMenuCheck({target}) {
     ResponsiveUIManager.setMenuCheckFor(target);
   },
 
   initMenuCheckListenerFor(window) {
     const { tabContainer } = window.gBrowser;
     tabContainer.addEventListener("TabSelect", this.handleMenuCheck);
   },
@@ -267,25 +234,23 @@ const ResponsiveUIManager = exports.Resp
     const menu = window.document.getElementById("menu_responsiveUI");
     if (menu) {
       menu.setAttribute("checked", this.isActiveForTab(tab));
     }
   },
 
   showRemoteOnlyNotification(window, tab, { trigger } = {}) {
     showNotification(window, tab, {
-      command: trigger == "command",
+      toolboxButton: trigger == "toolbox",
       msg: l10n.getStr("responsive.remoteOnly"),
       priority: PriorityLevels.PRIORITY_CRITICAL_MEDIUM,
     });
   },
 };
 
-// GCLI commands in ./commands.js listen for events from this object to know
-// when the UI for a tab has opened or closed.
 EventEmitter.decorate(ResponsiveUIManager);
 
 /**
  * ResponsiveUI manages the responsive design tool for a specific tab.  The
  * actual tool itself lives in a separate chrome:// document that is loaded into
  * the tab upon opening responsive design.  This object acts a helper to
  * integrate the tool into the surrounding browser UI as needed.
  */
@@ -684,17 +649,17 @@ ResponsiveUI.prototype = {
   /**
    * Helper for tests. Assumes a single viewport for now.
    */
   getViewportSize() {
     return this.toolWindow.getViewportSize();
   },
 
   /**
-   * Helper for tests, GCLI, etc. Assumes a single viewport for now.
+   * Helper for tests, etc. Assumes a single viewport for now.
    */
   async setViewportSize(size) {
     await this.inited;
     this.toolWindow.setViewportSize(size);
   },
 
   /**
    * Helper for tests/reloading the viewport. Assumes a single viewport for now.
--- a/devtools/client/responsive.html/utils/notification.js
+++ b/devtools/client/responsive.html/utils/notification.js
@@ -12,29 +12,29 @@ loader.lazyRequireGetter(this, "gDevTool
  * a toolbox is currently open for this tab.
  *
  * @param window
  *        The main browser chrome window.
  * @param tab
  *        The browser tab.
  * @param options
  *        Other options associated with opening.  Currently includes:
- *        - `command`: Whether initiated via GCLI command bar or toolbox button
+ *        - `toolbox`: Whether initiated via toolbox button
  *        - `msg`: String to show in the notification
  *        - `priority`: Priority level for the notification, which affects the icon and
  *                      overall appearance.
  */
-function showNotification(window, tab, { command, msg, priority } = {}) {
+function showNotification(window, tab, { toolboxButton, msg, priority } = {}) {
   // Default to using the browser's per-tab notification box
   let nbox = window.gBrowser.getNotificationBox(tab.linkedBrowser);
 
-  // If opening was initiated by GCLI command bar or toolbox button, check for an open
+  // If opening was initiated by a toolbox button, check for an open
   // toolbox for the tab.  If one exists, use the toolbox's notification box so that the
   // message is placed closer to the action taken by the user.
-  if (command) {
+  if (toolboxButton) {
     const target = TargetFactory.forTab(tab);
     const toolbox = gDevTools.getToolbox(target);
     if (toolbox) {
       nbox = toolbox.notificationBox;
     }
   }
 
   const value = "devtools-responsive";
deleted file mode 100644
--- a/devtools/client/shared/developer-toolbar.js
+++ /dev/null
@@ -1,103 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-"use strict";
-
-const promise = require("promise");
-
-loader.lazyRequireGetter(this, "gcliInit", "devtools/shared/gcli/commands/index");
-
-/**
- * A collection of utilities to help working with commands
- */
-var CommandUtils = {
-  /**
-   * Caches requisitions created when calling executeOnTarget:
-   * Target => Requisition Promise
-   */
-  _requisitions: new WeakMap(),
-
-  /**
-   * Utility to execute a command string on a given target
-   */
-  async executeOnTarget(target, command) {
-    let requisitionPromise = this._requisitions.get(target);
-    if (!requisitionPromise) {
-      requisitionPromise = this.createRequisition(target, {
-        environment: CommandUtils.createEnvironment({ target }, "target")
-      });
-      // Store the promise to avoid races by storing the promise immediately
-      this._requisitions.set(target, requisitionPromise);
-    }
-    const requisition = await requisitionPromise;
-    requisition.updateExec(command);
-  },
-
-  /**
-   * Utility to ensure that things are loaded in the correct order
-   */
-  createRequisition: function(target, options) {
-    if (!gcliInit) {
-      return promise.reject("Unable to load gcli");
-    }
-    return gcliInit.getSystem(target).then(system => {
-      const Requisition = require("gcli/cli").Requisition;
-      return new Requisition(system, options);
-    });
-  },
-
-  /**
-   * Destroy the remote side of the requisition as well as the local side
-   */
-  destroyRequisition: function(requisition, target) {
-    requisition.destroy();
-    gcliInit.releaseSystem(target);
-  },
-
-  /**
-   * A helper function to create the environment object that is passed to
-   * GCLI commands.
-   * @param targetContainer An object containing a 'target' property which
-   * reflects the current debug target
-   */
-  createEnvironment: function(container, targetProperty = "target") {
-    if (!container[targetProperty].toString ||
-        !/TabTarget/.test(container[targetProperty].toString())) {
-      throw new Error("Missing target");
-    }
-
-    return {
-      get target() {
-        if (!container[targetProperty].toString ||
-            !/TabTarget/.test(container[targetProperty].toString())) {
-          throw new Error("Removed target");
-        }
-
-        return container[targetProperty];
-      },
-
-      get chromeWindow() {
-        return this.target.tab.ownerDocument.defaultView;
-      },
-
-      get chromeDocument() {
-        return this.target.tab.ownerDocument.defaultView.document;
-      },
-
-      get window() {
-        // throw new
-        //    Error("environment.window is not available in runAt:client commands");
-        return this.chromeWindow.gBrowser.contentWindowAsCPOW;
-      },
-
-      get document() {
-        // throw new
-        //    Error("environment.document is not available in runAt:client commands");
-        return this.chromeWindow.gBrowser.contentDocumentAsCPOW;
-      }
-    };
-  },
-};
-
-exports.CommandUtils = CommandUtils;
--- a/devtools/client/shared/keycodes.js
+++ b/devtools/client/shared/keycodes.js
@@ -1,28 +1,24 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 "use strict";
 
-// This was copied (and slightly modified) from
-// devtools/shared/gcli/source/lib/gcli/util/util.js, which in turn
-// says:
-
 /**
  * Keyboard handling is a mess. http://unixpapa.com/js/key.html
  * It would be good to use DOM L3 Keyboard events,
  * http://www.w3.org/TR/2010/WD-DOM-Level-3-Events-20100907/#events-keyboardevents
  * however only Webkit supports them, and there isn't a shim on Modernizr:
  * https://github.com/Modernizr/Modernizr/wiki/HTML5-Cross-browser-Polyfills
  * and when the code that uses this KeyEvent was written, nothing was clear,
  * so instead, we're using this unmodern shim:
  * http://stackoverflow.com/questions/5681146/chrome-10-keyevent-or-something-similar-to-firefoxs-keyevent
- * See BUG 664991: GCLI's keyboard handling should be updated to use DOM-L3
+ * See BUG 664991: keyboard handling should be updated to use DOM-L3
  * https://bugzilla.mozilla.org/show_bug.cgi?id=664991
  */
 
 exports.KeyCodes = {
   DOM_VK_CANCEL: 3,
   DOM_VK_HELP: 6,
   DOM_VK_BACK_SPACE: 8,
   DOM_VK_TAB: 9,
@@ -131,16 +127,13 @@ exports.KeyCodes = {
   DOM_VK_PERIOD: 190,
   DOM_VK_SLASH: 191,
   DOM_VK_BACK_QUOTE: 192,
   DOM_VK_OPEN_BRACKET: 219,
   DOM_VK_BACK_SLASH: 220,
   DOM_VK_CLOSE_BRACKET: 221,
   DOM_VK_QUOTE: 222,
   DOM_VK_META: 224,
-
-  // A few that did not appear in gcli, but that are apparently used
-  // in devtools.
   DOM_VK_COLON: 58,
   DOM_VK_VOLUME_MUTE: 181,
   DOM_VK_VOLUME_DOWN: 182,
   DOM_VK_VOLUME_UP: 183,
 };
--- a/devtools/client/shared/moz.build
+++ b/devtools/client/shared/moz.build
@@ -20,17 +20,16 @@ DIRS += [
 ]
 
 DevToolsModules(
     'autocomplete-popup.js',
     'browser-loader.js',
     'css-angle.js',
     'curl.js',
     'demangle.js',
-    'developer-toolbar.js',
     'devices.js',
     'DOMHelpers.jsm',
     'enum.js',
     'file-saver.js',
     'focus.js',
     'getjson.js',
     'inplace-editor.js',
     'key-shortcuts.js',
--- a/devtools/client/shared/telemetry.js
+++ b/devtools/client/shared/telemetry.js
@@ -667,17 +667,16 @@ function getChartsFromToolId(id) {
   if (id === "NEWANIMATIONINSPECTOR") {
     id = "ANIMATIONINSPECTOR";
   }
 
   switch (id) {
     case "ABOUTDEBUGGING":
     case "BROWSERCONSOLE":
     case "CANVASDEBUGGER":
-    case "DEVELOPERTOOLBAR":
     case "DOM":
     case "INSPECTOR":
     case "JSBROWSERDEBUGGER":
     case "JSDEBUGGER":
     case "JSPROFILER":
     case "MEMORY":
     case "NETMONITOR":
     case "OPTIONS":
--- a/devtools/client/shared/test/browser.ini
+++ b/devtools/client/shared/test/browser.ini
@@ -193,15 +193,14 @@ skip-if = !e10s || os == "win" # RDM onl
 [browser_telemetry_toolboxtabs_jsprofiler.js]
 [browser_telemetry_toolboxtabs_netmonitor.js]
 [browser_telemetry_toolboxtabs_options.js]
 [browser_telemetry_toolboxtabs_shadereditor.js]
 [browser_telemetry_toolboxtabs_storage.js]
 [browser_telemetry_toolboxtabs_styleeditor.js]
 [browser_telemetry_toolboxtabs_webaudioeditor.js]
 [browser_telemetry_toolboxtabs_webconsole.js]
-[browser_templater_basic.js]
 [browser_treeWidget_basic.js]
 [browser_treeWidget_keyboard_interaction.js]
 [browser_treeWidget_mouse_interaction.js]
 [browser_devices.js]
 skip-if = verify
 [browser_theme_switching.js]
deleted file mode 100644
--- a/devtools/client/shared/test/browser_templater_basic.js
+++ /dev/null
@@ -1,287 +0,0 @@
-/* Any copyright is dedicated to the Public Domain.
-   http://creativecommons.org/publicdomain/zero/1.0/ */
-
-"use strict";
-
-// Tests that the DOM Template engine works properly
-
-/*
- * These tests run both in Mozilla/Mochitest and plain browsers (as does
- * domtemplate)
- * We should endevour to keep the source in sync.
- */
-
-const {template} = require("devtools/shared/gcli/templater");
-
-const TEST_URI = TEST_URI_ROOT + "doc_templater_basic.html";
-
-var test = async function() {
-  await addTab("about:blank");
-  const [host,, doc] = await createHost("bottom", TEST_URI);
-
-  info("Starting DOM Templater Tests");
-  runTest(0, host, doc);
-};
-
-function runTest(index, host, doc) {
-  const options = tests[index] = tests[index]();
-  const holder = doc.createElement("div");
-  holder.id = options.name;
-  const body = doc.body;
-  body.appendChild(holder);
-  // eslint-disable-next-line no-unsanitized/property
-  holder.innerHTML = options.template;
-
-  info("Running " + options.name);
-  template(holder, options.data, options.options);
-
-  if (typeof options.result == "string") {
-    is(holder.innerHTML, options.result, options.name);
-  } else {
-    ok(holder.innerHTML.match(options.result) != null,
-       options.name + " result='" + holder.innerHTML + "'");
-  }
-
-  if (options.also) {
-    options.also(options);
-  }
-
-  function runNextTest() {
-    index++;
-    if (index < tests.length) {
-      runTest(index, host, doc);
-    } else {
-      finished(host);
-    }
-  }
-
-  if (options.later) {
-    const ais = is.bind(this);
-
-    function createTester(testHolder, testOptions) {
-      return () => {
-        ais(testHolder.innerHTML, testOptions.later, testOptions.name + " later");
-        runNextTest();
-      };
-    }
-
-    executeSoon(createTester(holder, options));
-  } else {
-    runNextTest();
-  }
-}
-
-function finished(host) {
-  host.destroy();
-  gBrowser.removeCurrentTab();
-  info("Finishing DOM Templater Tests");
-  tests = null;
-  finish();
-}
-
-/**
- * Why have an array of functions that return data rather than just an array
- * of the data itself? Some of these tests contain calls to delayReply() which
- * sets up async processing using executeSoon(). Since the execution of these
- * tests is asynchronous, the delayed reply will probably arrive before the
- * test is executed, making the test be synchronous. So we wrap the data in a
- * function so we only set it up just before we use it.
- */
-var tests = [
-  () => ({
-    name: "simpleNesting",
-    template: '<div id="ex1">${nested.value}</div>',
-    data: { nested: { value: "pass 1" } },
-    result: '<div id="ex1">pass 1</div>'
-  }),
-
-  () => ({
-    name: "returnDom",
-    template: '<div id="ex2">${__element.ownerDocument.createTextNode(\'pass 2\')}</div>',
-    options: { allowEval: true },
-    data: {},
-    result: '<div id="ex2">pass 2</div>'
-  }),
-
-  () => ({
-    name: "srcChange",
-    template: '<img _src="${fred}" id="ex3">',
-    data: { fred: "green.png" },
-    result: /<img( id="ex3")? src="green.png"( id="ex3")?>/
-  }),
-
-  () => ({
-    name: "ifTrue",
-    template: '<p if="${name !== \'jim\'}">hello ${name}</p>',
-    options: { allowEval: true },
-    data: { name: "fred" },
-    result: "<p>hello fred</p>"
-  }),
-
-  () => ({
-    name: "ifFalse",
-    template: '<p if="${name !== \'jim\'}">hello ${name}</p>',
-    options: { allowEval: true },
-    data: { name: "jim" },
-    result: ""
-  }),
-
-  () => ({
-    name: "simpleLoop",
-    template: '<p foreach="index in ${[ 1, 2, 3 ]}">${index}</p>',
-    options: { allowEval: true },
-    data: {},
-    result: "<p>1</p><p>2</p><p>3</p>"
-  }),
-
-  () => ({
-    name: "loopElement",
-    template: '<loop foreach="i in ${array}">${i}</loop>',
-    data: { array: [ 1, 2, 3 ] },
-    result: "123"
-  }),
-
-  // Bug 692028: DOMTemplate memory leak with asynchronous arrays
-  // Bug 692031: DOMTemplate async loops do not drop the loop element
-  () => ({
-    name: "asyncLoopElement",
-    template: '<loop foreach="i in ${array}">${i}</loop>',
-    data: { array: delayReply([1, 2, 3]) },
-    result: "<span></span>",
-    later: "123"
-  }),
-
-  () => ({
-    name: "saveElement",
-    template: '<p save="${element}">${name}</p>',
-    data: { name: "pass 8" },
-    result: "<p>pass 8</p>",
-    also: function(options) {
-      is(options.data.element.innerHTML, "pass 8", "saveElement saved");
-      delete options.data.element;
-    }
-  }),
-
-  () => ({
-    name: "useElement",
-    template: '<p id="pass9">${adjust(__element)}</p>',
-    options: { allowEval: true },
-    data: {
-      adjust: function(element) {
-        is("pass9", element.id, "useElement adjust");
-        return "pass 9b";
-      }
-    },
-    result: '<p id="pass9">pass 9b</p>'
-  }),
-
-  () => ({
-    name: "asyncInline",
-    template: "${delayed}",
-    data: { delayed: delayReply("inline") },
-    result: "<span></span>",
-    later: "inline"
-  }),
-
-  // Bug 692028: DOMTemplate memory leak with asynchronous arrays
-  () => ({
-    name: "asyncArray",
-    template: '<p foreach="i in ${delayed}">${i}</p>',
-    data: { delayed: delayReply([1, 2, 3]) },
-    result: "<span></span>",
-    later: "<p>1</p><p>2</p><p>3</p>"
-  }),
-
-  () => ({
-    name: "asyncMember",
-    template: '<p foreach="i in ${delayed}">${i}</p>',
-    data: { delayed: [delayReply(4), delayReply(5), delayReply(6)] },
-    result: "<span></span><span></span><span></span>",
-    later: "<p>4</p><p>5</p><p>6</p>"
-  }),
-
-  // Bug 692028: DOMTemplate memory leak with asynchronous arrays
-  () => ({
-    name: "asyncBoth",
-    template: '<p foreach="i in ${delayed}">${i}</p>',
-    data: {
-      delayed: delayReply([
-        delayReply(4),
-        delayReply(5),
-        delayReply(6)
-      ])
-    },
-    result: "<span></span>",
-    later: "<p>4</p><p>5</p><p>6</p>"
-  }),
-
-  // Bug 701762: DOMTemplate fails when ${foo()} returns undefined
-  () => ({
-    name: "functionReturningUndefiend",
-    template: "<p>${foo()}</p>",
-    options: { allowEval: true },
-    data: {
-      foo: function() {}
-    },
-    result: "<p>undefined</p>"
-  }),
-
-  // Bug 702642: DOMTemplate is relatively slow when evaluating JS ${}
-  () => ({
-    name: "propertySimple",
-    template: "<p>${a.b.c}</p>",
-    data: { a: { b: { c: "hello" } } },
-    result: "<p>hello</p>"
-  }),
-
-  () => ({
-    name: "propertyPass",
-    template: "<p>${Math.max(1, 2)}</p>",
-    options: { allowEval: true },
-    result: "<p>2</p>"
-  }),
-
-  () => ({
-    name: "propertyFail",
-    template: "<p>${Math.max(1, 2)}</p>",
-    result: "<p>${Math.max(1, 2)}</p>"
-  }),
-
-  // Bug 723431: DOMTemplate should allow customisation of display of
-  // null/undefined values
-  () => ({
-    name: "propertyUndefAttrFull",
-    template: "<p>${nullvar}|${undefinedvar1}|${undefinedvar2}</p>",
-    data: { nullvar: null, undefinedvar1: undefined },
-    result: "<p>null|undefined|undefined</p>"
-  }),
-
-  () => ({
-    name: "propertyUndefAttrBlank",
-    template: "<p>${nullvar}|${undefinedvar1}|${undefinedvar2}</p>",
-    data: { nullvar: null, undefinedvar1: undefined },
-    options: { blankNullUndefined: true },
-    result: "<p>||</p>"
-  }),
-
-  /* eslint-disable max-len */
-  () => ({
-    name: "propertyUndefAttrFull",
-    template: '<div><p value="${nullvar}"></p><p value="${undefinedvar1}"></p><p value="${undefinedvar2}"></p></div>',
-    data: { nullvar: null, undefinedvar1: undefined },
-    result: '<div><p value="null"></p><p value="undefined"></p><p value="undefined"></p></div>'
-  }),
-
-  () => ({
-    name: "propertyUndefAttrBlank",
-    template: '<div><p value="${nullvar}"></p><p value="${undefinedvar1}"></p><p value="${undefinedvar2}"></p></div>',
-    data: { nullvar: null, undefinedvar1: undefined },
-    options: { blankNullUndefined: true },
-    result: '<div><p value=""></p><p value=""></p><p value=""></p></div>'
-  })
-  /* eslint-enable max-len */
-];
-
-function delayReply(data) {
-  return new Promise(resolve => resolve(data));
-}
deleted file mode 100644
--- a/devtools/client/themes/images/gcli_sec_bad.svg
+++ /dev/null
@@ -1,7 +0,0 @@
-<svg width="30" height="30" xmlns="http://www.w3.org/2000/svg">
-  <circle cx="15" cy="15" r="15" fill="#e74c3c"/>
-  <g stroke="white" stroke-width="3">
-    <line x1="9" y1="9" x2="21" y2="21"/>
-    <line x1="21" y1="9" x2="9" y2="21"/>
-  </g>
-</svg>
\ No newline at end of file
deleted file mode 100644
--- a/devtools/client/themes/images/gcli_sec_good.svg
+++ /dev/null
@@ -1,4 +0,0 @@
-<svg width="30" height="30" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 60 60">
-  <circle cx="30" cy="30" r="30" fill="#2CBB0F"/>
-  <polygon points="17,32 25,39 26,39 44,18 45,18 48,21 27,46 25,46 14,36 13,36 16,33 16,32" fill="white"/>
-</svg>
\ No newline at end of file
deleted file mode 100644
--- a/devtools/client/themes/images/gcli_sec_moderate.svg
+++ /dev/null
@@ -1,4 +0,0 @@
-<svg width="30" height="30" xmlns="http://www.w3.org/2000/svg">
-  <circle cx="15" cy="15" r="15" fill="#F5B400"/>
-  <rect x="7.5" y="13" width="15" height="4" fill="white"/>
-</svg>
\ No newline at end of file
--- a/devtools/docs/files/adding-files.md
+++ b/devtools/docs/files/adding-files.md
@@ -146,18 +146,18 @@ Locale URLs differ somewhat based on whe
 
 Example:
 
 * File: `/devtools/client/locales/en-US/debugger.dtd`
 * Usage: `chrome://devtools/locale/debugger.dtd`
 
 Example:
 
-* File: `/devtools/shared/locales/en-US/gcli.properties`
-* Usage: `chrome://devtools-shared/locale/gcli.properties`
+* File: `/devtools/shared/locales/en-US/screenshot.properties`
+* Usage: `chrome://devtools-shared/locale/screenshot.properties`
 
 ### Guidelines
 
 Localization files should follow a set of guidelines aimed at making it easier for people to translate the labels in these files in many languages.
 
 [Find these guidelines on MDN](https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices).
 
 In particular, it's important to write self-explanatory comments for new keys, deleting unused keys, changing the key name when changing the meaning of a string, and more. So make sure you read through these guidelines should you have to modify a localization file in your patch.
--- a/devtools/docs/frontend/telemetry.md
+++ b/devtools/docs/frontend/telemetry.md
@@ -116,17 +116,17 @@ devtools.main:
     objects: ["tools"]
     bug_numbers: [1416024]
     notification_emails: ["dev-developer-tools@lists.mozilla.org", "hkirschner@mozilla.com"]
     record_in_processes: ["main"]
     description: User opens devtools toolbox.
     release_channel_collection: opt-out
     expiry_version: never
     extra_keys:
-      entrypoint: How was the toolbox opened? CommandLine, ContextMenu, DeveloperToolbar, HamburgerMenu, KeyShortcut, SessionRestore or SystemMenu
+      entrypoint: How was the toolbox opened? CommandLine, ContextMenu, HamburgerMenu, KeyShortcut, SessionRestore or SystemMenu
       first_panel: The name of the first panel opened.
       host: "Toolbox host (positioning): bottom, side, window or other."
       splitconsole: Indicates whether the split console was open.
       width: Toolbox width (px).
 ```
 
 ### 2. Using Histograms.json probes in DevTools code
 
deleted file mode 100644
--- a/devtools/server/actors/gcli.js
+++ /dev/null
@@ -1,228 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-"use strict";
-
-const { Actor, ActorClassWithSpec } = require("devtools/shared/protocol");
-const { gcliSpec } = require("devtools/shared/specs/gcli");
-const { createSystem } = require("gcli/system");
-
-/**
- * Manage remote connections that want to talk to GCLI
- */
-const GcliActor = ActorClassWithSpec(gcliSpec, {
-  initialize: function(conn, targetActor) {
-    Actor.prototype.initialize.call(this, conn);
-
-    this._commandsChanged = this._commandsChanged.bind(this);
-
-    this._targetActor = targetActor;
-    // see _getRequisition()
-    this._requisitionPromise = undefined;
-  },
-
-  destroy: function() {
-    Actor.prototype.destroy.call(this);
-
-    // If _getRequisition has not been called, just bail quickly
-    if (this._requisitionPromise == null) {
-      this._commandsChanged = undefined;
-      this._targetActor = undefined;
-      return Promise.resolve();
-    }
-
-    return this._getRequisition().then(requisition => {
-      requisition.destroy();
-
-      this._system.commands.onCommandsChange.remove(this._commandsChanged);
-      this._system.destroy();
-      this._system = undefined;
-
-      this._requisitionPromise = undefined;
-      this._targetActor = undefined;
-
-      this._commandsChanged = undefined;
-    });
-  },
-
-  /**
-   * Load a module into the requisition
-   */
-  _testOnlyAddItemsByModule: function(names) {
-    return this._getRequisition().then(requisition => {
-      return requisition.system.addItemsByModule(names);
-    });
-  },
-
-  /**
-   * Unload a module from the requisition
-   */
-  _testOnlyRemoveItemsByModule: function(names) {
-    return this._getRequisition().then(requisition => {
-      return requisition.system.removeItemsByModule(names);
-    });
-  },
-
-  /**
-   * Retrieve a list of the remotely executable commands
-   * @param customProps Array of strings containing additional properties which,
-   * if specified in the command spec, will be included in the JSON. Normally we
-   * transfer only the properties required for GCLI to function.
-   */
-  specs: function(customProps) {
-    return this._getRequisition().then(requisition => {
-      return requisition.system.commands.getCommandSpecs(customProps);
-    });
-  },
-
-  /**
-   * Execute a GCLI command
-   * @return a promise of an object with the following properties:
-   * - data: The output of the command
-   * - type: The type of the data to allow selection of a converter
-   * - error: True if the output was considered an error
-   */
-  execute: function(typed) {
-    return this._getRequisition().then(requisition => {
-      return requisition.updateExec(typed).then(output => output.toJson());
-    });
-  },
-
-  /**
-   * Get the state of an input string. i.e. requisition.getStateData()
-   */
-  state: function(typed, start, rank) {
-    return this._getRequisition().then(requisition => {
-      return requisition.update(typed).then(() => {
-        return requisition.getStateData(start, rank);
-      });
-    });
-  },
-
-  /**
-   * Call type.parse to check validity. Used by the remote type
-   * @return a promise of an object with the following properties:
-   * - status: Of of the following strings: VALID|INCOMPLETE|ERROR
-   * - message: The message to display to the user
-   * - predictions: An array of suggested values for the given parameter
-   */
-  parseType: function(typed, paramName) {
-    return this._getRequisition().then(requisition => {
-      return requisition.update(typed).then(() => {
-        const assignment = requisition.getAssignment(paramName);
-        return Promise.resolve(assignment.predictions).then(predictions => {
-          return {
-            status: assignment.getStatus().toString(),
-            message: assignment.message,
-            predictions: predictions
-          };
-        });
-      });
-    });
-  },
-
-  /**
-   * Get the incremented/decremented value of some type
-   * @return a promise of a string containing the new argument text
-   */
-  nudgeType: function(typed, by, paramName) {
-    return this.requisition.update(typed).then(() => {
-      const assignment = this.requisition.getAssignment(paramName);
-      return this.requisition.nudge(assignment, by).then(() => {
-        return assignment.arg == null ? undefined : assignment.arg.text;
-      });
-    });
-  },
-
-  /**
-   * Perform a lookup on a selection type to get the allowed values
-   */
-  getSelectionLookup: function(commandName, paramName) {
-    return this._getRequisition().then(requisition => {
-      const command = requisition.system.commands.get(commandName);
-      if (command == null) {
-        throw new Error("No command called '" + commandName + "'");
-      }
-
-      let type;
-      command.params.forEach(param => {
-        if (param.name === paramName) {
-          type = param.type;
-        }
-      });
-
-      if (type == null) {
-        throw new Error("No parameter called '" + paramName + "' in '" +
-                        commandName + "'");
-      }
-
-      const reply = type.getLookup(requisition.executionContext);
-      return Promise.resolve(reply).then(lookup => {
-        // lookup returns an array of objects with name/value properties and
-        // the values might not be JSONable, so remove them
-        return lookup.map(info => ({ name: info.name }));
-      });
-    });
-  },
-
-  /**
-   * Lazy init for a Requisition
-   */
-  _getRequisition: function() {
-    if (this._targetActor == null) {
-      throw new Error("GcliActor used post-destroy");
-    }
-
-    if (this._requisitionPromise != null) {
-      return this._requisitionPromise;
-    }
-
-    const Requisition = require("gcli/cli").Requisition;
-    const targetActor = this._targetActor;
-
-    this._system = createSystem({ location: "server" });
-    this._system.commands.onCommandsChange.add(this._commandsChanged);
-
-    const gcliInit = require("devtools/shared/gcli/commands/index");
-    gcliInit.addAllItemsByModule(this._system);
-
-    // this._requisitionPromise should be created synchronously with the call
-    // to _getRequisition so that destroy can tell whether there is an async
-    // init in progress
-    this._requisitionPromise = this._system.load().then(() => {
-      const environment = {
-        get chromeWindow() {
-          throw new Error("environment.chromeWindow is not available in runAt:server" +
-            " commands");
-        },
-
-        get chromeDocument() {
-          throw new Error("environment.chromeDocument is not available in runAt:server" +
-            " commands");
-        },
-
-        get window() {
-          return targetActor.window;
-        },
-
-        get document() {
-          return targetActor.window && targetActor.window.document;
-        }
-      };
-
-      return new Requisition(this._system, { environment: environment });
-    });
-
-    return this._requisitionPromise;
-  },
-
-  /**
-   * Pass events from requisition.system.commands.onCommandsChange upwards
-   */
-  _commandsChanged: function() {
-    this.emit("commands-changed");
-  },
-});
-
-exports.GcliActor = GcliActor;
--- a/devtools/server/actors/highlighters.js
+++ b/devtools/server/actors/highlighters.js
@@ -548,18 +548,17 @@ exports.CustomHighlighterActor = protoco
  * highlighters to work: the window, docShell, event listener target, ...
  * It also emits "will-navigate", "navigate" and "window-ready" events,
  * similarly to the BrowsingContextTargetActor.
  *
  * It can be initialized either from a BrowsingContextTargetActor (which is the
  * most frequent way of using it, since highlighters are usually initialized by
  * the HighlighterActor or CustomHighlighterActor, which have a targetActor
  * reference). It can also be initialized just with a window object (which is
- * useful for when a highlighter is used outside of the debugger server context,
- * for instance from a gcli command).
+ * useful for when a highlighter is used outside of the debugger server context.
  */
 function HighlighterEnvironment() {
   this.relayTargetActorWindowReady = this.relayTargetActorWindowReady.bind(this);
   this.relayTargetActorNavigate = this.relayTargetActorNavigate.bind(this);
   this.relayTargetActorWillNavigate = this.relayTargetActorWillNavigate.bind(this);
 
   EventEmitter.decorate(this);
 }
--- a/devtools/server/actors/moz.build
+++ b/devtools/server/actors/moz.build
@@ -33,17 +33,16 @@ DevToolsModules(
     'css-properties.js',
     'csscoverage.js',
     'device.js',
     'emulation.js',
     'environment.js',
     'errordocs.js',
     'frame.js',
     'framerate.js',
-    'gcli.js',
     'heap-snapshot-file.js',
     'highlighters.css',
     'highlighters.js',
     'layout.js',
     'memory.js',
     'network-event.js',
     'network-monitor.js',
     'object.js',
--- a/devtools/server/actors/webconsole/commands.js
+++ b/devtools/server/actors/webconsole/commands.js
@@ -111,17 +111,17 @@ function parseCommand(tokens) {
     if (token.type === KEY) {
       const nextTokenIndex = i + 1;
       const nextToken = tokens[nextTokenIndex];
       let values = args[token.value] || DEFAULT_VALUE;
       if (nextToken && nextToken.type === ARG) {
         const { value, offset } = collectString(nextToken, tokens, nextTokenIndex);
         // in order for JSON.stringify to correctly output values, they must be correctly
         // typed
-        // As per the GCLI documentation, we can only have one value associated with a
+        // As per the old GCLI documentation, we can only have one value associated with a
         // flag but multiple flags with the same name can exist and should be combined
         // into and array.  Here we are associating only the value on the right hand
         // side if it is of type `arg` as a single value; the second case initializes
         // an array, and the final case pushes a value to an existing array
         const typedValue = getTypedValue(value);
         if (values === DEFAULT_VALUE) {
           values = typedValue;
         } else if (!Array.isArray(values)) {
--- a/devtools/server/main.js
+++ b/devtools/server/main.js
@@ -353,21 +353,16 @@ var DebuggerServer = {
       constructor: "StyleSheetsActor",
       type: { target: true }
     });
     this.registerModule("devtools/server/actors/storage", {
       prefix: "storage",
       constructor: "StorageActor",
       type: { target: true }
     });
-    this.registerModule("devtools/server/actors/gcli", {
-      prefix: "gcli",
-      constructor: "GcliActor",
-      type: { target: true }
-    });
     this.registerModule("devtools/server/actors/memory", {
       prefix: "memory",
       constructor: "MemoryActor",
       type: { target: true }
     });
     this.registerModule("devtools/server/actors/framerate", {
       prefix: "framerate",
       constructor: "FramerateActor",
--- a/devtools/shared/Loader.jsm
+++ b/devtools/shared/Loader.jsm
@@ -28,18 +28,16 @@ this.EXPORTED_SYMBOLS = ["DevToolsLoader
  */
 function BuiltinProvider() {}
 BuiltinProvider.prototype = {
   load: function() {
     const paths = {
       // ⚠ DISCUSSION ON DEV-DEVELOPER-TOOLS REQUIRED BEFORE MODIFYING ⚠
       "devtools": "resource://devtools",
       // ⚠ DISCUSSION ON DEV-DEVELOPER-TOOLS REQUIRED BEFORE MODIFYING ⚠
-      "gcli": "resource://devtools/shared/gcli/source/lib/gcli",
-      // ⚠ DISCUSSION ON DEV-DEVELOPER-TOOLS REQUIRED BEFORE MODIFYING ⚠
       "acorn": "resource://devtools/shared/acorn",
       // ⚠ DISCUSSION ON DEV-DEVELOPER-TOOLS REQUIRED BEFORE MODIFYING ⚠
       "acorn/util/walk": "resource://devtools/shared/acorn/walk.js",
       // ⚠ DISCUSSION ON DEV-DEVELOPER-TOOLS REQUIRED BEFORE MODIFYING ⚠
       "source-map": "resource://devtools/shared/sourcemap/source-map.js",
       // ⚠ DISCUSSION ON DEV-DEVELOPER-TOOLS REQUIRED BEFORE MODIFYING ⚠
       // Allow access to xpcshell test items from the loader.
       "xpcshell-test": "resource://test",
deleted file mode 100644
--- a/devtools/shared/fronts/gcli.js
+++ /dev/null
@@ -1,40 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-"use strict";
-
-const { Front, FrontClassWithSpec } = require("devtools/shared/protocol");
-const { gcliSpec } = require("devtools/shared/specs/gcli");
-
-/**
- *
- */
-const GcliFront = exports.GcliFront = FrontClassWithSpec(gcliSpec, {
-  initialize: function(client, tabForm) {
-    Front.prototype.initialize.call(this, client);
-    this.actorID = tabForm.gcliActor;
-
-    // XXX: This is the first actor type in its hierarchy to use the protocol
-    // library, so we're going to self-own on the client side for now.
-    this.manage(this);
-  },
-});
-
-// A cache of created fronts: WeakMap<Client, Front>
-const knownFronts = new WeakMap();
-
-/**
- * Create a GcliFront only when needed (returns a promise)
- * For notes on target.makeRemote(), see
- * https://bugzilla.mozilla.org/show_bug.cgi?id=1016330#c7
- */
-exports.GcliFront.create = function(target) {
-  return target.makeRemote().then(() => {
-    let front = knownFronts.get(target.client);
-    if (front == null && target.form.gcliActor != null) {
-      front = new GcliFront(target.client, target.form);
-      knownFronts.set(target.client, front);
-    }
-    return front;
-  });
-};
--- a/devtools/shared/fronts/moz.build
+++ b/devtools/shared/fronts/moz.build
@@ -14,17 +14,16 @@ DevToolsModules(
     'animation.js',
     'call-watcher.js',
     'canvas.js',
     'css-properties.js',
     'csscoverage.js',
     'device.js',
     'emulation.js',
     'framerate.js',
-    'gcli.js',
     'highlighters.js',
     'inspector.js',
     'layout.js',
     'memory.js',
     'node.js',
     'perf.js',
     'performance-recording.js',
     'performance.js',
deleted file mode 100644
--- a/devtools/shared/gcli/command-state.js
+++ /dev/null
@@ -1,78 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-"use strict";
-
-const EventEmitter = require("devtools/shared/event-emitter");
-
-const getTargetId = ({tab}) => tab.linkedBrowser.outerWindowID;
-const enabledCommands = new Map();
-
-/**
- * The `CommandState` is a singleton that provides utility methods to keep the commands'
- * state in sync between the toolbox, the toolbar and the content.
- */
-const CommandState = EventEmitter.decorate({
-  /**
-   * Returns if a command is enabled on a given target.
-   *
-   * @param {Object} target
-   *                  The target object must have a tab's reference.
-   * @param {String} command
-   *                  The command's name used in gcli.
-   * @ returns {Boolean} returns `false` if the command is not enabled for the target
-   *                    given, or if the target given hasn't a tab; `true` otherwise.
-   */
-  isEnabledForTarget(target, command) {
-    if (!target.tab || !enabledCommands.has(command)) {
-      return false;
-    }
-
-    return enabledCommands.get(command).has(getTargetId(target));
-  },
-
-  /**
-   * Enables a command on a given target.
-   * Emits a "changed" event to notify potential observers about the new commands state.
-   *
-   * @param {Object} target
-   *                  The target object must have a tab's reference.
-   * @param {String} command
-   *                  The command's name used in gcli.
-   */
-  enableForTarget(target, command) {
-    if (!target.tab) {
-      return;
-    }
-
-    if (!enabledCommands.has(command)) {
-      enabledCommands.set(command, new Set());
-    }
-
-    enabledCommands.get(command).add(getTargetId(target));
-
-    CommandState.emit("changed", {target, command});
-  },
-
-  /**
-   * Disabled a command on a given target.
-   * Emits a "changed" event to notify potential observers about the new commands state.
-   *
-   * @param {Object} target
-   *                  The target object must have a tab's reference.
-   * @param {String} command
-   *                  The command's name used in gcli.
-   */
-  disableForTarget(target, command) {
-    if (!target.tab || !enabledCommands.has(command)) {
-      return;
-    }
-
-    enabledCommands.get(command).delete(getTargetId(target));
-
-    CommandState.emit("changed", {target, command});
-  },
-});
-exports.CommandState = CommandState;
-
deleted file mode 100644
--- a/devtools/shared/gcli/commands/index.js
+++ /dev/null
@@ -1,159 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-"use strict";
-
-const { createSystem, connectFront, disconnectFront } = require("gcli/system");
-const { GcliFront } = require("devtools/shared/fronts/gcli");
-
-/**
- * This is the basic list of modules that should be loaded into each
- * requisition instance whether server side or client side
- */
-exports.baseModules = [
-  "gcli/types/delegate",
-  "gcli/types/selection",
-  "gcli/types/array",
-
-  "gcli/types/boolean",
-  "gcli/types/command",
-  "gcli/types/date",
-  "gcli/types/file",
-  "gcli/types/javascript",
-  "gcli/types/node",
-  "gcli/types/number",
-  "gcli/types/resource",
-  "gcli/types/setting",
-  "gcli/types/string",
-  "gcli/types/union",
-  "gcli/types/url",
-
-  "gcli/fields/fields",
-  "gcli/fields/delegate",
-  "gcli/fields/selection",
-
-  "gcli/ui/focus",
-  "gcli/ui/intro",
-
-  "gcli/converters/converters",
-  "gcli/converters/basic",
-  "gcli/converters/terminal",
-
-  "gcli/languages/command",
-  "gcli/languages/javascript",
-
-  "gcli/commands/clear",
-  "gcli/commands/context",
-  "gcli/commands/help",
-  "gcli/commands/pref",
-];
-
-/**
- * Some commands belong to a tool (see getToolModules). This is a list of the
- * modules that are *not* owned by a tool.
- */
-exports.devtoolsModules = [
-  "devtools/shared/gcli/commands/measure",
-  "devtools/shared/gcli/commands/paintflashing",
-  "devtools/shared/gcli/commands/rulers",
-  "devtools/shared/gcli/commands/screenshot",
-];
-
-/**
- * Register commands from tools with 'command: [ "some/module" ]' definitions.
- * The map/reduce incantation squashes the array of arrays to a single array.
- */
-try {
-  const { defaultTools } = require("devtools/client/definitions");
-  exports.devtoolsToolModules = defaultTools.map(def => def.commands || [])
-                                   .reduce((prev, curr) => prev.concat(curr), []);
-} catch (e) {
-  // "devtools/client/definitions" is only accessible from Firefox
-  exports.devtoolsToolModules = [];
-}
-
-/**
- * Register commands from toolbox buttons with 'command: [ "some/module" ]'
- * definitions.  The map/reduce incantation squashes the array of arrays to a
- * single array.
- */
-try {
-  const { ToolboxButtons } = require("devtools/client/definitions");
-  exports.devtoolsButtonModules = ToolboxButtons.map(def => def.commands || [])
-                                     .reduce((prev, curr) => prev.concat(curr), []);
-} catch (e) {
-  // "devtools/client/definitions" is only accessible from Firefox
-  exports.devtoolsButtonModules = [];
-}
-
-/**
- * Add modules to a system for use in a content process (but don't call load)
- */
-exports.addAllItemsByModule = function(system) {
-  system.addItemsByModule(exports.baseModules, { delayedLoad: true });
-  system.addItemsByModule(exports.devtoolsModules, { delayedLoad: true });
-  system.addItemsByModule(exports.devtoolsToolModules, { delayedLoad: true });
-  system.addItemsByModule(exports.devtoolsButtonModules, { delayedLoad: true });
-};
-
-/**
- * This is WeakMap<Target, Links> where Links is an object that looks like
- *   { refs: number, promise: Promise<System>, front: GcliFront }
- */
-var linksForTarget = new WeakMap();
-
-/**
- * The toolbox uses the following properties on a command to allow it to be
- * added to the toolbox toolbar
- */
-var customProperties = [ "buttonId", "buttonClass", "tooltipText" ];
-
-/**
- * Create a system which connects to a GCLI in a remote target
- * @return Promise<System> for the given target
- */
-exports.getSystem = function(target) {
-  const existingLinks = linksForTarget.get(target);
-  if (existingLinks != null) {
-    existingLinks.refs++;
-    return existingLinks.promise;
-  }
-
-  const system = createSystem({ location: "client" });
-
-  exports.addAllItemsByModule(system);
-
-  // Load the client system
-  const links = {
-    refs: 1,
-    system,
-    promise: system.load().then(() => {
-      return GcliFront.create(target).then(front => {
-        links.front = front;
-        return connectFront(system, front, customProperties).then(() => system);
-      });
-    })
-  };
-
-  linksForTarget.set(target, links);
-  return links.promise;
-};
-
-/**
- * Someone that called getSystem doesn't need it any more, so decrement the
- * count of users of the system for that target, and destroy if needed
- */
-exports.releaseSystem = function(target) {
-  const links = linksForTarget.get(target);
-  if (links == null) {
-    throw new Error("releaseSystem called for unknown target");
-  }
-
-  links.refs--;
-  if (links.refs === 0) {
-    disconnectFront(links.system, links.front);
-    links.system.destroy();
-    linksForTarget.delete(target);
-  }
-};
deleted file mode 100644
--- a/devtools/shared/gcli/commands/measure.js
+++ /dev/null
@@ -1,100 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-"use strict";
-
-const EventEmitter = require("devtools/shared/event-emitter");
-
-loader.lazyRequireGetter(this, "CommandState",
-  "devtools/shared/gcli/command-state", true);
-
-const l10n = require("gcli/l10n");
-require("devtools/server/actors/inspector/inspector");
-const { HighlighterEnvironment } =
-  require("devtools/server/actors/highlighters");
-const { MeasuringToolHighlighter } =
-  require("devtools/server/actors/highlighters/measuring-tool");
-
-const highlighters = new WeakMap();
-
-exports.items = [
-  // The client measure command is used to maintain the toolbar button state
-  // only and redirects to the server command to actually toggle the measuring
-  // tool (see `measure_server` below).
-  {
-    name: "measure",
-    runAt: "client",
-    description: l10n.lookup("measureDesc"),
-    manual: l10n.lookup("measureManual"),
-    buttonId: "command-button-measure",
-    buttonClass: "command-button",
-    tooltipText: l10n.lookup("measureTooltip"),
-    state: {
-      isChecked: (target) => CommandState.isEnabledForTarget(target, "measure"),
-      onChange: (target, handler) => CommandState.on("changed", handler),
-      offChange: (target, handler) => CommandState.off("changed", handler)
-    },
-    exec: function* (args, context) {
-      const { target } = context.environment;
-
-      // Pipe the call to the server command.
-      const response = yield context.updateExec("measure_server");
-      const isEnabled = response.data;
-
-      if (isEnabled) {
-        CommandState.enableForTarget(target, "measure");
-      } else {
-        CommandState.disableForTarget(target, "measure");
-      }
-
-      // Toggle off the button when the page navigates because the measuring
-      // tool is removed automatically by the MeasuringToolHighlighter on the
-      // server then.
-      target.once("will-navigate", () =>
-        CommandState.disableForTarget(target, "measure"));
-    }
-  },
-  // The server measure command is hidden by default, it's just used by the
-  // client command.
-  {
-    name: "measure_server",
-    runAt: "server",
-    hidden: true,
-    returnType: "highlighterVisibility",
-    exec: function(args, context) {
-      const env = context.environment;
-      const { document } = env;
-
-      // Calling the command again after the measuring tool has been shown once,
-      // hides it.
-      if (highlighters.has(document)) {
-        const { highlighter } = highlighters.get(document);
-        highlighter.destroy();
-        return false;
-      }
-
-      // Otherwise, display the measuring tool.
-      const environment = new HighlighterEnvironment();
-      environment.initFromWindow(env.window);
-      const highlighter = new MeasuringToolHighlighter(environment);
-
-      // Store the instance of the measuring tool highlighter for this document
-      // so we can hide it later.
-      highlighters.set(document, { highlighter, environment });
-
-      // Listen to the highlighter's destroy event which may happen if the
-      // window is refreshed or closed with the measuring tool shown.
-      EventEmitter.once(highlighter, "destroy", () => {
-        if (highlighters.has(document)) {
-          const { environment: toDestroy } = highlighters.get(document);
-          toDestroy.destroy();
-          highlighters.delete(document);
-        }
-      });
-
-      highlighter.show();
-      return true;
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/commands/moz.build
+++ /dev/null
@@ -1,13 +0,0 @@
-# -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
-# vim: set filetype=python:
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-DevToolsModules(
-    'index.js',
-    'measure.js',
-    'paintflashing.js',
-    'rulers.js',
-    'screenshot.js',
-)
deleted file mode 100644
--- a/devtools/shared/gcli/commands/paintflashing.js
+++ /dev/null
@@ -1,182 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-"use strict";
-
-loader.lazyRequireGetter(this, "CommandState",
-  "devtools/shared/gcli/command-state", true);
-
-var telemetry;
-try {
-  const Telemetry = require("devtools/client/shared/telemetry");
-  telemetry = new Telemetry();
-} catch (e) {
-  // DevTools Telemetry module only available in Firefox
-}
-
-const gcli = require("gcli/index");
-const l10n = require("gcli/l10n");
-
-/**
- * Fire events and telemetry when paintFlashing happens
- */
-function onPaintFlashingChanged(target, flashing) {
-  if (flashing) {
-    CommandState.enableForTarget(target, "paintflashing");
-  } else {
-    CommandState.disableForTarget(target, "paintflashing");
-  }
-
-  target.once("will-navigate", () =>
-    CommandState.disableForTarget(target, "paintflashing"));
-
-  if (!telemetry) {
-    return;
-  }
-  if (flashing) {
-    telemetry.toolOpened("paintflashing");
-  } else {
-    telemetry.toolClosed("paintflashing");
-  }
-}
-
-/**
- * Alter the paintFlashing state of a window and report on the new value.
- * This works with chrome or content windows.
- *
- * This is a bizarre method that you could argue should be broken up into
- * separate getter and setter functions, however keeping it as one helps
- * to simplify the commands below.
- *
- * @param state {string} One of:
- * - "on" which does window.paintFlashing = true
- * - "off" which does window.paintFlashing = false
- * - "toggle" which does window.paintFlashing = !window.paintFlashing
- * - "query" which does nothing
- * @return The new value of the window.paintFlashing flag
- */
-function setPaintFlashing(window, state) {
-  const winUtils = window.windowUtils;
-
-  if (!["on", "off", "toggle", "query"].includes(state)) {
-    throw new Error(`Unsupported state: ${state}`);
-  }
-
-  if (state === "on") {
-    winUtils.paintFlashing = true;
-  } else if (state === "off") {
-    winUtils.paintFlashing = false;
-  } else if (state === "toggle") {
-    winUtils.paintFlashing = !winUtils.paintFlashing;
-  }
-
-  return winUtils.paintFlashing;
-}
-
-exports.items = [
-  {
-    name: "paintflashing",
-    description: l10n.lookup("paintflashingDesc")
-  },
-  {
-    item: "command",
-    runAt: "client",
-    name: "paintflashing on",
-    description: l10n.lookup("paintflashingOnDesc"),
-    manual: l10n.lookup("paintflashingManual"),
-    params: [{
-      group: "options",
-      params: [
-        {
-          type: "boolean",
-          name: "chrome",
-          get hidden() {
-            return gcli.hiddenByChromePref();
-          },
-          description: l10n.lookup("paintflashingChromeDesc"),
-        }
-      ]
-    }],
-    exec: function* (args, context) {
-      if (!args.chrome) {
-        const output = yield context.updateExec("paintflashing_server --state on");
-
-        onPaintFlashingChanged(context.environment.target, output.data);
-      } else {
-        setPaintFlashing(context.environment.chromeWindow, "on");
-      }
-    }
-  },
-  {
-    item: "command",
-    runAt: "client",
-    name: "paintflashing off",
-    description: l10n.lookup("paintflashingOffDesc"),
-    manual: l10n.lookup("paintflashingManual"),
-    params: [{
-      group: "options",
-      params: [
-        {
-          type: "boolean",
-          name: "chrome",
-          get hidden() {
-            return gcli.hiddenByChromePref();
-          },
-          description: l10n.lookup("paintflashingChromeDesc"),
-        }
-      ]
-    }],
-    exec: function* (args, context) {
-      if (!args.chrome) {
-        const output = yield context.updateExec("paintflashing_server --state off");
-
-        onPaintFlashingChanged(context.environment.target, output.data);
-      } else {
-        setPaintFlashing(context.environment.chromeWindow, "off");
-      }
-    }
-  },
-  {
-    item: "command",
-    runAt: "client",
-    name: "paintflashing toggle",
-    hidden: true,
-    buttonId: "command-button-paintflashing",
-    buttonClass: "command-button",
-    state: {
-      isChecked: (target) => CommandState.isEnabledForTarget(target, "paintflashing"),
-      onChange: (_, handler) => CommandState.on("changed", handler),
-      offChange: (_, handler) => CommandState.off("changed", handler),
-    },
-    tooltipText: l10n.lookup("paintflashingTooltip"),
-    description: l10n.lookup("paintflashingToggleDesc"),
-    manual: l10n.lookup("paintflashingManual"),
-    exec: function* (args, context) {
-      const output = yield context.updateExec("paintflashing_server --state toggle");
-
-      onPaintFlashingChanged(context.environment.target, output.data);
-    }
-  },
-  {
-    item: "command",
-    runAt: "server",
-    name: "paintflashing_server",
-    hidden: true,
-    params: [
-      {
-        name: "state",
-        type: {
-          name: "selection",
-          data: [ "on", "off", "toggle", "query" ]
-        }
-      },
-    ],
-    returnType: "paintFlashingState",
-    exec: function(args, context) {
-      const { window } = context.environment;
-
-      return setPaintFlashing(window, args.state);
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/commands/rulers.js
+++ /dev/null
@@ -1,98 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-"use strict";
-
-const EventEmitter = require("devtools/shared/event-emitter");
-
-loader.lazyRequireGetter(this, "CommandState",
-  "devtools/shared/gcli/command-state", true);
-
-const l10n = require("gcli/l10n");
-require("devtools/server/actors/inspector/inspector");
-const { HighlighterEnvironment } =
-  require("devtools/server/actors/highlighters");
-const { RulersHighlighter } =
-  require("devtools/server/actors/highlighters/rulers");
-
-const highlighters = new WeakMap();
-
-exports.items = [
-  // The client rulers command is used to maintain the toolbar button state only
-  // and redirects to the server command to actually toggle the rulers (see
-  // rulers_server below).
-  {
-    name: "rulers",
-    runAt: "client",
-    description: l10n.lookup("rulersDesc"),
-    manual: l10n.lookup("rulersManual"),
-    buttonId: "command-button-rulers",
-    buttonClass: "command-button",
-    tooltipText: l10n.lookup("rulersTooltip"),
-    state: {
-      isChecked: (target) => CommandState.isEnabledForTarget(target, "rulers"),
-      onChange: (target, handler) => CommandState.on("changed", handler),
-      offChange: (target, handler) => CommandState.off("changed", handler)
-    },
-    exec: function* (args, context) {
-      const { target } = context.environment;
-
-      // Pipe the call to the server command.
-      const response = yield context.updateExec("rulers_server");
-      const isEnabled = response.data;
-
-      if (isEnabled) {
-        CommandState.enableForTarget(target, "rulers");
-      } else {
-        CommandState.disableForTarget(target, "rulers");
-      }
-
-      // Toggle off the button when the page navigates because the rulers are
-      // removed automatically by the RulersHighlighter on the server then.
-      target.once("will-navigate", () => CommandState.disableForTarget(target, "rulers"));
-    }
-  },
-  // The server rulers command is hidden by default, it's just used by the
-  // client command.
-  {
-    name: "rulers_server",
-    runAt: "server",
-    hidden: true,
-    returnType: "highlighterVisibility",
-    exec: function(args, context) {
-      const env = context.environment;
-      const { document } = env;
-
-      // Calling the command again after the rulers have been shown once hides
-      // them.
-      if (highlighters.has(document)) {
-        const { highlighter } = highlighters.get(document);
-        highlighter.destroy();
-        return false;
-      }
-
-      // Otherwise, display the rulers.
-      const environment = new HighlighterEnvironment();
-      environment.initFromWindow(env.window);
-      const highlighter = new RulersHighlighter(environment);
-
-      // Store the instance of the rulers highlighter for this document so we
-      // can hide it later.
-      highlighters.set(document, { highlighter, environment });
-
-      // Listen to the highlighter's destroy event which may happen if the
-      // window is refreshed or closed with the rulers shown.
-      EventEmitter.once(highlighter, "destroy", () => {
-        if (highlighters.has(document)) {
-          const { environment: toDestroy } = highlighters.get(document);
-          toDestroy.destroy();
-          highlighters.delete(document);
-        }
-      });
-
-      highlighter.show();
-      return true;
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/commands/screenshot.js
+++ /dev/null
@@ -1,593 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-"use strict";
-
-const { Cc, Ci, Cr, Cu } = require("chrome");
-const ChromeUtils = require("ChromeUtils");
-const l10n = require("gcli/l10n");
-const Services = require("Services");
-const { NetUtil } = require("resource://gre/modules/NetUtil.jsm");
-const { getRect } = require("devtools/shared/layout/utils");
-const defer = require("devtools/shared/defer");
-const { Task } = require("devtools/shared/task");
-
-loader.lazyRequireGetter(this, "openContentLink", "devtools/client/shared/link", true);
-
-loader.lazyImporter(this, "Downloads", "resource://gre/modules/Downloads.jsm");
-loader.lazyImporter(this, "OS", "resource://gre/modules/osfile.jsm");
-loader.lazyImporter(this, "FileUtils", "resource://gre/modules/FileUtils.jsm");
-loader.lazyImporter(this, "PrivateBrowsingUtils",
-                          "resource://gre/modules/PrivateBrowsingUtils.jsm");
-
-// String used as an indication to generate default file name in the following
-// format: "Screen Shot yyyy-mm-dd at HH.MM.SS.png"
-const FILENAME_DEFAULT_VALUE = " ";
-const CONTAINER_FLASHING_DURATION = 500;
-
-/*
- * There are 2 commands and 1 converter here. The 2 commands are nearly
- * identical except that one runs on the client and one in the server.
- *
- * The server command is hidden, and is designed to be called from the client
- * command.
- */
-
-/**
- * Both commands have the same initial filename parameter
- */
-const filenameParam = {
-  name: "filename",
-  type: "string",
-  defaultValue: FILENAME_DEFAULT_VALUE,
-  description: l10n.lookup("screenshotFilenameDesc"),
-  manual: l10n.lookup("screenshotFilenameManual")
-};
-
-/**
- * Both commands have almost the same set of standard optional parameters, except for the
- * type of the --selector option, which can be a node only on the server.
- */
-const getScreenshotCommandParams = function(isClient) {
-  return {
-    group: l10n.lookup("screenshotGroupOptions"),
-    params: [
-      {
-        name: "clipboard",
-        type: "boolean",
-        description: l10n.lookup("screenshotClipboardDesc"),
-        manual: l10n.lookup("screenshotClipboardManual")
-      },
-      {
-        name: "imgur",
-        type: "boolean",
-        description: l10n.lookup("screenshotImgurDesc"),
-        manual: l10n.lookup("screenshotImgurManual")
-      },
-      {
-        name: "delay",
-        type: { name: "number", min: 0 },
-        defaultValue: 0,
-        description: l10n.lookup("screenshotDelayDesc"),
-        manual: l10n.lookup("screenshotDelayManual")
-      },
-      {
-        name: "dpr",
-        type: { name: "number", min: 0, allowFloat: true },
-        defaultValue: 0,
-        description: l10n.lookup("screenshotDPRDesc"),
-        manual: l10n.lookup("screenshotDPRManual")
-      },
-      {
-        name: "fullpage",
-        type: "boolean",
-        description: l10n.lookup("screenshotFullPageDesc"),
-        manual: l10n.lookup("screenshotFullPageManual")
-      },
-      {
-        name: "selector",
-        // On the client side, don't try to parse the selector as a node as it will
-        // trigger an unsafe CPOW.
-        type: isClient ? "string" : "node",
-        defaultValue: null,
-        description: l10n.lookup("inspectNodeDesc"),
-        manual: l10n.lookup("inspectNodeManual")
-      },
-      {
-        name: "file",
-        type: "boolean",
-        description: l10n.lookup("screenshotFileDesc"),
-        manual: l10n.lookup("screenshotFileManual"),
-      },
-    ]
-  };
-};
-
-const clientScreenshotParams = getScreenshotCommandParams(true);
-const serverScreenshotParams = getScreenshotCommandParams(false);
-
-exports.items = [
-  {
-    /**
-     * Format an 'imageSummary' (as output by the screenshot command).
-     * An 'imageSummary' is a simple JSON object that looks like this:
-     *
-     * {
-     *   destinations: [ "..." ], // Required array of descriptions of the
-     *                            // locations of the result image (the command
-     *                            // can have multiple outputs)
-     *   data: "...",             // Optional Base64 encoded image data
-     *   width:1024, height:768,  // Dimensions of the image data, required
-     *                            // if data != null
-     *   filename: "...",         // If set, clicking the image will open the
-     *                            // folder containing the given file
-     *   href: "...",             // If set, clicking the image will open the
-     *                            // link in a new tab
-     * }
-     */
-    item: "converter",
-    from: "imageSummary",
-    to: "dom",
-    exec: function(imageSummary, context) {
-      const document = context.document;
-      const root = document.createElement("div");
-
-      // Add a line to the result for each destination
-      imageSummary.destinations.forEach(destination => {
-        const title = document.createElement("div");
-        title.textContent = destination;
-        root.appendChild(title);
-      });
-
-      // Add the thumbnail image
-      if (imageSummary.data != null) {
-        const image = context.document.createElement("div");
-        const previewHeight = parseInt(256 * imageSummary.height / imageSummary.width,
-                                       10);
-        const style = "" +
-            "width: 256px;" +
-            "height: " + previewHeight + "px;" +
-            "max-height: 256px;" +
-            "background-image: url('" + imageSummary.data + "');" +
-            "background-size: 256px " + previewHeight + "px;" +
-            "margin: 4px;" +
-            "display: block;";
-        image.setAttribute("style", style);
-        root.appendChild(image);
-      }
-
-      // Click handler
-      if (imageSummary.href || imageSummary.filename) {
-        root.style.cursor = "pointer";
-        root.addEventListener("click", () => {
-          if (imageSummary.href) {
-            openContentLink(imageSummary.href);
-          } else if (imageSummary.filename) {
-            const file = Cc["@mozilla.org/file/local;1"].createInstance(Ci.nsIFile);
-            file.initWithPath(imageSummary.filename);
-            file.reveal();
-          }
-        });
-      }
-
-      return root;
-    }
-  },
-  {
-    item: "command",
-    runAt: "client",
-    name: "screenshot",
-    description: l10n.lookup("screenshotDesc"),
-    manual: l10n.lookup("screenshotManual"),
-    returnType: "imageSummary",
-    buttonId: "command-button-screenshot",
-    buttonClass: "command-button",
-    tooltipText: l10n.lookup("screenshotTooltipPage"),
-    params: [
-      filenameParam,
-      clientScreenshotParams,
-    ],
-    exec: function(args, context) {
-      // Re-execute the command on the server
-      const command = context.typed.replace(/^screenshot/, "screenshot_server");
-      const capture = context.updateExec(command).then(output => {
-        return output.error ? Promise.reject(output.data) : output.data;
-      });
-
-      simulateCameraEffect(context.environment.chromeDocument, "shutter");
-      return capture.then(saveScreenshot.bind(null, args, context));
-    },
-  },
-  {
-    item: "command",
-    runAt: "server",
-    name: "screenshot_server",
-    hidden: true,
-    returnType: "imageSummary",
-    params: [
-      filenameParam,
-      serverScreenshotParams,
-    ],
-    exec: function(args, context) {
-      return captureScreenshot(args, context.environment.document);
-    },
-  }
-];
-
-/**
- * This function is called to simulate camera effects
- */
-function simulateCameraEffect(document, effect) {
-  const window = document.defaultView;
-  if (effect === "shutter") {
-    if (Services.prefs.getBoolPref("devtools.screenshot.audio.enabled")) {
-      const audioCamera = new window.Audio("resource://devtools/client/themes/audio/shutter.wav");
-      audioCamera.play();
-    }
-  }
-  if (effect == "flash") {
-    const frames = Cu.cloneInto({ opacity: [ 0, 1 ] }, window);
-    document.documentElement.animate(frames, CONTAINER_FLASHING_DURATION);
-  }
-}
-
-/**
- * This function simply handles the --delay argument before calling
- * createScreenshotData
- */
-function captureScreenshot(args, document) {
-  if (args.delay > 0) {
-    return new Promise((resolve, reject) => {
-      document.defaultView.setTimeout(() => {
-        createScreenshotData(document, args).then(resolve, reject);
-      }, args.delay * 1000);
-    });
-  }
-  return createScreenshotData(document, args);
-}
-
-/**
- * There are several possible destinations for the screenshot, SKIP is used
- * in saveScreenshot() whenever one of them is not used
- */
-const SKIP = Promise.resolve();
-
-/**
- * Save the captured screenshot to one of several destinations.
- */
-function saveScreenshot(args, context, reply) {
-  const fileNeeded = args.filename != FILENAME_DEFAULT_VALUE ||
-    (!args.imgur && !args.clipboard) || args.file;
-
-  return Promise.all([
-    args.clipboard ? saveToClipboard(context, reply) : SKIP,
-    args.imgur ? uploadToImgur(reply) : SKIP,
-    fileNeeded ? saveToFile(context, reply) : SKIP,
-  ]).then(() => reply);
-}
-
-/**
- * This does the dirty work of creating a base64 string out of an
- * area of the browser window
- */
-function createScreenshotData(document, args) {
-  const window = document.defaultView;
-  let left = 0;
-  let top = 0;
-  let width;
-  let height;
-  const currentX = window.scrollX;
-  const currentY = window.scrollY;
-
-  let filename = getFilename(args.filename);
-
-  if (args.fullpage) {
-    // Bug 961832: GCLI screenshot shows fixed position element in wrong
-    // position if we don't scroll to top
-    window.scrollTo(0, 0);
-    width = window.innerWidth + window.scrollMaxX - window.scrollMinX;
-    height = window.innerHeight + window.scrollMaxY - window.scrollMinY;
-    filename = filename.replace(".png", "-fullpage.png");
-  } else if (args.selector) {
-    ({ top, left, width, height } = getRect(window, args.selector, window));
-  } else {
-    left = window.scrollX;
-    top = window.scrollY;
-    width = window.innerWidth;
-    height = window.innerHeight;
-  }
-
-  // Only adjust for scrollbars when considering the full window
-  if (!args.selector) {
-    const winUtils = window.windowUtils;
-    const scrollbarHeight = {};
-    const scrollbarWidth = {};
-    winUtils.getScrollbarSize(false, scrollbarWidth, scrollbarHeight);
-    width -= scrollbarWidth.value;
-    height -= scrollbarHeight.value;
-  }
-
-  const canvas = document.createElementNS("http://www.w3.org/1999/xhtml", "canvas");
-  const ctx = canvas.getContext("2d");
-  const ratio = args.dpr ? args.dpr : window.devicePixelRatio;
-  canvas.width = width * ratio;
-  canvas.height = height * ratio;
-  ctx.scale(ratio, ratio);
-  ctx.drawWindow(window, left, top, width, height, "#fff");
-  const data = canvas.toDataURL("image/png", "");
-
-  // See comment above on bug 961832
-  if (args.fullpage) {
-    window.scrollTo(currentX, currentY);
-  }
-
-  simulateCameraEffect(document, "flash");
-
-  return Promise.resolve({
-    destinations: [],
-    data: data,
-    height: height,
-    width: width,
-    filename: filename,
-  });
-}
-
-/**
- * We may have a filename specified in args, or we might have to generate
- * one.
- */
-function getFilename(defaultName) {
-  // Create a name for the file if not present
-  if (defaultName != FILENAME_DEFAULT_VALUE) {
-    return defaultName;
-  }
-
-  const date = new Date();
-  let dateString = date.getFullYear() + "-" + (date.getMonth() + 1) +
-                  "-" + date.getDate();
-  dateString = dateString.split("-").map(function(part) {
-    if (part.length == 1) {
-      part = "0" + part;
-    }
-    return part;
-  }).join("-");
-
-  const timeString = date.toTimeString().replace(/:/g, ".").split(" ")[0];
-  return l10n.lookupFormat("screenshotGeneratedFilename",
-                           [ dateString, timeString ]) + ".png";
-}
-
-/**
- * Save the image data to the clipboard. This returns a promise, so it can
- * be treated exactly like imgur / file processing, but it's really sync
- * for now.
- */
-function saveToClipboard(context, reply) {
-  return new Promise(resolve => {
-    try {
-      const channel = NetUtil.newChannel({
-        uri: reply.data,
-        loadUsingSystemPrincipal: true,
-        contentPolicyType: Ci.nsIContentPolicy.TYPE_INTERNAL_IMAGE
-      });
-      const input = channel.open2();
-
-      const loadContext = context.environment.chromeWindow.docShell
-                                 .QueryInterface(Ci.nsILoadContext);
-
-      const callback = {
-        onImageReady(container, status) {
-          if (!container) {
-            console.error("imgTools.decodeImageAsync failed");
-            reply.destinations.push(l10n.lookup("screenshotErrorCopying"));
-            resolve();
-            return;
-          }
-
-          try {
-            const wrapped = Cc["@mozilla.org/supports-interface-pointer;1"]
-                              .createInstance(Ci.nsISupportsInterfacePointer);
-            wrapped.data = container;
-
-            const trans = Cc["@mozilla.org/widget/transferable;1"]
-                            .createInstance(Ci.nsITransferable);
-            trans.init(loadContext);
-            trans.addDataFlavor(channel.contentType);
-            trans.setTransferData(channel.contentType, wrapped, -1);
-
-            Services.clipboard.setData(trans, null, Ci.nsIClipboard.kGlobalClipboard);
-
-            reply.destinations.push(l10n.lookup("screenshotCopied"));
-          } catch (ex) {
-            console.error(ex);
-            reply.destinations.push(l10n.lookup("screenshotErrorCopying"));
-          }
-          resolve();
-        }
-      };
-
-      const threadManager = Cc["@mozilla.org/thread-manager;1"].getService();
-      const imgTools = Cc["@mozilla.org/image/tools;1"]
-                          .getService(Ci.imgITools);
-      imgTools.decodeImageAsync(input, channel.contentType, callback,
-                                threadManager.currentThread);
-    } catch (ex) {
-      console.error(ex);
-      reply.destinations.push(l10n.lookup("screenshotErrorCopying"));
-      resolve();
-    }
-  });
-}
-
-/**
- * Upload screenshot data to Imgur, returning a promise of a URL (as a string)
- */
-function uploadToImgur(reply) {
-  return new Promise((resolve, reject) => {
-    const xhr = new XMLHttpRequest();
-    const fd = new FormData();
-    fd.append("image", reply.data.split(",")[1]);
-    fd.append("type", "base64");
-    fd.append("title", reply.filename);
-
-    const postURL = Services.prefs.getCharPref("devtools.gcli.imgurUploadURL");
-    const clientID = "Client-ID " +
-                     Services.prefs.getCharPref("devtools.gcli.imgurClientID");
-
-    xhr.open("POST", postURL);
-    xhr.setRequestHeader("Authorization", clientID);
-    xhr.send(fd);
-    xhr.responseType = "json";
-
-    xhr.onreadystatechange = function() {
-      if (xhr.readyState == 4) {
-        if (xhr.status == 200) {
-          reply.href = xhr.response.data.link;
-          reply.destinations.push(l10n.lookupFormat("screenshotImgurUploaded",
-                                                    [ reply.href ]));
-        } else {
-          reply.destinations.push(l10n.lookup("screenshotImgurError"));
-        }
-
-        resolve();
-      }
-    };
-  });
-}
-
-/**
- * Progress listener that forwards calls to a transfer object.
- *
- * This is used below in saveToFile to forward progress updates from the
- * nsIWebBrowserPersist object that does the actual saving to the nsITransfer
- * which just represents the operation for the Download Manager.  This keeps the
- * Download Manager updated on saving progress and completion, so that it gives
- * visual feedback from the downloads toolbar button when the save is done.
- *
- * It also allows the browser window to show auth prompts if needed (should not
- * be needed for saving screenshots).
- *
- * This code is borrowed directly from contentAreaUtils.js.
- */
-function DownloadListener(win, transfer) {
-  this.window = win;
-  this.transfer = transfer;
-
-  // For most method calls, forward to the transfer object.
-  for (const name in transfer) {
-    if (name != "QueryInterface" &&
-        name != "onStateChange") {
-      this[name] = (...args) => transfer[name].apply(transfer, args);
-    }
-  }
-
-  // Allow saveToFile to await completion for error handling
-  this._completedDeferred = defer();
-  this.completed = this._completedDeferred.promise;
-}
-
-DownloadListener.prototype = {
-  QueryInterface: ChromeUtils.generateQI(["nsIInterfaceRequestor",
-                                          "nsIWebProgressListener",
-                                          "nsIWebProgressListener2"]),
-
-  getInterface: function(iid) {
-    if (iid.equals(Ci.nsIAuthPrompt) ||
-        iid.equals(Ci.nsIAuthPrompt2)) {
-      const ww = Cc["@mozilla.org/embedcomp/window-watcher;1"]
-                 .getService(Ci.nsIPromptFactory);
-      return ww.getPrompt(this.window, iid);
-    }
-
-    throw Cr.NS_ERROR_NO_INTERFACE;
-  },
-
-  onStateChange: function(webProgress, request, state, status) {
-    // Check if the download has completed
-    if ((state & Ci.nsIWebProgressListener.STATE_STOP) &&
-        (state & Ci.nsIWebProgressListener.STATE_IS_NETWORK)) {
-      if (status == Cr.NS_OK) {
-        this._completedDeferred.resolve();
-      } else {
-        this._completedDeferred.reject();
-      }
-    }
-
-    this.transfer.onStateChange.apply(this.transfer, arguments);
-  }
-};
-
-/**
- * Save the screenshot data to disk, returning a promise which is resolved on
- * completion.
- */
-var saveToFile = Task.async(function* (context, reply) {
-  const document = context.environment.chromeDocument;
-  const window = context.environment.chromeWindow;
-
-  // Check there is a .png extension to filename
-  if (!reply.filename.match(/.png$/i)) {
-    reply.filename += ".png";
-  }
-
-  const downloadsDir = yield Downloads.getPreferredDownloadsDirectory();
-  const downloadsDirExists = yield OS.File.exists(downloadsDir);
-  if (downloadsDirExists) {
-    // If filename is absolute, it will override the downloads directory and
-    // still be applied as expected.
-    reply.filename = OS.Path.join(downloadsDir, reply.filename);
-  }
-
-  const sourceURI = Services.io.newURI(reply.data);
-  const targetFile = new FileUtils.File(reply.filename);
-  const targetFileURI = Services.io.newFileURI(targetFile);
-
-  // Create download and track its progress.
-  // This is adapted from saveURL in contentAreaUtils.js, but simplified greatly
-  // and modified to allow saving to arbitrary paths on disk.  Using these
-  // objects as opposed to just writing with OS.File allows us to tie into the
-  // download manager to record a download entry and to get visual feedback from
-  // the downloads toolbar button when the save is done.
-  const nsIWBP = Ci.nsIWebBrowserPersist;
-  const flags = nsIWBP.PERSIST_FLAGS_REPLACE_EXISTING_FILES |
-                nsIWBP.PERSIST_FLAGS_BYPASS_CACHE |
-                nsIWBP.PERSIST_FLAGS_AUTODETECT_APPLY_CONVERSION;
-  const isPrivate =
-    PrivateBrowsingUtils.isContentWindowPrivate(document.defaultView);
-  const persist = Cc["@mozilla.org/embedding/browser/nsWebBrowserPersist;1"]
-                  .createInstance(Ci.nsIWebBrowserPersist);
-  persist.persistFlags = flags;
-  const tr = Cc["@mozilla.org/transfer;1"].createInstance(Ci.nsITransfer);
-  tr.init(sourceURI,
-          targetFileURI,
-          "",
-          null,
-          null,
-          null,
-          persist,
-          isPrivate);
-  const listener = new DownloadListener(window, tr);
-  persist.progressListener = listener;
-  const principal = Services.scriptSecurityManager.getSystemPrincipal();
-  persist.savePrivacyAwareURI(sourceURI,
-                              principal,
-                              0,
-                              document.documentURIObject,
-                              Ci.nsIHttpChannel.REFERRER_POLICY_UNSET,
-                              null,
-                              null,
-                              targetFileURI,
-                              isPrivate);
-
-  try {
-    // Await successful completion of the save via the listener
-    yield listener.completed;
-    reply.destinations.push(l10n.lookup("screenshotSavedToFile") +
-                            ` "${reply.filename}"`);
-  } catch (ex) {
-    console.error(ex);
-    reply.destinations.push(l10n.lookup("screenshotErrorSavingToFile") + " " +
-                            reply.filename);
-  }
-});
deleted file mode 100644
--- a/devtools/shared/gcli/moz.build
+++ /dev/null
@@ -1,24 +0,0 @@
-# -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
-# vim: set filetype=python:
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-DIRS += [
-    'commands',
-    'source/lib/gcli',
-    'source/lib/gcli/connectors',
-    'source/lib/gcli/converters',
-    'source/lib/gcli/commands',
-    'source/lib/gcli/fields',
-    'source/lib/gcli/languages',
-    'source/lib/gcli/mozui',
-    'source/lib/gcli/types',
-    'source/lib/gcli/ui',
-    'source/lib/gcli/util',
-]
-
-DevToolsModules(
-    'command-state.js',
-    'templater.js'
-)
deleted file mode 100644
--- a/devtools/shared/gcli/source/LICENSE
+++ /dev/null
@@ -1,202 +0,0 @@
-
-                                 Apache License
-                           Version 2.0, January 2004
-                        http://www.apache.org/licenses/
-
-   TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION
-
-   1. Definitions.
-
-      "License" shall mean the terms and conditions for use, reproduction,
-      and distribution as defined by Sections 1 through 9 of this document.
-
-      "Licensor" shall mean the copyright owner or entity authorized by
-      the copyright owner that is granting the License.
-
-      "Legal Entity" shall mean the union of the acting entity and all
-      other entities that control, are controlled by, or are under common
-      control with that entity. For the purposes of this definition,
-      "control" means (i) the power, direct or indirect, to cause the
-      direction or management of such entity, whether by contract or
-      otherwise, or (ii) ownership of fifty percent (50%) or more of the
-      outstanding shares, or (iii) beneficial ownership of such entity.
-
-      "You" (or "Your") shall mean an individual or Legal Entity
-      exercising permissions granted by this License.
-
-      "Source" form shall mean the preferred form for making modifications,
-      including but not limited to software source code, documentation
-      source, and configuration files.
-
-      "Object" form shall mean any form resulting from mechanical
-      transformation or translation of a Source form, including but
-      not limited to compiled object code, generated documentation,
-      and conversions to other media types.
-
-      "Work" shall mean the work of authorship, whether in Source or
-      Object form, made available under the License, as indicated by a
-      copyright notice that is included in or attached to the work
-      (an example is provided in the Appendix below).
-
-      "Derivative Works" shall mean any work, whether in Source or Object
-      form, that is based on (or derived from) the Work and for which the
-      editorial revisions, annotations, elaborations, or other modifications
-      represent, as a whole, an original work of authorship. For the purposes
-      of this License, Derivative Works shall not include works that remain
-      separable from, or merely link (or bind by name) to the interfaces of,
-      the Work and Derivative Works thereof.
-
-      "Contribution" shall mean any work of authorship, including
-      the original version of the Work and any modifications or additions
-      to that Work or Derivative Works thereof, that is intentionally
-      submitted to Licensor for inclusion in the Work by the copyright owner
-      or by an individual or Legal Entity authorized to submit on behalf of
-      the copyright owner. For the purposes of this definition, "submitted"
-      means any form of electronic, verbal, or written communication sent
-      to the Licensor or its representatives, including but not limited to
-      communication on electronic mailing lists, source code control systems,
-      and issue tracking systems that are managed by, or on behalf of, the
-      Licensor for the purpose of discussing and improving the Work, but
-      excluding communication that is conspicuously marked or otherwise
-      designated in writing by the copyright owner as "Not a Contribution."
-
-      "Contributor" shall mean Licensor and any individual or Legal Entity
-      on behalf of whom a Contribution has been received by Licensor and
-      subsequently incorporated within the Work.
-
-   2. Grant of Copyright License. Subject to the terms and conditions of
-      this License, each Contributor hereby grants to You a perpetual,
-      worldwide, non-exclusive, no-charge, royalty-free, irrevocable
-      copyright license to reproduce, prepare Derivative Works of,
-      publicly display, publicly perform, sublicense, and distribute the
-      Work and such Derivative Works in Source or Object form.
-
-   3. Grant of Patent License. Subject to the terms and conditions of
-      this License, each Contributor hereby grants to You a perpetual,
-      worldwide, non-exclusive, no-charge, royalty-free, irrevocable
-      (except as stated in this section) patent license to make, have made,
-      use, offer to sell, sell, import, and otherwise transfer the Work,
-      where such license applies only to those patent claims licensable
-      by such Contributor that are necessarily infringed by their
-      Contribution(s) alone or by combination of their Contribution(s)
-      with the Work to which such Contribution(s) was submitted. If You
-      institute patent litigation against any entity (including a
-      cross-claim or counterclaim in a lawsuit) alleging that the Work
-      or a Contribution incorporated within the Work constitutes direct
-      or contributory patent infringement, then any patent licenses
-      granted to You under this License for that Work shall terminate
-      as of the date such litigation is filed.
-
-   4. Redistribution. You may reproduce and distribute copies of the
-      Work or Derivative Works thereof in any medium, with or without
-      modifications, and in Source or Object form, provided that You
-      meet the following conditions:
-
-      (a) You must give any other recipients of the Work or
-          Derivative Works a copy of this License; and
-
-      (b) You must cause any modified files to carry prominent notices
-          stating that You changed the files; and
-
-      (c) You must retain, in the Source form of any Derivative Works
-          that You distribute, all copyright, patent, trademark, and
-          attribution notices from the Source form of the Work,
-          excluding those notices that do not pertain to any part of
-          the Derivative Works; and
-
-      (d) If the Work includes a "NOTICE" text file as part of its
-          distribution, then any Derivative Works that You distribute must
-          include a readable copy of the attribution notices contained
-          within such NOTICE file, excluding those notices that do not
-          pertain to any part of the Derivative Works, in at least one
-          of the following places: within a NOTICE text file distributed
-          as part of the Derivative Works; within the Source form or
-          documentation, if provided along with the Derivative Works; or,
-          within a display generated by the Derivative Works, if and
-          wherever such third-party notices normally appear. The contents
-          of the NOTICE file are for informational purposes only and
-          do not modify the License. You may add Your own attribution
-          notices within Derivative Works that You distribute, alongside
-          or as an addendum to the NOTICE text from the Work, provided
-          that such additional attribution notices cannot be construed
-          as modifying the License.
-
-      You may add Your own copyright statement to Your modifications and
-      may provide additional or different license terms and conditions
-      for use, reproduction, or distribution of Your modifications, or
-      for any such Derivative Works as a whole, provided Your use,
-      reproduction, and distribution of the Work otherwise complies with
-      the conditions stated in this License.
-
-   5. Submission of Contributions. Unless You explicitly state otherwise,
-      any Contribution intentionally submitted for inclusion in the Work
-      by You to the Licensor shall be under the terms and conditions of
-      this License, without any additional terms or conditions.
-      Notwithstanding the above, nothing herein shall supersede or modify
-      the terms of any separate license agreement you may have executed
-      with Licensor regarding such Contributions.
-
-   6. Trademarks. This License does not grant permission to use the trade
-      names, trademarks, service marks, or product names of the Licensor,
-      except as required for reasonable and customary use in describing the
-      origin of the Work and reproducing the content of the NOTICE file.
-
-   7. Disclaimer of Warranty. Unless required by applicable law or
-      agreed to in writing, Licensor provides the Work (and each
-      Contributor provides its Contributions) on an "AS IS" BASIS,
-      WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
-      implied, including, without limitation, any warranties or conditions
-      of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A
-      PARTICULAR PURPOSE. You are solely responsible for determining the
-      appropriateness of using or redistributing the Work and assume any
-      risks associated with Your exercise of permissions under this License.
-
-   8. Limitation of Liability. In no event and under no legal theory,
-      whether in tort (including negligence), contract, or otherwise,
-      unless required by applicable law (such as deliberate and grossly
-      negligent acts) or agreed to in writing, shall any Contributor be
-      liable to You for damages, including any direct, indirect, special,
-      incidental, or consequential damages of any character arising as a
-      result of this License or out of the use or inability to use the
-      Work (including but not limited to damages for loss of goodwill,
-      work stoppage, computer failure or malfunction, or any and all
-      other commercial damages or losses), even if such Contributor
-      has been advised of the possibility of such damages.
-
-   9. Accepting Warranty or Additional Liability. While redistributing
-      the Work or Derivative Works thereof, You may choose to offer,
-      and charge a fee for, acceptance of support, warranty, indemnity,
-      or other liability obligations and/or rights consistent with this
-      License. However, in accepting such obligations, You may act only
-      on Your own behalf and on Your sole responsibility, not on behalf
-      of any other Contributor, and only if You agree to indemnify,
-      defend, and hold each Contributor harmless for any liability
-      incurred by, or claims asserted against, such Contributor by reason
-      of your accepting any such warranty or additional liability.
-
-   END OF TERMS AND CONDITIONS
-
-   APPENDIX: How to apply the Apache License to your work.
-
-      To apply the Apache License to your work, attach the following
-      boilerplate notice, with the fields enclosed by brackets "[]"
-      replaced with your own identifying information. (Don't include
-      the brackets!)  The text should be enclosed in the appropriate
-      comment syntax for the file format. We also recommend that a
-      file or class name and description of purpose be included on the
-      same "printed page" as the copyright notice for easier
-      identification within third-party archives.
-
-   Copyright [yyyy] [name of copyright owner]
-
-   Licensed under the Apache License, Version 2.0 (the "License");
-   you may not use this file except in compliance with the License.
-   You may obtain a copy of the License at
-
-       http://www.apache.org/licenses/LICENSE-2.0
-
-   Unless required by applicable law or agreed to in writing, software
-   distributed under the License is distributed on an "AS IS" BASIS,
-   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-   See the License for the specific language governing permissions and
-   limitations under the License.
deleted file mode 100644
--- a/devtools/shared/gcli/source/docs/design.md
+++ /dev/null
@@ -1,102 +0,0 @@
-
-# The Design of GCLI
-
-## Design Goals
-
-GCLI should be:
-
-- primarily for technical users.
-- as fast as a traditional CLI. It should be possible to put your head down,
-  and look at the keyboard and use GCLI 'blind' at full speed without making
-  mistakes.
-- principled about the way it encourages people to build commands. There is
-  benefit from unifying the underlying concepts.
-- automatically helpful.
-
-GCLI should not attempt to:
-
-- convert existing GUI users to a CLI.
-- use natural language input. The closest we should get to natural language is
-  thinking of commands as ```verb noun --adjective```.
-- gain a touch based interface. Whilst it's possible (even probable) that touch
-  can provide further benefits to command line users, that can wait while we
-  catch up with 1985.
-- slavishly follow the syntax of existing commands, predictability is more
-  important.
-- be a programming language. Shell scripts are mini programming languages but
-  we have JavaScript sat just next door. It's better to integrate than compete.
-
-
-## Design Challenges
-
-What has changed since 1970 that might cause us to make changes to the design
-of the command line?
-
-
-### Connection limitations
-
-Unix pre-dates the Internet and treats almost everything as a file. Since the
-Internet it could be more useful to use URIs as ways to identify sources of data.
-
-
-### Memory limitations
-
-Modern computers have something like 6 orders of magnitude more memory than the
-PDP-7 on which Unix was developed. Innovations like stdin/stdout and pipes are
-ways to connect systems without long-term storage of the results. The ability
-to store results for some time (potentially in more than one format)
-significantly reduces the need for these concepts. We should make the results
-of past commands addressable for re-use at a later time.
-
-There are a number of possible policies for eviction of items from the history.
-We should investigate options other than a simple stack.
-
-
-### Multi-tasking limitations
-
-Multi-tasking was a problem in 1970; the problem was getting a computer to do
-many jobs on 1 core. Today the problem is getting a computer to do one job on
-many cores. However we're stuck with this legacy in 2 ways. Firstly that the
-default is to force everything to wait until the previous job is finished, but
-more importantly that output from parallel jobs frequently collides
-
-    $ find / -ctime 5d -print &
-    $ find / -uid 0 -print &
-    // good luck working out what came from where
-
-    $ tail -f logfile.txt &
-    $ vi main.c
-    // have a nice time editing that file
-
-GCLI should allow commands to be asynchronous and will provide UI elements to
-inform the user of job completion. It will also keep asynchronous command
-output contained within it's own display area.
-
-
-### Output limitations
-
-The PDP-7 had a teletype. There is something like 4 orders of magnitude more
-information that can be displayed on a modern display than a 80x24 character
-based console. We can use this flexibility to provide better help to the user
-in entering their command.
-
-The additional display richness can also allow interaction with result output.
-Command output can include links to follow-up commands, and even ask for
-additional input. (e.g. "your search returned zero results do you want to try
-again with a different search string")
-
-There is no reason why output must be static. For example, it could be
-informative to see the results of an "ls" command alter given changes made by
-subsequent commands. (It should be noted that there are times when historical
-information is important too)
-
-
-### Integration limitations
-
-In 1970, command execution meant retrieving a program from storage, and running
-it. This required minimal interaction between the command line processor and
-the program being run, and was good for resource constrained systems.
-This lack of interaction resulted in the processing of command line arguments
-being done everywhere, when the task was better suited to command line.
-We should provide metadata about the commands being run, to allow the command
-line to process, interpret and provide help on the input.
deleted file mode 100644
--- a/devtools/shared/gcli/source/docs/developing-gcli.md
+++ /dev/null
@@ -1,213 +0,0 @@
-
-# Developing GCLI
-
-## About the code
-
-The majority of the GCLI source is stored in the ``lib`` directory.
-
-The ``docs`` directory contains documentation.
-The ``scripts`` directory contains RequireJS that GCLI uses.
-The ``build`` directory contains files used when creating builds.
-The ``mozilla`` directory contains the mercurial patch queue of patches to apply
-to mozilla-central.
-The ``selenium-tests`` directory contains selenium web-page integration tests.
-
-The source in the ``lib`` directory is split into 4 sections:
-
-- ``lib/demo`` contains commands used in the demo page. It is not needed except
-  for demo purposes.
-- ``lib/test`` contains a small test harness for testing GCLI.
-- ``lib/gclitest`` contains tests that run in the test harness
-- ``lib/gcli`` contains the actual meat
-
-GCLI is split into a UI portion and a Model/Controller portion.
-
-
-## The GCLI Model
-
-The heart of GCLI is a ``Requisition``, which is an AST for the input. A
-``Requisition`` is a command that we'd like to execute, and we're filling out
-all the inputs required to execute the command.
-
-A ``Requisition`` has a ``Command`` that is to be executed. Each Command has a
-number of ``Parameter``s, each of which has a name and a type as detailed
-above.
-
-As you type, your input is split into ``Argument``s, which are then assigned to
-``Parameter``s using ``Assignment``s. Each ``Assignment`` has a ``Conversion``
-which stores the input argument along with the value that is was converted into
-according to the type of the parameter.
-
-There are special assignments called ``CommandAssignment`` which the
-``Requisition`` uses to link to the command to execute, and
-``UnassignedAssignment``used to store arguments that do not have a parameter
-to be assigned to.
-
-
-## The GCLI UI
-
-There are several components of the GCLI UI. Each can have a script portion,
-some template HTML and a CSS file. The template HTML is processed by
-``domtemplate`` before use.
-
-DomTemplate is fully documented in [it's own repository]
-(https://github.com/joewalker/domtemplate).
-
-The components are:
-
-- ``Inputter`` controls the input field, processing special keyboard events and
-  making sure that it stays in sync with the Requisition.
-- ``Completer`` updates a div that is located behind the input field and used
-  to display completion advice and hint highlights. It is stored in
-  completer.js.
-- ``Display`` is responsible for containing the popup hints that are displayed
-  above the command line. Typically Display contains a Hinter and a RequestsView
-  although these are not both required. Display itself is optional, and isn't
-  planned for use in the first release of GCLI in Firefox.
-- ``Hinter`` Is used to display input hints. It shows either a Menu or an
-  ArgFetch component depending on the state of the Requisition
-- ``Menu`` is used initially to select the command to be executed. It can act
-  somewhat like the Start menu on windows.
-- ``ArgFetch`` Once the command to be executed has been selected, ArgFetch
-  shows a 'dialog' allowing the user to enter the parameters to the selected
-  command.
-- ``RequestsView`` Contains a set of ``RequestView`` components, each of which
-  displays a command that has been invoked. RequestsView is a poor name, and
-  should better be called ReportView
-
-ArgFetch displays a number of Fields. There are fields for most of the Types
-discussed earlier. See 'Writing Fields' above for more information.
-
-
-## Testing
-
-GCLI contains 2 test suites:
-
-- JS level testing is run with the ``test`` command. The tests are located in
-  ``lib/gclitest`` and they use the test runner in ``lib/test``. This is fairly
-  comprehensive, however it does not do UI level testing.
-  If writing a new test it needs to be registered in ``lib/gclitest/index``.
-  For an example of how to write tests, see ``lib/gclitest/testSplit.js``.
-  The test functions are implemented in ``lib/test/assert``.
-- Browser integration tests are included in ``browser_webconsole_gcli_*.js``,
-  in ``toolkit/components/console/hudservice/tests/browser``. These are
-  run with the rest of the Mozilla test suite.
-
-
-## Coding Conventions
-
-The coding conventions for the GCLI project come from the Bespin/Skywriter and
-Ace projects. They are roughly [Crockford]
-(http://javascript.crockford.com/code.html) with a few exceptions and
-additions:
-
-* ``var`` does not need to be at the top of each function, we'd like to move
-  to ``let`` when it's generally available, and ``let`` doesn't have the same
-  semantic twists as ``var``.
-
-* Strings are generally enclosed in single quotes.
-
-* ``eval`` is to be avoided, but we don't declare it evil.
-
-The [Google JavaScript conventions]
-(https://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml) are
-more detailed, we tend to deviate in:
-
-* Custom exceptions: We generally just use ``throw new Error('message');``
-
-* Multi-level prototype hierarchies: Allowed; we don't have ``goog.inherits()``
-
-* ``else`` begins on a line by itself:
-
-        if (thing) {
-          doThis();
-        }
-        else {
-          doThat();
-        }
-
-
-## Startup
-
-Internally GCLI modules have ``startup()``/``shutdown()`` functions which are
-called on module init from the top level ``index.js`` of that 'package'.
-
-In order to initialize a package all that is needed is to require the package
-index (e.g. ``require('package/index')``).
-
-The ``shutdown()`` function was useful when GCLI was used in Bespin as part of
-dynamic registration/de-registration. It is not known if this feature will be
-useful in the future. So it has not been entirely removed, it may be at some
-future date.
-
-
-## Running the Unit Tests
-
-Start the GCLI static server:
-
-    cd path/to/gcli
-    node gcli.js
-
-Now point your browser to http://localhost:9999/localtest.html. When the page
-loads the tests will be automatically run outputting to the console, or you can
-enter the ``test`` command to run the unit tests.
-
-
-## Contributing Code
-
-Please could you do the following to help minimize the amount of rework that we
-do:
-
-1. Check the unit tests run correctly (see **Running the Unit Tests** above)
-2. Check the code follows the style guide. At a minimum it should look like the
-   code around it. For more detailed notes, see **Coding Conventions** above
-3. Help me review your work by using good commit comments. Which means 2 things
-   * Well formatted messages, i.e. 50 char summary including bug tag, followed
-     by a blank line followed by a more in-depth message wrapped to 72 chars
-     per line. This is basically the format used by the Linux Kernel. See the
-     [commit log](https://github.com/joewalker/gcli/commits/master) for
-     examples. The be extra helpful, please use the "shortdesc-BUGNUM: " if
-     possible which also helps in reviews.
-   * Commit your changes as a story. Make it easy for me to understand the
-     changes that you've made.
-4. Sign your work. To improve tracking of who did what, we follow the sign-off
-   procedure used in the Linux Kernel.
-   The sign-off is a simple line at the end of the explanation for the
-   patch, which certifies that you wrote it or otherwise have the right to
-   pass it on as an open-source patch.  The rules are pretty simple: if you
-   can certify the below:
-
-        Developer's Certificate of Origin 1.1
-
-        By making a contribution to this project, I certify that:
-
-        (a) The contribution was created in whole or in part by me and I
-            have the right to submit it under the open source license
-            indicated in the file; or
-
-        (b) The contribution is based upon previous work that, to the best
-            of my knowledge, is covered under an appropriate open source
-            license and I have the right under that license to submit that
-            work with modifications, whether created in whole or in part
-            by me, under the same open source license (unless I am
-            permitted to submit under a different license), as indicated
-            in the file; or
-
-        (c) The contribution was provided directly to me by some other
-            person who certified (a), (b) or (c) and I have not modified
-            it.
-
-        (d) I understand and agree that this project and the contribution
-            are public and that a record of the contribution (including all
-            personal information I submit with it, including my sign-off) is
-            maintained indefinitely and may be redistributed consistent with
-            this project or the open source license(s) involved.
-
-   then you just add a line saying
-
-        Signed-off-by: Random J Developer <random@developer.example.org>
-
-   using your real name (sorry, no pseudonyms or anonymous contributions.)
-
-Thanks for wanting to contribute code.
-
deleted file mode 100644
--- a/devtools/shared/gcli/source/docs/index.md
+++ /dev/null
@@ -1,150 +0,0 @@
-
-# About GCLI
-
-## GCLI is a Graphical Command Line Interpreter.
-
-GCLI is a command line for modern computers. When command lines were invented,
-computers were resource-limited, disconnected systems with slow multi-tasking
-and poor displays. The design of the Unix CLI made sense in 1970, but over 40
-years on, considering the pace of change, there are many improvements we can
-make.
-
-CLIs generally suffer from poor discoverability; It's hard when faced with a
-blank command line to work out what to do. As a result the majority of programs
-today use purely graphical user interfaces, however in doing so, they lose some
-of the benefits of CLIs. CLIs are still used because generally, in the hands of
-a skilled user they are faster, and have a wider range of available options.
-
-GCLI attempts to get the best of the GUI world and the CLI world to produce
-something that is both easy to use and learn as well as fast and powerful.
-
-GCLI has a type system to help ensure that users are inputting valid commands
-and to enable us to provide sensible context sensitive help. GCLI provides
-integration with JavaScript rather than being an alternative (like CoffeeScript).
-
-
-## History
-
-GCLI was born as part of the
-[Bespin](http://ajaxian.com/archives/canvas-for-a-text-editor) project and was
-[discussed at the time](http://j.mp/bespin-cli). The command line component
-survived the rename of Bepsin to Skywriter and the merger with Ace, got a name
-of it's own (Cockpit) which didn't last long before the project was named GCLI.
-It is now being used in the Firefox's web console where it doesn't have a
-separate identity but it's still called GCLI outside of Firefox. It is also
-used in [Eclipse Orion](http://www.eclipse.org/orion/).
-
-
-## Environments
-
-GCLI is designed to work in a number of environments:
-
-1. As a component of Firefox developer tools.
-2. As an adjunct to Orion/Ace and other online editors.
-3. As a plugin to any web-page wishing to provide its own set of commands.
-4. As part of a standalone web browser extension with it's own set of commands.
-
-
-## Related Pages
-
-Other sources of GCLI documentation:
-
-- [Writing Commands](writing-commands.md)
-- [Writing Types](writing-types.md)
-- [Developing GCLI](developing-gcli.md)
-- [Writing Tests](writing-tests.md) / [Running Tests](running-tests.md)
-- [The Design of GCLI](design.md)
-- Source
-  - The most up-to-date source is in [this Github repository](https://github.com/joewalker/gcli/).
-  - When a feature is 'done' it's merged into the [Mozilla clone](https://github.com/mozilla/gcli/).
-  - From which it flows into [Mozilla Central](https://hg.mozilla.org/mozilla-central/file/tip/devtools/client/commandline).
-- [Demo of GCLI](http://mozilla.github.com/gcli/) with an arbitrary set of demo
-  commands
-- Other Documentation
-  - [Embedding docs](https://github.com/mozilla/gcli/blob/master/docs/index.md)
-  - [Status page](http://mozilla.github.com/devtools/2011/status.html#gcli)
-
-
-## Accessibility
-
-GCLI uses ARIA roles to guide a screen-reader as to the important sections to
-voice. We welcome [feedback on how these roles are implemented](https://bugzilla.mozilla.org/enter_bug.cgi?product=Firefox&component=Developer+Tools:+Graphic+Commandline+and+Toolbar&rep_platform=All&op_sys=All&short_desc=GCLI).
-
-The command line uses TAB as a method of completing current input, this
-prevents use of TAB for keyboard navigation. Instead of using TAB to move to
-the next field you can use F6. In addition to F6, ALT+TAB, CTRL+TAB, META+TAB
-make an attempt to move the focus on. How well this works depends on your
-OS/browser combination.
-
-
-## Embedding GCLI
-
-There are 3 basic steps in using GCLI in your system.
-
-1. Import a GCLI JavaScript file.
-   For serious use of GCLI you are likely to be creating a custom build (see
-   below) however if you just want to have a quick play, you can use
-   ``gcli-uncompressed.js`` from [the gh-pages branch of GCLI]
-   (https://github.com/mozilla/gcli/tree/gh-pages)
-   Just place the following wherever you place your script files.
-
-        <script src="path/to/gcli-uncompressed.js" type="text/javascript"></script>
-
-2. Having imported GCLI, we need to tell it where to display. The simplest
-   method is to include an elements with the id of ``gcli-input`` and
-   ``gcli-display``.
-
-        <input id="gcli-input" type="text"/>
-        <div id="gcli-display"></div>
-
-3. Tell GCLI what commands to make available. See the sections on Writing
-   Commands, Writing Types and Writing Fields for more information.
-
-   GCLI uses the CommonJS AMD format for it's files, so a 'require' statement
-   is needed to get started.
-
-        require([ 'gcli/index' ], function(gcli) {
-          gcli.add(...);         // Register custom commands/types/etc
-          gcli.createTerminal(); // Create a user interface
-        });
-
-   The createTerminal() function takes an ``options`` objects which allows
-   customization. At the current time the documentation of these object is left
-   to the source.
-
-
-## Backwards Compatibility
-
-The goals of the GCLI project are:
-
-- Aim for very good backwards compatibility with code required from an
-  'index' module. This means we will not break code without a cycle of
-  deprecation warnings.
-
-  There are currently 3 'index' modules:
-  - gcli/index (all you need to get started with GCLI)
-  - demo/index (a number of demo commands)
-  - gclitest/index (GCLI test suite)
-
-  Code from these modules uses the module pattern to prevent access to internal
-  functions, so in essence, if you can get to it from an index module, you
-  should be ok.
-
-- We try to avoid needless change to other modules, however we don't make any
-  promises, and don't provide a deprecation cycle.
-
-  Code from other modules uses classes rather than modules, so member variables
-  are exposed. Many classes mark private members using the `_underscorePrefix`
-  pattern. Particular care should be taken if access is needed to a private
-  member.
-
-
-## Creating Custom Builds
-
-GCLI uses [DryIce](https://github.com/mozilla/dryice) to create custom builds.
-If dryice is installed (``npm install .``) then you can create a built
-version of GCLI simply using ``node gcli.js standard``. DryIce supplies a custom
-module loader to replace RequireJS for built applications.
-
-The build will be output to the ``built`` directory. The directory will be
-created if it doesn't exist.
deleted file mode 100644
--- a/devtools/shared/gcli/source/docs/running-tests.md
+++ /dev/null
@@ -1,60 +0,0 @@
-
-# Running Tests
-
-GCLI has a test suite that can be run in a number of different environments.
-Some of the tests don't work in all environments. These should be automatically
-skipped when not applicable.
-
-
-## Web
-
-Running a limited set of test from the web is the easiest. Simply load
-'localtest.html' and the unit tests should be run automatically, with results
-displayed on the console. Tests can be re-run using the 'test' command.
-
-It also creates a function 'testCommands()' to be run at a JS prompt, which
-enables the test commands for debugging purposes.
-
-
-## Firefox
-
-GCLI's test suite integrates with Mochitest and runs automatically on each test
-run. Dryice packages the tests to format them for the Firefox build system.
-
-For more information about running Mochitest on Firefox (including GCLI) see
-[the MDN, Mochitest docs](https://developer.mozilla.org/en/Mochitest)
-
-
-# Node
-
-Running the test suite under node can be done as follows:
-
-    $ node gcli.js test
-
-Or, using the `test` command:
-
-    $ node gcli.js
-    Serving GCLI to http://localhost:9999/
-    This is also a limited GCLI prompt.
-    Type 'help' for a list of commands, CTRL+C twice to exit:
-    : test
-    
-    testCli: Pass (funcs=9, checks=208)
-    testCompletion: Pass (funcs=1, checks=139)
-    testExec: Pass (funcs=1, checks=133)
-    testHistory: Pass (funcs=3, checks=13)
-    ....
-    
-    Summary: Pass (951 checks)
-
-
-# Travis CI
-
-GCLI check-ins are automatically tested by [Travis CI](https://travis-ci.org/joewalker/gcli).
-
-
-# Test Case Generation
-
-GCLI can generate test cases automagically. Load ```localtest.html```, type a
-command to be tested into GCLI, and the press F2. GCLI will output to the
-console a template test case for the entered command.
deleted file mode 100644
--- a/devtools/shared/gcli/source/docs/writing-commands.md
+++ /dev/null
@@ -1,757 +0,0 @@
-
-# Writing Commands
-
-## Basics
-
-GCLI has opinions about how commands should be written, and it encourages you
-to do The Right Thing. The opinions are based on helping users convert their
-intentions to commands and commands to what's actually going to happen.
-
-- Related commands should be sub-commands of a parent command. One of the goals
-  of GCLI is to support a large number of commands without things becoming
-  confusing, this will require some sort of namespacing or there will be
-  many people wanting to implement the ``add`` command. This style of
-  writing commands has become common place in Unix as the number of commands
-  has gone up.
-  The ```context``` command allows users to focus on a parent command, promoting
-  its sub-commands above others.
-
-- Each command should do exactly and only one thing. An example of a Unix
-  command that breaks this principle is the ``tar`` command
-
-        $ tar -zcf foo.tar.gz .
-        $ tar -zxf foo.tar.gz .
-
-  These 2 commands do exactly opposite things. Many a file has died as a result
-  of a x/c typo. In GCLI this would be better expressed:
-
-        $ tar create foo.tar.gz -z .
-        $ tar extract foo.tar.gz -z .
-
-  There may be commands (like tar) which have enough history behind them
-  that we shouldn't force everyone to re-learn a new syntax. The can be achieved
-  by having a single string parameter and parsing the input in the command)
-
-- Avoid errors. We try to avoid the user having to start again with a command
-  due to some problem. The majority of problems are simple typos which we can
-  catch using command metadata, but there are 2 things command authors can do
-  to prevent breakage.
-
-  - Where possible avoid the need to validate command line parameters in the
-    exec function. This can be done by good parameter design (see 'do exactly
-    and only one thing' above)
-
-  - If there is an obvious fix for an unpredictable problem, offer the
-    solution in the command output. So rather than use request.error (see
-    Request Object below) output some HTML which contains a link to a fixed
-    command line.
-
-Currently these concepts are not enforced at a code level, but they could be in
-the future.
-
-
-## How commands work
-
-This is how to create a basic ``greet`` command:
-
-    gcli.addItems([{
-      name: 'greet',
-      description: 'Show a greeting',
-      params: [
-        {
-          name: 'name',
-          type: 'string',
-          description: 'The name to greet'
-        }
-      ],
-      returnType: 'string',
-      exec: function(args, context) {
-        return 'Hello, ' + args.name;
-      }
-    }]);
-
-This command is used as follows:
-
-    : greet Joe
-    Hello, Joe
-
-Some terminology that isn't always obvious: a function has 'parameters', and
-when you call a function, you pass 'arguments' to it.
-
-
-## Internationalization (i18n)
-
-There are several ways that GCLI commands can be localized. The best method
-depends on what context you are writing your command for.
-
-### Firefox Embedding
-
-GCLI supports Mozilla style localization. To add a command that will only ever
-be used embedded in Firefox, this is the way to go. Your strings should be
-stored in ``toolkit/locales/en-US/chrome/global/devtools/gclicommands.properties``,
-And you should access them using ``let l10n = require("gcli/l10n")`` and then
-``l10n.lookup(...)`` or ``l10n.lookupFormat()``
-
-For examples of existing commands, take a look in
-``devtools/client/webconsole/GcliCommands.jsm``, which contains most of the
-current GCLI commands. If you will be adding a number of new commands, then
-consider starting a new JSM.
-
-Your command will then look something like this:
-
-    gcli.addItems([{
-      name: 'greet',
-      description: gcli.lookup("greetDesc")
-      ...
-    }]);
-
-### Web Commands
-
-There are 2 ways to provide translated strings for web use. The first is to
-supply the translated strings in the description:
-
-    gcli.addItems([{
-      name: 'greet',
-      description: {
-        'root': 'Show a greeting',
-        'fr-fr': 'Afficher un message d'accueil',
-        'de-de': 'Zeige einen Gruß',
-        'gk-gk': 'Εμφάνιση ένα χαιρετισμό',
-        ...
-      }
-      ...
-    }]);
-
-Each description should contain at least a 'root' entry which is the
-default if no better match is found. This method has the benefit of being
-compact and simple, however it has the significant drawback of being wasteful
-of memory and bandwidth to transmit and store a significant number of strings,
-the majority of which will never be used.
-
-More efficient is to supply a lookup key and ask GCLI to lookup the key from an
-appropriate localized strings file:
-
-    gcli.addItems([{
-      name: 'greet',
-      description: { 'key': 'demoGreetingDesc' }
-      ...
-    }]);
-
-For web usage, the central store of localized strings is
-``lib/gcli/nls/strings.js``. Other string files can be added using the
-``l10n.registerStringsSource(...)`` function.
-
-This method can be used both in Firefox and on the Web (see the help command
-for an example). However this method has the drawback that it will not work
-with DryIce built files until we fix bug 683844.
-
-
-## Default argument values
-
-The ``greet`` command requires the entry of the ``name`` parameter. This
-parameter can be made optional with the addition of a ``defaultValue`` to the
-parameter:
-
-    gcli.addItems([{
-      name: 'greet',
-      description: 'Show a message to someone',
-      params: [
-        {
-          name: 'name',
-          type: 'string',
-          description: 'The name to greet',
-          defaultValue: 'World!'
-        }
-      ],
-      returnType: 'string',
-      exec: function(args, context) {
-        return "Hello, " + args.name;
-      }
-    }]);
-
-Now we can also use the ``greet`` command as follows:
-
-    : greet
-    Hello, World!
-
-
-## Positional vs. named arguments
-
-Arguments can be entered either positionally or as named arguments. Generally
-users will prefer to type the positional version, however the named alternative
-can be more self documenting.
-
-For example, we can also invoke the greet command as follows:
-
-    : greet --name Joe
-    Hello, Joe
-
-
-## Short argument names
-
-GCLI allows you to specify a 'short' character for any parameter:
-
-    gcli.addItems([{
-      name: 'greet',
-      params: [
-        {
-          name: 'name',
-          short: 'n',
-          type: 'string',
-          ...
-        }
-      ],
-      ...
-    }]);
-
-This is used as follows:
-
-    : greet -n Fred
-    Hello, Fred
-
-Currently GCLI does not allow short parameter merging (i.e. ```ls -la```)
-however this is planned.
-
-
-## Parameter types
-
-Initially the available types are:
-
-- string
-- boolean
-- number
-- selection
-- delegate
-- date
-- array
-- file
-- node
-- nodelist
-- resource
-- command
-- setting
-
-This list can be extended. See [Writing Types](writing-types.md) on types for
-more information.
-
-The following examples assume the following definition of the ```greet```
-command:
-
-    gcli.addItems([{
-      name: 'greet',
-      params: [
-        { name: 'name', type: 'string' },
-        { name: 'repeat', type: 'number' }
-      ],
-      ...
-    }]);
-
-Parameters can be specified either with named arguments:
-
-    : greet --name Joe --repeat 2
-
-And sometimes positionally:
-
-    : greet Joe 2
-
-Parameters can be specified positionally if they are considered 'important'.
-Unimportant parameters must be specified with a named argument.
-
-Named arguments can be specified anywhere on the command line (after the
-command itself) however positional arguments must be in order. So
-these examples are the same:
-
-    : greet --name Joe --repeat 2
-    : greet --repeat 2 --name Joe
-
-However (obviously) these are not the same:
-
-    : greet Joe 2
-    : greet 2 Joe
-
-(The second would be an error because 'Joe' is not a number).
-
-Named arguments are assigned first, then the remaining arguments are assigned
-to the remaining parameters. So the following is valid and unambiguous:
-
-    : greet 2 --name Joe
-
-Positional parameters quickly become unwieldy with long parameter lists so we
-recommend only having 2 or 3 important parameters. GCLI provides hints for
-important parameters more obviously than unimportant ones.
-
-Parameters are 'important' if they are not in a parameter group. The easiest way
-to achieve this is to use the ```option: true``` property.
-
-For example, using:
-
-    gcli.addItems([{
-      name: 'greet',
-      params: [
-        { name: 'name', type: 'string' },
-        { name: 'repeat', type: 'number', option: true, defaultValue: 1 }
-      ],
-      ...
-    }]);
-
-Would mean that this is an error
-
-    : greet Joe 2
-
-You would instead need to do the following:
-
-    : greet Joe --repeat 2
-
-For more on parameter groups, see below.
-
-In addition to being 'important' and 'unimportant' parameters can also be
-optional. If is possible to be important and optional, but it is not possible
-to be unimportant and non-optional.
-
-Parameters are optional if they either:
-- Have a ```defaultValue``` property
-- Are of ```type=boolean``` (boolean arguments automatically default to being false)
-
-There is currently no way to make parameters mutually exclusive.
-
-
-## Selection types
-
-Parameters can have a type of ``selection``. For example:
-
-    gcli.addItems([{
-      name: 'greet',
-      params: [
-        { name: 'name', ... },
-        {
-          name: 'lang',
-          description: 'In which language should we greet',
-          type: { name: 'selection', data: [ 'en', 'fr', 'de', 'es', 'gk' ] },
-          defaultValue: 'en'
-        }
-      ],
-      ...
-    }]);
-
-GCLI will enforce that the value of ``arg.lang`` was one of the values
-specified. Alternatively ``data`` can be a function which returns an array of
-strings.
-
-The ``data`` property is useful when the underlying type is a string but it
-doesn't work when the underlying type is something else. For this use the
-``lookup`` property as follows:
-
-      type: {
-        name: 'selection',
-        lookup: {
-          'en': Locale.EN,
-          'fr': Locale.FR,
-          ...
-        }
-      },
-
-Similarly, ``lookup`` can be a function returning the data of this type.
-
-
-## Number types
-
-Number types are mostly self explanatory, they have one special property which
-is the ability to specify upper and lower bounds for the number:
-
-    gcli.addItems([{
-      name: 'volume',
-      params: [
-        {
-          name: 'vol',
-          description: 'How loud should we go',
-          type: { name: 'number', min: 0, max: 11 }
-        }
-      ],
-      ...
-    }]);
-
-You can also specify a ``step`` property which specifies by what amount we
-should increment and decrement the values. The ``min``, ``max``, and ``step``
-properties are used by the command line when up and down are pressed and in
-the input type of a dialog generated from this command.
-
-
-## Delegate types
-
-Delegate types are needed when the type of some parameter depends on the type
-of another parameter. For example:
-
-    : set height 100
-    : set name "Joe Walker"
-
-We can achieve this as follows:
-
-    gcli.addItems([{
-      name: 'set',
-      params: [
-        {
-          name: 'setting',
-          type: { name: 'selection', values: [ 'height', 'name' ] }
-        },
-        {
-          name: 'value',
-          type: {
-            name: 'delegate',
-            delegateType: function() { ... }
-          }
-        }
-      ],
-      ...
-    }]);
-
-Several details are left out of this example, like how the delegateType()
-function knows what the current setting is. See the ``pref`` command for an
-example.
-
-
-## Array types
-
-Parameters can have a type of ``array``. For example:
-
-    gcli.addItems([{
-      name: 'greet',
-      params: [
-        {
-          name: 'names',
-          type: { name: 'array', subtype: 'string' },
-          description: 'The names to greet',
-          defaultValue: [ 'World!' ]
-        }
-      ],
-      ...
-      exec: function(args, context) {
-        return "Hello, " + args.names.join(', ') + '.';
-      }
-    }]);
-
-This would be used as follows:
-
-    : greet Fred Jim Shiela
-    Hello, Fred, Jim, Shiela.
-
-Or using named arguments:
-
-    : greet --names Fred --names Jim --names Shiela
-    Hello, Fred, Jim, Shiela.
-
-There can only be one ungrouped parameter with an array type, and it must be
-at the end of the list of parameters (i.e. just before any parameter groups).
-This avoids confusion as to which parameter an argument should be assigned.
-
-
-## Sub-commands
-
-It is common for commands to be groups into those with similar functionality.
-Examples include virtually all VCS commands, ``apt-get``, etc. There are many
-examples of commands that should be structured as in a sub-command style -
-``tar`` being the obvious example, but others include ``crontab``.
-
-Groups of commands are specified with the top level command not having an
-exec function:
-
-    gcli.addItems([
-      {
-        name: 'tar',
-        description: 'Commands to manipulate archives',
-      },
-      {
-        name: 'tar create',
-        description: 'Create a new archive',
-        exec: function(args, context) { ... },
-        ...
-      },
-      {
-        name: 'tar extract',
-        description: 'Extract from an archive',
-        exec: function(args, context) { ... },
-        ...
-      }
-    ]);
-
-
-## Parameter groups
-
-Parameters can be grouped into sections.
-
-There are 3 ways to assign a parameter to a group.
-
-The simplest uses ```option: true``` to put a parameter into the default
-'Options' group:
-
-    gcli.addItems([{
-      name: 'greet',
-      params: [
-        { name: 'repeat', type: 'number', option: true }
-      ],
-      ...
-    }]);
-
-The ```option``` property can also take a string to use an alternative parameter
-group:
-
-    gcli.addItems([{
-      name: 'greet',
-      params: [
-        { name: 'repeat', type: 'number', option: 'Advanced' }
-      ],
-      ...
-    }]);
-
-An example of how this can be useful is 'git' which categorizes parameters into
-'porcelain' and 'plumbing'.
-
-Finally, parameters can be grouped together as follows:
-
-    gcli.addItems([{
-      name: 'greet',
-      params: [
-        { name: 'name', type: 'string', description: 'The name to greet' },
-        {
-          group: 'Advanced Options',
-          params: [
-            { name: 'repeat', type: 'number', defaultValue: 1 },
-            { name: 'debug', type: 'boolean' }
-          ]
-        }
-      ],
-      ...
-    }]);
-
-This could be used as follows:
-
-    : greet Joe --repeat 2 --debug
-    About to send greeting
-    Hello, Joe
-    Hello, Joe
-    Done!
-
-Parameter groups must come after non-grouped parameters because non-grouped
-parameters can be assigned positionally, so their index is important. We don't
-want 'holes' in the order caused by parameter groups.
-
-
-## Command metadata
-
-Each command should have the following properties:
-
-- A string ``name``.
-- A short ``description`` string. Generally no more than 20 characters without
-  a terminating period/fullstop.
-- A function to ``exec``ute. (Optional for the parent containing sub-commands)
-  See below for more details.
-
-And optionally the following extra properties:
-
-- A declaration of the accepted ``params``.
-- A ``hidden`` property to stop the command showing up in requests for help.
-- A ``context`` property which defines the scope of the function that we're
-  calling. Rather than simply call ``exec()``, we do ``exec.call(context)``.
-- A ``manual`` property which allows a fuller description of the purpose of the
-  command.
-- A ``returnType`` specifying how we should handle the value returned from the
-  exec function.
-
-The ``params`` property is an array of objects, one for each parameter. Each
-parameter object should have the following 3 properties:
-
-- A string ``name``.
-- A short string ``description`` as for the command.
-- A ``type`` which refers to an existing Type (see Writing Types).
-
-Optionally each parameter can have these properties:
-
-- A ``defaultValue`` (which should be in the type specified in ``type``).
-  The defaultValue will be used when there is no argument supplied for this
-  parameter on the command line.
-  If the parameter has a ``defaultValue``, other than ``undefined`` then the
-  parameter is optional, and if unspecified on the command line, the matching
-  argument will have this value when the function is called.
-  If ``defaultValue`` is missing, or if it is set to ``undefined``, then the
-  system will ensure that a value is provided before anything is executed.
-  There are 2 special cases:
-  - If the type is ``selection``, then defaultValue must not be undefined.
-    The defaultValue must either be ``null`` (meaning that a value must be
-    supplied by the user) or one of the selection values.
-  - If the type is ``boolean``, then ``defaultValue:false`` is implied and
-    can't be changed. Boolean toggles are assumed to be off by default, and
-    should be named to match.
-- A ``manual`` property for parameters is exactly analogous to the ``manual``
-  property for commands - descriptive text that is longer than than 20
-  characters.
-
-
-## The Command Function (exec)
-
-The parameters to the exec function are designed to be useful when you have a
-large number of parameters, and to give direct access to the environment (if
-used).
-
-    gcli.addItems([{
-      name: 'echo',
-      description: 'The message to display.',
-      params: [
-        {
-          name: 'message',
-          type: 'string',
-          description: 'The message to display.'
-        }
-      ],
-      returnType: 'string',
-      exec: function(args, context) {
-        return args.message;
-      }
-    }]);
-
-The ``args`` object contains the values specified on the params section and
-provided on the command line. In this example it would contain the message for
-display as ``args.message``.
-
-The ``context`` object has the following signature:
-
-    {
-      environment: ..., // environment object passed to createTerminal()
-      exec: ...,        // function to execute a command
-      update: ...,      // function to alter the text of the input area
-      createView: ...,  // function to help creating rich output
-      defer: ...,       // function to create a deferred promise
-    }
-
-The ``environment`` object is opaque to GCLI. It can be used for providing
-arbitrary data to your commands about their environment. It is most useful
-when more than one command line exists on a page with similar commands in both
-which should act in their own ways.
-An example use for ``environment`` would be a page with several tabs, each
-containing an editor with a command line. Commands executed in those editors
-should apply to the relevant editor.
-The ``environment`` object is passed to GCLI at startup (probably in the
-``createTerminal()`` function).
-
-The ``document`` object is also passed to GCLI at startup. In some environments
-(e.g. embedded in Firefox) there is no global ``document``. This object
-provides a way to create DOM nodes.
-
-``defer()`` allows commands to execute asynchronously.
-
-
-## Returning data
-
-The command meta-data specifies the type of data returned by the command using
-the ``returnValue`` setting.
-
-``returnValue`` processing is currently functioning, but incomplete, and being
-tracked in [Bug 657595](http://bugzil.la/657595). Currently you should specify
-a ``returnType`` of ``string`` or ``html``. If using HTML, you can return
-either an HTML string or a DOM node.
-
-In the future, JSON will be strongly encouraged as the return type, with some
-formatting functions to convert the JSON to HTML.
-
-Asynchronous output is achieved using a promise created from the ``context``
-parameter: ``context.defer()``.
-
-Some examples of this is practice:
-
-    { returnType: "string" }
-    ...
-    return "example";
-
-GCLI interprets the output as a plain string. It will be escaped before display
-and available as input to other commands as a plain string.
-
-    { returnType: "html" }
-    ...
-    return "<p>Hello</p>";
-
-GCLI will interpret this as HTML, and parse it for display.
-
-    { returnType: "dom" }
-    ...
-    return util.createElement(context.document, 'div');
-
-``util.createElement`` is a utility to ensure use of the XHTML namespace in XUL
-and other XML documents. In an HTML document it's functionally equivalent to
-``context.document.createElement('div')``. If your command is likely to be used
-in Firefox or another XML environment, you should use it. You can import it
-with ``var util = require('util/util');``.
-
-GCLI will use the returned HTML element as returned. See notes on ``context``
-above.
-
-    { returnType: "number" }
-    ...
-    return 42;
-
-GCLI will display the element in a similar way to a string, but it the value
-will be available to future commands as a number.
-
-    { returnType: "date" }
-    ...
-    return new Date();
-
-    { returnType: "file" }
-    ...
-    return new File();
-
-Both these examples return data as a given type, for which a converter will
-be required before the value can be displayed. The type system is likely to
-change before this is finalized. Please contact the author for more
-information.
-
-    { returnType: "string" }
-    ...
-    var deferred = context.defer();
-    setTimeout(function() {
-      deferred.resolve("hello");
-    }, 500);
-    return deferred.promise;
-
-Errors can be signaled by throwing an exception. GCLI will display the message
-property (or the toString() value if there is no message property). (However
-see *3 principles for writing commands* above for ways to avoid doing this).
-
-
-## Specifying Types
-
-Types are generally specified by a simple string, e.g. ``'string'``. For most
-types this is enough detail. There are a number of exceptions:
-
-* Array types. We declare a parameter to be an array of things using ``[]``,
-  for example: ``number[]``.
-* Selection types. There are 3 ways to specify the options in a selection:
-  * Using a lookup map
-
-            type: {
-              name: 'selection',
-              lookup: { one:1, two:2, three:3 }
-            }
-
-    (The boolean type is effectively just a selection that uses
-    ``lookup:{ 'true': true, 'false': false }``)
-
-  * Using given strings
-
-            type: {
-              name: 'selection',
-              data: [ 'left', 'center', 'right' ]
-            }
-
-  * Using named objects, (objects with a ``name`` property)
-
-            type: {
-              name: 'selection',
-              data: [
-                { name: 'Google', url: 'http://www.google.com/' },
-                { name: 'Microsoft', url: 'http://www.microsoft.com/' },
-                { name: 'Yahoo', url: 'http://www.yahoo.com/' }
-              ]
-            }
-
-* Delegate type. It is generally best to inherit from Delegate in order to
-  provide a customization of this type. See settingValue for an example.
-
-See below for more information.
deleted file mode 100644
--- a/devtools/shared/gcli/source/docs/writing-tests.md
+++ /dev/null
@@ -1,20 +0,0 @@
-
-# Writing Tests
-
-There are several sources of GCLI tests and several environments in which they
-are run.
-
-The majority of GCLI tests are stored in
-[this repository](https://github.com/joewalker/gcli/) in files named like
-```./lib/gclitest/test*.js```. These tests run in Firefox, Chrome, Opera,
-and NodeJS/JsDom
-
-See [Running Tests](running-tests.md) for further details.
-
-GCLI comes with a generic unit test harness (in ```./lib/test/```) and a
-set of helpers for creating GCLI tests (in ```./lib/gclitest/helpers.js```).
-
-# GCLI tests in Firefox
-
-The build process converts the GCLI tests to run under Mochitest inside the
-Firefox unit tests. It also adds some
deleted file mode 100644
--- a/devtools/shared/gcli/source/docs/writing-types.md
+++ /dev/null
@@ -1,106 +0,0 @@
-
-# Writing Types
-
-Commands are a fundamental building block because they are what the users
-directly interacts with, however they are built on ``Type``s. There are a
-number of built in types:
-
-* string. This is a JavaScript string
-* number. A JavaScript number
-* boolean. A JavaScript boolean
-* selection. This is an selection from a number of alternatives
-* delegate. This type could change depending on other factors, but is well
-  defined when one of the conversion routines is called.
-
-There are a number of additional types defined by Pilot and GCLI as
-extensions to the ``selection`` and ``delegate`` types
-
-* setting. One of the defined settings
-* settingValue. A value that can be applied to an associated setting.
-* command. One of the defined commands
-
-Most of our types are 'static' e.g. there is only one type of 'string', however
-some types like 'selection' and 'delegate' are customizable.
-
-All types must inherit from Type and have the following methods:
-
-    /**
-     * Convert the given <tt>value</tt> to a string representation.
-     * Where possible, there should be round-tripping between values and their
-     * string representations.
-     */
-    stringify: function(value) { return 'string version of value'; },
-
-    /**
-     * Convert the given <tt>str</tt> to an instance of this type.
-     * Where possible, there should be round-tripping between values and their
-     * string representations.
-     * @return Conversion
-     */
-    parse: function(str) { return new Conversion(...); },
-
-    /**
-     * The plug-in system, and other things need to know what this type is
-     * called. The name alone is not enough to fully specify a type. Types like
-     * 'selection' and 'delegate' need extra data, however this function returns
-     * only the name, not the extra data.
-     * <p>In old bespin, equality was based on the name. This may turn out to be
-     * important in Ace too.
-     */
-    name: 'example',
-
-In addition, defining the following function can be helpful, although Type
-contains default implementations:
-
-* nudge(value, by)
-
-Type, Conversion and Status are all declared by commands.js.
-
-The values produced by the parse function can be of any type, but if you are
-producing your own, you are strongly encouraged to include properties called
-``name`` and ``description`` where it makes sense. There are a number of
-places in GCLI where the UI will be able to provide better help to users if
-your values include these properties.
-
-
-# Writing Fields
-
-Fields are visual representations of types. For simple types like string it is
-enough to use ``<input type=...>``, however more complex types we may wish to
-provide a custom widget to allow the user to enter values of the given type.
-
-This is an example of a very simple new password field type:
-
-    function PasswordField(doc) {
-      this.doc = doc;
-    }
-
-    PasswordField.prototype = Object.create(Field.prototype);
-
-    PasswordField.prototype.createElement = function(assignment) {
-      this.assignment = assignment;
-      this.input = dom.createElement(this.doc, 'input');
-      this.input.type = 'password';
-      this.input.value = assignment.arg ? assignment.arg.text : '';
-
-      this.onKeyup = function() {
-          this.assignment.setValue(this.input.value);
-      }.bind(this);
-      this.input.addEventListener('keyup', this.onKeyup, false);
-
-      this.onChange = function() {
-          this.input.value = this.assignment.arg.text;
-      };
-      this.assignment.onAssignmentChange.add(this.onChange, this);
-
-      return this.input;
-    };
-
-    PasswordField.prototype.destroy = function() {
-      this.input.removeEventListener('keyup', this.onKeyup, false);
-      this.assignment.onAssignmentChange.remove(this.onChange, this);
-    };
-
-    PasswordField.claim = function(type) {
-      return type.name === 'password' ? Field.claim.MATCH : Field.claim.NO_MATCH;
-    };
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/cli.js
+++ /dev/null
@@ -1,2209 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var util = require('./util/util');
-var host = require('./util/host');
-var l10n = require('./util/l10n');
-
-var view = require('./ui/view');
-var Parameter = require('./commands/commands').Parameter;
-var CommandOutputManager = require('./commands/commands').CommandOutputManager;
-
-var Status = require('./types/types').Status;
-var Conversion = require('./types/types').Conversion;
-var commandModule = require('./types/command');
-var selectionModule = require('./types/selection');
-
-var Argument = require('./types/types').Argument;
-var ArrayArgument = require('./types/types').ArrayArgument;
-var NamedArgument = require('./types/types').NamedArgument;
-var TrueNamedArgument = require('./types/types').TrueNamedArgument;
-var MergedArgument = require('./types/types').MergedArgument;
-var ScriptArgument = require('./types/types').ScriptArgument;
-
-var RESOLVED = Promise.resolve(undefined);
-
-// Helper to produce a `deferred` object
-// using DOM Promise
-function defer() {
-  let resolve, reject;
-  let p = new Promise((a, b) => {
-    resolve = a;
-    reject = b;
-  });
-  return {
-    promise: p,
-    resolve: resolve,
-    reject: reject
-  };
-}
-
-/**
- * This is a list of the known command line components to enable certain
- * privileged commands to alter parts of a running command line. It is an array
- * of objects shaped like:
- *   { conversionContext:..., executionContext:..., mapping:... }
- * So lookup is O(n) where 'n' is the number of command lines.
- */
-var instances = [];
-
-/**
- * An indexOf that looks-up both types of context
- */
-function instanceIndex(context) {
-  for (var i = 0; i < instances.length; i++) {
-    var instance = instances[i];
-    if (instance.conversionContext === context ||
-        instance.executionContext === context) {
-      return i;
-    }
-  }
-  return -1;
-}
-
-/**
- * findInstance gets access to a Terminal object given a conversionContext or
- * an executionContext (it doesn't have to be a terminal object, just whatever
- * was passed into addMapping()
- */
-exports.getMapping = function(context) {
-  var index = instanceIndex(context);
-  if (index === -1) {
-    console.log('Missing mapping for context: ', context);
-    console.log('Known contexts: ', instances);
-    throw new Error('Missing mapping for context');
-  }
-  return instances[index].mapping;
-};
-
-/**
- * Add a requisition context->terminal mapping
- */
-var addMapping = function(requisition) {
-  if (instanceIndex(requisition.conversionContext) !== -1) {
-    throw new Error('Remote existing mapping before adding a new one');
-  }
-
-  instances.push({
-    conversionContext: requisition.conversionContext,
-    executionContext: requisition.executionContext,
-    mapping: { requisition: requisition }
-  });
-};
-
-/**
- * Remove a requisition context->terminal mapping
- */
-var removeMapping = function(requisition) {
-  var index = instanceIndex(requisition.conversionContext);
-  instances.splice(index, 1);
-};
-
-/**
- * Assignment is a link between a parameter and the data for that parameter.
- * The data for the parameter is available as in the preferred type and as
- * an Argument for the CLI.
- * <p>We also record validity information where applicable.
- * <p>For values, null and undefined have distinct definitions. null means
- * that a value has been provided, undefined means that it has not.
- * Thus, null is a valid default value, and common because it identifies an
- * parameter that is optional. undefined means there is no value from
- * the command line.
- * @constructor
- */
-function Assignment(param) {
-  // The parameter that we are assigning to
-  this.param = param;
-  this.conversion = undefined;
-}
-
-/**
- * Easy accessor for conversion.arg.
- * This is a read-only property because writes to arg should be done through
- * the 'conversion' property.
- */
-Object.defineProperty(Assignment.prototype, 'arg', {
-  get: function() {
-    return this.conversion == null ? undefined : this.conversion.arg;
-  },
-  enumerable: true
-});
-
-/**
- * Easy accessor for conversion.value.
- * This is a read-only property because writes to value should be done through
- * the 'conversion' property.
- */
-Object.defineProperty(Assignment.prototype, 'value', {
-  get: function() {
-    return this.conversion == null ? undefined : this.conversion.value;
-  },
-  enumerable: true
-});
-
-/**
- * Easy (and safe) accessor for conversion.message
- */
-Object.defineProperty(Assignment.prototype, 'message', {
-  get: function() {
-    if (this.conversion != null && this.conversion.message) {
-      return this.conversion.message;
-    }
-    // ERROR conversions have messages, VALID conversions don't need one, so
-    // we just need to consider INCOMPLETE conversions.
-    if (this.getStatus() === Status.INCOMPLETE) {
-      return l10n.lookupFormat('cliIncompleteParam', [ this.param.name ]);
-    }
-    return '';
-  },
-  enumerable: true
-});
-
-/**
- * Easy (and safe) accessor for conversion.getPredictions()
- * @return An array of objects with name and value elements. For example:
- * [ { name:'bestmatch', value:foo1 }, { name:'next', value:foo2 }, ... ]
- */
-Assignment.prototype.getPredictions = function(context) {
-  return this.conversion == null ? [] : this.conversion.getPredictions(context);
-};
-
-/**
- * Accessor for a prediction by index.
- * This is useful above <tt>getPredictions()[index]</tt> because it normalizes
- * index to be within the bounds of the predictions, which means that the UI
- * can maintain an index of which prediction to choose without caring how many
- * predictions there are.
- * @param rank The index of the prediction to choose
- */
-Assignment.prototype.getPredictionRanked = function(context, rank) {
-  if (rank == null) {
-    rank = 0;
-  }
-
-  if (this.isInName()) {
-    return Promise.resolve(undefined);
-  }
-
-  return this.getPredictions(context).then(predictions => {
-    if (predictions.length === 0) {
-      return undefined;
-    }
-
-    rank = rank % predictions.length;
-    if (rank < 0) {
-      rank = predictions.length + rank;
-    }
-    return predictions[rank];
-  });
-};
-
-/**
- * Some places want to take special action if we are in the name part of a
- * named argument (i.e. the '--foo' bit).
- * Currently this does not take actual cursor position into account, it just
- * assumes that the cursor is at the end. In the future we will probably want
- * to take this into account.
- */
-Assignment.prototype.isInName = function() {
-  return this.conversion.arg.type === 'NamedArgument' &&
-         this.conversion.arg.prefix.slice(-1) !== ' ';
-};
-
-/**
- * Work out what the status of the current conversion is which involves looking
- * not only at the conversion, but also checking if data has been provided
- * where it should.
- * @param arg For assignments with multiple args (e.g. array assignments) we
- * can narrow the search for status to a single argument.
- */
-Assignment.prototype.getStatus = function(arg) {
-  if (this.param.isDataRequired && !this.conversion.isDataProvided()) {
-    return Status.INCOMPLETE;
-  }
-
-  // Selection/Boolean types with a defined range of values will say that
-  // '' is INCOMPLETE, but the parameter may be optional, so we don't ask
-  // if the user doesn't need to enter something and hasn't done so.
-  if (!this.param.isDataRequired && this.arg.type === 'BlankArgument') {
-    return Status.VALID;
-  }
-
-  return this.conversion.getStatus(arg);
-};
-
-/**
- * Helper when we're rebuilding command lines.
- */
-Assignment.prototype.toString = function() {
-  return this.conversion.toString();
-};
-
-/**
- * For test/debug use only. The output from this function is subject to wanton
- * random change without notice, and should not be relied upon to even exist
- * at some later date.
- */
-Object.defineProperty(Assignment.prototype, '_summaryJson', {
-  get: function() {
-    return {
-      param: this.param.name + '/' + this.param.type.name,
-      defaultValue: this.param.defaultValue,
-      arg: this.conversion.arg._summaryJson,
-      value: this.value,
-      message: this.message,
-      status: this.getStatus().toString()
-    };
-  },
-  enumerable: true
-});
-
-exports.Assignment = Assignment;
-
-
-/**
- * How to dynamically execute JavaScript code
- */
-var customEval = eval;
-
-/**
- * Setup a function to be called in place of 'eval', generally for security
- * reasons
- */
-exports.setEvalFunction = function(newCustomEval) {
-  customEval = newCustomEval;
-};
-
-/**
- * Remove the binding done by setEvalFunction().
- * We purposely set customEval to undefined rather than to 'eval' because there
- * is an implication of setEvalFunction that we're in a security sensitive
- * situation. What if we can trick GCLI into calling unsetEvalFunction() at the
- * wrong time?
- * So to properly undo the effects of setEvalFunction(), you need to call
- * setEvalFunction(eval) rather than unsetEvalFunction(), however the latter is
- * preferred in most cases.
- */
-exports.unsetEvalFunction = function() {
-  customEval = undefined;
-};
-
-/**
- * 'eval' command
- */
-var evalCmd = {
-  item: 'command',
-  name: '{',
-  params: [
-    {
-      name: 'javascript',
-      type: 'javascript',
-      description: ''
-    }
-  ],
-  hidden: true,
-  description: { key: 'cliEvalJavascript' },
-  exec: function(args, context) {
-    var reply = customEval(args.javascript);
-    return context.typedData(typeof reply, reply);
-  },
-  isCommandRegexp: /^\s*\{\s*/
-};
-
-exports.items = [ evalCmd ];
-
-/**
- * This is a special assignment to reflect the command itself.
- */
-function CommandAssignment(requisition) {
-  var commandParamMetadata = {
-    name: '__command',
-    type: { name: 'command', allowNonExec: false }
-  };
-  // This is a hack so that rather than reply with a generic description of the
-  // command assignment, we reply with the description of the assigned command,
-  // (using a generic term if there is no assigned command)
-  var self = this;
-  Object.defineProperty(commandParamMetadata, 'description', {
-    get: function() {
-      var value = self.value;
-      return value && value.description ?
-          value.description :
-          'The command to execute';
-    },
-    enumerable: true
-  });
-  this.param = new Parameter(requisition.system.types, commandParamMetadata);
-}
-
-CommandAssignment.prototype = Object.create(Assignment.prototype);
-
-CommandAssignment.prototype.getStatus = function(arg) {
-  return Status.combine(
-    Assignment.prototype.getStatus.call(this, arg),
-    this.conversion.value && this.conversion.value.exec ?
-            Status.VALID : Status.INCOMPLETE
-  );
-};
-
-exports.CommandAssignment = CommandAssignment;
-
-
-/**
- * Special assignment used when ignoring parameters that don't have a home
- */
-function UnassignedAssignment(requisition, arg) {
-  var isIncompleteName = (arg.text.charAt(0) === '-');
-  this.param = new Parameter(requisition.system.types, {
-    name: '__unassigned',
-    description: l10n.lookup('cliOptions'),
-    type: {
-      name: 'param',
-      requisition: requisition,
-      isIncompleteName: isIncompleteName
-    }
-  });
-
-  // It would be nice to do 'conversion = parm.type.parse(arg, ...)' except
-  // that type.parse returns a promise (even though it's synchronous in this
-  // case)
-  if (isIncompleteName) {
-    var lookup = commandModule.getDisplayedParamLookup(requisition);
-    var predictions = selectionModule.findPredictions(arg, lookup);
-    this.conversion = selectionModule.convertPredictions(arg, predictions);
-  }
-  else {
-    var message = l10n.lookup('cliUnusedArg');
-    this.conversion = new Conversion(undefined, arg, Status.ERROR, message);
-  }
-
-  this.conversion.assignment = this;
-}
-
-UnassignedAssignment.prototype = Object.create(Assignment.prototype);
-
-UnassignedAssignment.prototype.getStatus = function(arg) {
-  return this.conversion.getStatus();
-};
-
-var logErrors = true;
-
-/**
- * Allow tests that expect failures to avoid clogging up the console
- */
-Object.defineProperty(exports, 'logErrors', {
-  get: function() {
-    return logErrors;
-  },
-  set: function(val) {
-    logErrors = val;
-  },
-  enumerable: true
-});
-
-/**
- * A Requisition collects the information needed to execute a command.
- *
- * (For a definition of the term, see http://en.wikipedia.org/wiki/Requisition)
- * This term is used because carries the notion of a work-flow, or process to
- * getting the information to execute a command correct.
- * There is little point in a requisition for parameter-less commands because
- * there is no information to collect. A Requisition is a collection of
- * assignments of values to parameters, each handled by an instance of
- * Assignment.
- *
- * @param system Allows access to the various plug-in points in GCLI. At a
- * minimum it must contain commands and types objects.
- * @param options A set of options to customize how GCLI is used. Includes:
- * - environment An optional opaque object passed to commands in the
- *   Execution Context.
- * - document A DOM Document passed to commands using the Execution Context in
- *   order to allow creation of DOM nodes. If missing Requisition will use the
- *   global 'document', or leave undefined.
- * - commandOutputManager A custom commandOutputManager to which output should
- *   be sent
- * @constructor
- */
-function Requisition(system, options) {
-  options = options || {};
-
-  this.environment = options.environment || {};
-  this.document = options.document;
-  if (this.document == null) {
-    try {
-      this.document = document;
-    }
-    catch (ex) {
-      // Ignore
-    }
-  }
-
-  this.commandOutputManager = options.commandOutputManager || new CommandOutputManager();
-  this.system = system;
-
-  this.shell = {
-    cwd: '/', // Where we store the current working directory
-    env: {}   // Where we store the current environment
-  };
-
-  // The command that we are about to execute.
-  // @see setCommandConversion()
-  this.commandAssignment = new CommandAssignment(this);
-
-  // The object that stores of Assignment objects that we are filling out.
-  // The Assignment objects are stored under their param.name for named
-  // lookup. Note: We make use of the property of Javascript objects that
-  // they are not just hashmaps, but linked-list hashmaps which iterate in
-  // insertion order.
-  // _assignments excludes the commandAssignment.
-  this._assignments = {};
-
-  // The count of assignments. Excludes the commandAssignment
-  this.assignmentCount = 0;
-
-  // Used to store cli arguments in the order entered on the cli
-  this._args = [];
-
-  // Used to store cli arguments that were not assigned to parameters
-  this._unassigned = [];
-
-  // Changes can be asynchronous, when one update starts before another
-  // finishes we abandon the former change
-  this._nextUpdateId = 0;
-
-  // We can set a prefix to typed commands to make it easier to focus on
-  // Allowing us to type "add -a; commit" in place of "git add -a; git commit"
-  this.prefix = '';
-
-  addMapping(this);
-  this._setBlankAssignment(this.commandAssignment);
-
-  // If a command calls context.update then the UI needs some way to be
-  // informed of the change
-  this.onExternalUpdate = util.createEvent('Requisition.onExternalUpdate');
-}
-
-/**
- * Avoid memory leaks
- */
-Requisition.prototype.destroy = function() {
-  this.document = undefined;
-  this.environment = undefined;
-  removeMapping(this);
-};
-
-/**
- * If we're about to make an asynchronous change when other async changes could
- * overtake this one, then we want to be able to bail out if overtaken. The
- * value passed back from beginChange should be passed to endChangeCheckOrder
- * on completion of calculation, before the results are applied in order to
- * check that the calculation has not been overtaken
- */
-Requisition.prototype._beginChange = function() {
-  var updateId = this._nextUpdateId;
-  this._nextUpdateId++;
-  return updateId;
-};
-
-/**
- * Check to see if another change has started since updateId started.
- * This allows us to bail out of an update.
- * It's hard to make updates atomic because until you've responded to a parse
- * of the command argument, you don't know how to parse the arguments to that
- * command.
- */
-Requisition.prototype._isChangeCurrent = function(updateId) {
-  return updateId + 1 === this._nextUpdateId;
-};
-
-/**
- * See notes on beginChange
- */
-Requisition.prototype._endChangeCheckOrder = function(updateId) {
-  if (updateId + 1 !== this._nextUpdateId) {
-    // An update that started after we did has already finished, so our
-    // changes are out of date. Abandon further work.
-    return false;
-  }
-
-  return true;
-};
-
-var legacy = false;
-
-/**
- * Functions and data related to the execution of a command
- */
-Object.defineProperty(Requisition.prototype, 'executionContext', {
-  get: function() {
-    if (this._executionContext == null) {
-      this._executionContext = {
-        defer: defer,
-        typedData: function(type, data) {
-          return {
-            isTypedData: true,
-            data: data,
-            type: type
-          };
-        },
-        getArgsObject: this.getArgsObject.bind(this)
-      };
-
-      // Alias requisition so we're clear about what's what
-      var requisition = this;
-      Object.defineProperty(this._executionContext, 'prefix', {
-        get: function() { return requisition.prefix; },
-        enumerable: true
-      });
-      Object.defineProperty(this._executionContext, 'typed', {
-        get: function() { return requisition.toString(); },
-        enumerable: true
-      });
-      Object.defineProperty(this._executionContext, 'environment', {
-        get: function() { return requisition.environment; },
-        enumerable: true
-      });
-      Object.defineProperty(this._executionContext, 'shell', {
-        get: function() { return requisition.shell; },
-        enumerable: true
-      });
-      Object.defineProperty(this._executionContext, 'system', {
-        get: function() { return requisition.system; },
-        enumerable: true
-      });
-
-      this._executionContext.updateExec = this._contextUpdateExec.bind(this);
-
-      if (legacy) {
-        this._executionContext.createView = view.createView;
-        this._executionContext.exec = this.exec.bind(this);
-        this._executionContext.update = this._contextUpdate.bind(this);
-
-        Object.defineProperty(this._executionContext, 'document', {
-          get: function() { return requisition.document; },
-          enumerable: true
-        });
-      }
-    }
-
-    return this._executionContext;
-  },
-  enumerable: true
-});
-
-/**
- * Functions and data related to the conversion of the output of a command
- */
-Object.defineProperty(Requisition.prototype, 'conversionContext', {
-  get: function() {
-    if (this._conversionContext == null) {
-      this._conversionContext = {
-        defer: defer,
-
-        createView: view.createView,
-        exec: this.exec.bind(this),
-        update: this._contextUpdate.bind(this),
-        updateExec: this._contextUpdateExec.bind(this)
-      };
-
-      // Alias requisition so we're clear about what's what
-      var requisition = this;
-
-      Object.defineProperty(this._conversionContext, 'document', {
-        get: function() { return requisition.document; },
-        enumerable: true
-      });
-      Object.defineProperty(this._conversionContext, 'environment', {
-        get: function() { return requisition.environment; },
-        enumerable: true
-      });
-      Object.defineProperty(this._conversionContext, 'system', {
-        get: function() { return requisition.system; },
-        enumerable: true
-      });
-    }
-
-    return this._conversionContext;
-  },
-  enumerable: true
-});
-
-/**
- * Assignments have an order, so we need to store them in an array.
- * But we also need named access ...
- * @return The found assignment, or undefined, if no match was found
- */
-Requisition.prototype.getAssignment = function(nameOrNumber) {
-  var name = (typeof nameOrNumber === 'string') ?
-    nameOrNumber :
-    Object.keys(this._assignments)[nameOrNumber];
-  return this._assignments[name] || undefined;
-};
-
-/**
- * Where parameter name == assignment names - they are the same
- */
-Requisition.prototype.getParameterNames = function() {
-  return Object.keys(this._assignments);
-};
-
-/**
- * The overall status is the most severe status.
- * There is no such thing as an INCOMPLETE overall status because the
- * definition of INCOMPLETE takes into account the cursor position to say 'this
- * isn't quite ERROR because the user can fix it by typing', however overall,
- * this is still an error status.
- */
-Object.defineProperty(Requisition.prototype, 'status', {
-  get: function() {
-    var status = Status.VALID;
-    if (this._unassigned.length !== 0) {
-      var isAllIncomplete = true;
-      this._unassigned.forEach(function(assignment) {
-        if (!assignment.param.type.isIncompleteName) {
-          isAllIncomplete = false;
-        }
-      });
-      status = isAllIncomplete ? Status.INCOMPLETE : Status.ERROR;
-    }
-
-    this.getAssignments(true).forEach(function(assignment) {
-      var assignStatus = assignment.getStatus();
-      if (assignStatus > status) {
-        status = assignStatus;
-      }
-    }, this);
-    if (status === Status.INCOMPLETE) {
-      status = Status.ERROR;
-    }
-    return status;
-  },
-  enumerable : true
-});
-
-/**
- * If ``requisition.status != VALID`` message then return a string which
- * best describes what is wrong. Generally error messages are delivered by
- * looking at the error associated with the argument at the cursor, but there
- * are times when you just want to say 'tell me the worst'.
- * If ``requisition.status != VALID`` then return ``null``.
- */
-Requisition.prototype.getStatusMessage = function() {
-  if (this.commandAssignment.getStatus() !== Status.VALID) {
-    return l10n.lookupFormat('cliUnknownCommand2',
-                             [ this.commandAssignment.arg.text ]);
-  }
-
-  var assignments = this.getAssignments();
-  for (var i = 0; i < assignments.length; i++) {
-    if (assignments[i].getStatus() !== Status.VALID) {
-      return assignments[i].message;
-    }
-  }
-
-  if (this._unassigned.length !== 0) {
-    return l10n.lookup('cliUnusedArg');
-  }
-
-  return null;
-};
-
-/**
- * Extract the names and values of all the assignments, and return as
- * an object.
- */
-Requisition.prototype.getArgsObject = function() {
-  var args = {};
-  this.getAssignments().forEach(function(assignment) {
-    args[assignment.param.name] = assignment.conversion.isDataProvided() ?
-            assignment.value :
-            assignment.param.defaultValue;
-  }, this);
-  return args;
-};
-
-/**
- * Access the arguments as an array.
- * @param includeCommand By default only the parameter arguments are
- * returned unless (includeCommand === true), in which case the list is
- * prepended with commandAssignment.arg
- */
-Requisition.prototype.getAssignments = function(includeCommand) {
-  var assignments = [];
-  if (includeCommand === true) {
-    assignments.push(this.commandAssignment);
-  }
-  Object.keys(this._assignments).forEach(function(name) {
-    assignments.push(this.getAssignment(name));
-  }, this);
-  return assignments;
-};
-
-/**
- * There are a few places where we need to know what the 'next thing' is. What
- * is the user going to be filling out next (assuming they don't enter a named
- * argument). The next argument is the first in line that is both blank, and
- * that can be filled in positionally.
- * @return The next assignment to be used, or null if all the positional
- * parameters have values.
- */
-Requisition.prototype._getFirstBlankPositionalAssignment = function() {
-  var reply = null;
-  Object.keys(this._assignments).some(function(name) {
-    var assignment = this.getAssignment(name);
-    if (assignment.arg.type === 'BlankArgument' &&
-            assignment.param.isPositionalAllowed) {
-      reply = assignment;
-      return true; // i.e. break
-    }
-    return false;
-  }, this);
-  return reply;
-};
-
-/**
- * The update process is asynchronous, so there is (unavoidably) a window
- * where we've worked out the command but don't yet understand all the params.
- * If we try to do things to a requisition in this window we may get
- * inconsistent results. Asynchronous promises have made the window bigger.
- * The only time we've seen this in practice is during focus events due to
- * clicking on a shortcut. The focus want to check the cursor position while
- * the shortcut is updating the command line.
- * This function allows us to detect and back out of this problem.
- * We should be able to remove this function when all the state in a
- * requisition can be encapsulated and updated atomically.
- */
-Requisition.prototype.isUpToDate = function() {
-  if (!this._args) {
-    return false;
-  }
-  for (var i = 0; i < this._args.length; i++) {
-    if (this._args[i].assignment == null) {
-      return false;
-    }
-  }
-  return true;
-};
-
-/**
- * Look through the arguments attached to our assignments for the assignment
- * at the given position.
- * @param {number} cursor The cursor position to query
- */
-Requisition.prototype.getAssignmentAt = function(cursor) {
-  // We short circuit this one because we may have no args, or no args with
-  // any size and the alg below only finds arguments with size.
-  if (cursor === 0) {
-    return this.commandAssignment;
-  }
-
-  var assignForPos = [];
-  var i, j;
-  for (i = 0; i < this._args.length; i++) {
-    var arg = this._args[i];
-    var assignment = arg.assignment;
-
-    // prefix and text are clearly part of the argument
-    for (j = 0; j < arg.prefix.length; j++) {
-      assignForPos.push(assignment);
-    }
-    for (j = 0; j < arg.text.length; j++) {
-      assignForPos.push(assignment);
-    }
-
-    // suffix is part of the argument only if this is a named parameter,
-    // otherwise it looks forwards
-    if (arg.assignment.arg.type === 'NamedArgument') {
-      // leave the argument as it is
-    }
-    else if (this._args.length > i + 1) {
-      // first to the next argument
-      assignment = this._args[i + 1].assignment;
-    }
-    else {
-      // then to the first blank positional parameter, leaving 'as is' if none
-      var nextAssignment = this._getFirstBlankPositionalAssignment();
-      if (nextAssignment != null) {
-        assignment = nextAssignment;
-      }
-    }
-
-    for (j = 0; j < arg.suffix.length; j++) {
-      assignForPos.push(assignment);
-    }
-  }
-
-  // Possible shortcut, we don't really need to go through all the args
-  // to work out the solution to this
-
-  return assignForPos[cursor - 1];
-};
-
-/**
- * Extract a canonical version of the input
- * @return a promise of a string which is the canonical version of what was
- * typed
- */
-Requisition.prototype.toCanonicalString = function() {
-  var cmd = this.commandAssignment.value ?
-      this.commandAssignment.value.name :
-      this.commandAssignment.arg.text;
-
-  // Canonically, if we've opened with a { then we should have a } to close
-  var lineSuffix = '';
-  if (cmd === '{') {
-    var scriptSuffix = this.getAssignment(0).arg.suffix;
-    lineSuffix = (!scriptSuffix.includes('}')) ? ' }' : '';
-  }
-
-  var ctx = this.executionContext;
-
-  // First stringify all the arguments
-  var argPromise = util.promiseEach(this.getAssignments(), assignment => {
-    // Bug 664377: This will cause problems if there is a non-default value
-    // after a default value. Also we need to decide when to use
-    // named parameters in place of positional params. Both can wait.
-    if (assignment.value === assignment.param.defaultValue) {
-      return '';
-    }
-
-    var val = assignment.param.type.stringify(assignment.value, ctx);
-    return Promise.resolve(val).then(str => {
-      return ' ' + str;
-    });
-  });
-
-  return argPromise.then(strings => {
-    return cmd + strings.join('') + lineSuffix;
-  });
-};
-
-/**
- * Reconstitute the input from the args
- */
-Requisition.prototype.toString = function() {
-  if (!this._args) {
-    throw new Error('toString requires a command line. See source.');
-  }
-
-  return this._args.map(function(arg) {
-    return arg.toString();
-  }).join('');
-};
-
-/**
- * For test/debug use only. The output from this function is subject to wanton
- * random change without notice, and should not be relied upon to even exist
- * at some later date.
- */
-Object.defineProperty(Requisition.prototype, '_summaryJson', {
-  get: function() {
-    var summary = {
-      $args: this._args.map(function(arg) {
-        return arg._summaryJson;
-      }),
-      _command: this.commandAssignment._summaryJson,
-      _unassigned: this._unassigned.forEach(function(assignment) {
-        return assignment._summaryJson;
-      })
-    };
-
-    Object.keys(this._assignments).forEach(name => {
-      summary[name] = this.getAssignment(name)._summaryJson;
-    });
-
-    return summary;
-  },
-  enumerable: true
-});
-
-/**
- * When any assignment changes, we might need to update the _args array to
- * match and inform people of changes to the typed input text.
- */
-Requisition.prototype._setAssignmentInternal = function(assignment, conversion) {
-  var oldConversion = assignment.conversion;
-
-  assignment.conversion = conversion;
-  assignment.conversion.assignment = assignment;
-
-  // Do nothing if the conversion is unchanged
-  if (assignment.conversion.equals(oldConversion)) {
-    if (assignment === this.commandAssignment) {
-      this._setBlankArguments();
-    }
-    return;
-  }
-
-  // When the command changes, we need to keep a bunch of stuff in sync
-  if (assignment === this.commandAssignment) {
-    this._assignments = {};
-
-    var command = this.commandAssignment.value;
-    if (command) {
-      for (var i = 0; i < command.params.length; i++) {
-        var param = command.params[i];
-        var newAssignment = new Assignment(param);
-        this._setBlankAssignment(newAssignment);
-        this._assignments[param.name] = newAssignment;
-      }
-    }
-    this.assignmentCount = Object.keys(this._assignments).length;
-  }
-};
-
-/**
- * Internal function to alter the given assignment using the given arg.
- * @param assignment The assignment to alter
- * @param arg The new value for the assignment. An instance of Argument, or an
- * instance of Conversion, or null to set the blank value.
- * @param options There are a number of ways to customize how the assignment
- * is made, including:
- * - internal: (default:false) External updates are required to do more work,
- *   including adjusting the args in this requisition to stay in sync.
- *   On the other hand non internal changes use beginChange to back out of
- *   changes when overtaken asynchronously.
- *   Setting internal:true effectively means this is being called as part of
- *   the update process.
- * - matchPadding: (default:false) Alter the whitespace on the prefix and
- *   suffix of the new argument to match that of the old argument. This only
- *   makes sense with internal=false
- * @return A promise that resolves to undefined when the assignment is complete
- */
-Requisition.prototype.setAssignment = function(assignment, arg, options) {
-  options = options || {};
-  if (!options.internal) {
-    var originalArgs = assignment.arg.getArgs();
-
-    // Update the args array
-    var replacementArgs = arg.getArgs();
-    var maxLen = Math.max(originalArgs.length, replacementArgs.length);
-    for (var i = 0; i < maxLen; i++) {
-      // If there are no more original args, or if the original arg was blank
-      // (i.e. not typed by the user), we'll just need to add at the end
-      if (i >= originalArgs.length || originalArgs[i].type === 'BlankArgument') {
-        this._args.push(replacementArgs[i]);
-        continue;
-      }
-
-      var index = this._args.indexOf(originalArgs[i]);
-      if (index === -1) {
-        console.error('Couldn\'t find ', originalArgs[i], ' in ', this._args);
-        throw new Error('Couldn\'t find ' + originalArgs[i]);
-      }
-
-      // If there are no more replacement args, we just remove the original args
-      // Otherwise swap original args and replacements
-      if (i >= replacementArgs.length) {
-        this._args.splice(index, 1);
-      }
-      else {
-        if (options.matchPadding) {
-          if (replacementArgs[i].prefix.length === 0 &&
-              this._args[index].prefix.length !== 0) {
-            replacementArgs[i].prefix = this._args[index].prefix;
-          }
-          if (replacementArgs[i].suffix.length === 0 &&
-              this._args[index].suffix.length !== 0) {
-            replacementArgs[i].suffix = this._args[index].suffix;
-          }
-        }
-        this._args[index] = replacementArgs[i];
-      }
-    }
-  }
-
-  var updateId = options.internal ? null : this._beginChange();
-
-  var setAssignmentInternal = conversion => {
-    if (options.internal || this._isChangeCurrent(updateId)) {
-      this._setAssignmentInternal(assignment, conversion);
-    }
-
-    if (!options.internal) {
-      this._endChangeCheckOrder(updateId);
-    }
-
-    return Promise.resolve(undefined);
-  };
-
-  if (arg == null) {
-    var blank = assignment.param.type.getBlank(this.executionContext);
-    return setAssignmentInternal(blank);
-  }
-
-  if (typeof arg.getStatus === 'function') {
-    // It's not really an arg, it's a conversion already
-    return setAssignmentInternal(arg);
-  }
-
-  var parsed = assignment.param.type.parse(arg, this.executionContext);
-  return parsed.then(setAssignmentInternal);
-};
-
-/**
- * Reset an assignment to its default value.
- * For internal use only.
- * Happens synchronously.
- */
-Requisition.prototype._setBlankAssignment = function(assignment) {
-  var blank = assignment.param.type.getBlank(this.executionContext);
-  this._setAssignmentInternal(assignment, blank);
-};
-
-/**
- * Reset all the assignments to their default values.
- * For internal use only.
- * Happens synchronously.
- */
-Requisition.prototype._setBlankArguments = function() {
-  this.getAssignments().forEach(this._setBlankAssignment.bind(this));
-};
-
-/**
- * Input trace gives us an array of Argument tracing objects, one for each
- * character in the typed input, from which we can derive information about how
- * to display this typed input. It's a bit like toString on steroids.
- * <p>
- * The returned object has the following members:<ul>
- * <li>character: The character to which this arg trace refers.
- * <li>arg: The Argument to which this character is assigned.
- * <li>part: One of ['prefix'|'text'|suffix'] - how was this char understood
- * </ul>
- * <p>
- * The Argument objects are as output from tokenize() rather than as applied
- * to Assignments by _assign() (i.e. they are not instances of NamedArgument,
- * ArrayArgument, etc).
- * <p>
- * To get at the arguments applied to the assignments simply call
- * <tt>arg.assignment.arg</tt>. If <tt>arg.assignment.arg !== arg</tt> then
- * the arg applied to the assignment will contain the original arg.
- * See _assign() for details.
- */
-Requisition.prototype.createInputArgTrace = function() {
-  if (!this._args) {
-    throw new Error('createInputMap requires a command line. See source.');
-  }
-
-  var args = [];
-  var i;
-  this._args.forEach(function(arg) {
-    for (i = 0; i < arg.prefix.length; i++) {
-      args.push({ arg: arg, character: arg.prefix[i], part: 'prefix' });
-    }
-    for (i = 0; i < arg.text.length; i++) {
-      args.push({ arg: arg, character: arg.text[i], part: 'text' });
-    }
-    for (i = 0; i < arg.suffix.length; i++) {
-      args.push({ arg: arg, character: arg.suffix[i], part: 'suffix' });
-    }
-  });
-
-  return args;
-};
-
-/**
- * If the last character is whitespace then things that we suggest to add to
- * the end don't need a space prefix.
- * While this is quite a niche function, it has 2 benefits:
- * - it's more correct because we can distinguish between final whitespace that
- *   is part of an unclosed string, and parameter separating whitespace.
- * - also it's faster than toString() the whole thing and checking the end char
- * @return true iff the last character is interpreted as parameter separating
- * whitespace
- */
-Requisition.prototype.typedEndsWithSeparator = function() {
-  if (!this._args) {
-    throw new Error('typedEndsWithSeparator requires a command line. See source.');
-  }
-
-  if (this._args.length === 0) {
-    return false;
-  }
-
-  // This is not as easy as doing (this.toString().slice(-1) === ' ')
-  // See the doc comments above; We're checking for separators, not spaces
-  var lastArg = this._args.slice(-1)[0];
-  if (lastArg.suffix.slice(-1) === ' ') {
-    return true;
-  }
-
-  return lastArg.text === '' && lastArg.suffix === ''
-      && lastArg.prefix.slice(-1) === ' ';
-};
-
-/**
- * Return an array of Status scores so we can create a marked up
- * version of the command line input.
- * @param cursor We only take a status of INCOMPLETE to be INCOMPLETE when the
- * cursor is actually in the argument. Otherwise it's an error.
- * @return Array of objects each containing <tt>status</tt> property and a
- * <tt>string</tt> property containing the characters to which the status
- * applies. Concatenating the strings in order gives the original input.
- */
-Requisition.prototype.getInputStatusMarkup = function(cursor) {
-  var argTraces = this.createInputArgTrace();
-  // Generally the 'argument at the cursor' is the argument before the cursor
-  // unless it is before the first char, in which case we take the first.
-  cursor = cursor === 0 ? 0 : cursor - 1;
-  var cTrace = argTraces[cursor];
-
-  var markup = [];
-  for (var i = 0; i < argTraces.length; i++) {
-    var argTrace = argTraces[i];
-    var arg = argTrace.arg;
-    var status = Status.VALID;
-    // When things get very async we can get here while something else is
-    // doing an update, in which case arg.assignment == null, so we check first
-    if (argTrace.part === 'text' && arg.assignment != null) {
-      status = arg.assignment.getStatus(arg);
-      // Promote INCOMPLETE to ERROR  ...
-      if (status === Status.INCOMPLETE) {
-        // If the cursor is in the prefix or suffix of an argument then we
-        // don't consider it in the argument for the purposes of preventing
-        // the escalation to ERROR. However if this is a NamedArgument, then we
-        // allow the suffix (as space between 2 parts of the argument) to be in.
-        // We use arg.assignment.arg not arg because we're looking at the arg
-        // that got put into the assignment not as returned by tokenize()
-        var isNamed = (cTrace.arg.assignment.arg.type === 'NamedArgument');
-        var isInside = cTrace.part === 'text' ||
-                        (isNamed && cTrace.part === 'suffix');
-        if (arg.assignment !== cTrace.arg.assignment || !isInside) {
-          // And if we're not in the command
-          if (!(arg.assignment instanceof CommandAssignment)) {
-            status = Status.ERROR;
-          }
-        }
-      }
-    }
-
-    markup.push({ status: status, string: argTrace.character });
-  }
-
-  // De-dupe: merge entries where 2 adjacent have same status
-  i = 0;
-  while (i < markup.length - 1) {
-    if (markup[i].status === markup[i + 1].status) {
-      markup[i].string += markup[i + 1].string;
-      markup.splice(i + 1, 1);
-    }
-    else {
-      i++;
-    }
-  }
-
-  return markup;
-};
-
-/**
- * Describe the state of the current input in a way that allows display of
- * predictions and completion hints
- * @param start The location of the cursor
- * @param rank The index of the chosen prediction
- * @return A promise of an object containing the following properties:
- * - statusMarkup: An array of Status scores so we can create a marked up
- *   version of the command line input. See getInputStatusMarkup() for details
- * - unclosedJs: Is the entered command a JS command with no closing '}'?
- * - directTabText: A promise of the text that we *add* to the command line
- *   when TAB is pressed, to be displayed directly after the cursor. See also
- *   arrowTabText.
- * - emptyParameters: A promise of the text that describes the arguments that
- *   the user is yet to type.
- * - arrowTabText: A promise of the text that *replaces* the current argument
- *   when TAB is pressed, generally displayed after a "|->" symbol. See also
- *   directTabText.
- */
-Requisition.prototype.getStateData = function(start, rank) {
-  var typed = this.toString();
-  var current = this.getAssignmentAt(start);
-  var context = this.executionContext;
-  var predictionPromise = (typed.trim().length !== 0) ?
-                          current.getPredictionRanked(context, rank) :
-                          Promise.resolve(null);
-
-  return predictionPromise.then(prediction => {
-    // directTabText is for when the current input is a prefix of the completion
-    // arrowTabText is for when we need to use an -> to show what will be used
-    var directTabText = '';
-    var arrowTabText = '';
-    var emptyParameters = [];
-
-    if (typed.trim().length !== 0) {
-      var cArg = current.arg;
-
-      if (prediction) {
-        var tabText = prediction.name;
-        var existing = cArg.text;
-
-        // Normally the cursor being just before whitespace means that you are
-        // 'in' the previous argument, which means that the prediction is based
-        // on that argument, however NamedArguments break this by having 2 parts
-        // so we need to prepend the tabText with a space for NamedArguments,
-        // but only when there isn't already a space at the end of the prefix
-        // (i.e. ' --name' not ' --name ')
-        if (current.isInName()) {
-          tabText = ' ' + tabText;
-        }
-
-        if (existing !== tabText) {
-          // Decide to use directTabText or arrowTabText
-          // Strip any leading whitespace from the user inputted value because
-          // the tabText will never have leading whitespace.
-          var inputValue = existing.replace(/^\s*/, '');
-          var isStrictCompletion = tabText.indexOf(inputValue) === 0;
-          if (isStrictCompletion && start === typed.length) {
-            // Display the suffix of the prediction as the completion
-            var numLeadingSpaces = existing.match(/^(\s*)/)[0].length;
-
-            directTabText = tabText.slice(existing.length - numLeadingSpaces);
-          }
-          else {
-            // Display the '-> prediction' at the end of the completer element
-            // \u21E5 is the JS escape right arrow
-            arrowTabText = '\u21E5 ' + tabText;
-          }
-        }
-      }
-      else {
-        // There's no prediction, but if this is a named argument that needs a
-        // value (that is without any) then we need to show that one is needed
-        // For example 'git commit --message ', clearly needs some more text
-        if (cArg.type === 'NamedArgument' && cArg.valueArg == null) {
-          emptyParameters.push('<' + current.param.type.name + '>\u00a0');
-        }
-      }
-    }
-
-    // Add a space between the typed text (+ directTabText) and the hints,
-    // making sure we don't add 2 sets of padding
-    if (directTabText !== '') {
-      directTabText += '\u00a0'; // a.k.a &nbsp;
-    }
-    else if (!this.typedEndsWithSeparator()) {
-      emptyParameters.unshift('\u00a0');
-    }
-
-    // Calculate the list of parameters to be filled in
-    // We generate an array of emptyParameter markers for each positional
-    // parameter to the current command.
-    // Generally each emptyParameter marker begins with a space to separate it
-    // from whatever came before, unless what comes before ends in a space.
-
-    this.getAssignments().forEach(assignment => {
-      // Named arguments are handled with a group [options] marker
-      if (!assignment.param.isPositionalAllowed) {
-        return;
-      }
-
-      // No hints if we've got content for this parameter
-      if (assignment.arg.toString().trim() !== '') {
-        return;
-      }
-
-      // No hints if we have a prediction
-      if (directTabText !== '' && current === assignment) {
-        return;
-      }
-
-      var text = (assignment.param.isDataRequired) ?
-          '<' + assignment.param.name + '>\u00a0' :
-          '[' + assignment.param.name + ']\u00a0';
-
-      emptyParameters.push(text);
-    });
-
-    var command = this.commandAssignment.value;
-    var addOptionsMarker = false;
-
-    // We add an '[options]' marker when there are named parameters that are
-    // not filled in and not hidden, and we don't have any directTabText
-    if (command && command.hasNamedParameters) {
-      command.params.forEach(function(param) {
-        var arg = this.getAssignment(param.name).arg;
-        if (!param.isPositionalAllowed && !param.hidden
-                && arg.type === 'BlankArgument') {
-          addOptionsMarker = true;
-        }
-      }, this);
-    }
-
-    if (addOptionsMarker) {
-      // Add an nbsp if we don't have one at the end of the input or if
-      // this isn't the first param we've mentioned
-      emptyParameters.push('[options]\u00a0');
-    }
-
-    // Is the entered command a JS command with no closing '}'?
-    var unclosedJs = command && command.name === '{' &&
-                     !this.getAssignment(0).arg.suffix.includes('}');
-
-    return {
-      statusMarkup: this.getInputStatusMarkup(start),
-      unclosedJs: unclosedJs,
-      directTabText: directTabText,
-      arrowTabText: arrowTabText,
-      emptyParameters: emptyParameters
-    };
-  });
-};
-
-/**
- * Pressing TAB sometimes requires that we add a space to denote that we're on
- * to the 'next thing'.
- * @param assignment The assignment to which to append the space
- */
-Requisition.prototype._addSpace = function(assignment) {
-  var arg = assignment.arg.beget({ suffixSpace: true });
-  if (arg !== assignment.arg) {
-    return this.setAssignment(assignment, arg);
-  }
-  else {
-    return Promise.resolve(undefined);
-  }
-};
-
-/**
- * Complete the argument at <tt>cursor</tt>.
- * Basically the same as:
- *   assignment = getAssignmentAt(cursor);
- *   assignment.value = assignment.conversion.predictions[0];
- * Except it's done safely, and with particular care to where we place the
- * space, which is complex, and annoying if we get it wrong.
- *
- * WARNING: complete() can happen asynchronously.
- *
- * @param cursor The cursor configuration. Should have start and end properties
- * which should be set to start and end of the selection.
- * @param rank The index of the prediction that we should choose.
- * This number is not bounded by the size of the prediction array, we take the
- * modulus to get it within bounds
- * @return A promise which completes (with undefined) when any outstanding
- * completion tasks are done.
- */
-Requisition.prototype.complete = function(cursor, rank) {
-  var assignment = this.getAssignmentAt(cursor.start);
-
-  var context = this.executionContext;
-  var predictionPromise = assignment.getPredictionRanked(context, rank);
-  return predictionPromise.then(prediction => {
-    var outstanding = [];
-
-    // Note: Since complete is asynchronous we should perhaps have a system to
-    // bail out of making changes if the command line has changed since TAB
-    // was pressed. It's not yet clear if this will be a problem.
-
-    if (prediction == null) {
-      // No predictions generally means we shouldn't change anything on TAB,
-      // but TAB has the connotation of 'next thing' and when we're at the end
-      // of a thing that implies that we should add a space. i.e.
-      // 'help<TAB>' -> 'help '
-      // But we should only do this if the thing that we're 'completing' is
-      // valid and doesn't already end in a space.
-      if (assignment.arg.suffix.slice(-1) !== ' ' &&
-              assignment.getStatus() === Status.VALID) {
-        outstanding.push(this._addSpace(assignment));
-      }
-
-      // Also add a space if we are in the name part of an assignment, however
-      // this time we don't want the 'push the space to the next assignment'
-      // logic, so we don't use addSpace
-      if (assignment.isInName()) {
-        var newArg = assignment.arg.beget({ prefixPostSpace: true });
-        outstanding.push(this.setAssignment(assignment, newArg));
-      }
-    }
-    else {
-      // Mutate this argument to hold the completion
-      var arg = assignment.arg.beget({
-        text: prediction.name,
-        dontQuote: (assignment === this.commandAssignment)
-      });
-      var assignPromise = this.setAssignment(assignment, arg);
-
-      if (!prediction.incomplete) {
-        assignPromise = assignPromise.then(() => {
-          // The prediction is complete, add a space to let the user move-on
-          return this._addSpace(assignment).then(() => {
-            // Bug 779443 - Remove or explain the re-parse
-            if (assignment instanceof UnassignedAssignment) {
-              return this.update(this.toString());
-            }
-          });
-        });
-      }
-
-      outstanding.push(assignPromise);
-    }
-
-    return Promise.all(outstanding).then(() => {
-      return true;
-    });
-  });
-};
-
-/**
- * Replace the current value with the lower value if such a concept exists.
- */
-Requisition.prototype.nudge = function(assignment, by) {
-  var ctx = this.executionContext;
-  var val = assignment.param.type.nudge(assignment.value, by, ctx);
-  return Promise.resolve(val).then(replacement => {
-    if (replacement != null) {
-      var val = assignment.param.type.stringify(replacement, ctx);
-      return Promise.resolve(val).then(str => {
-        var arg = assignment.arg.beget({ text: str });
-        return this.setAssignment(assignment, arg);
-      });
-    }
-  });
-};
-
-/**
- * Helper to find the 'data-command' attribute, used by |update()|
- */
-function getDataCommandAttribute(element) {
-  var command = element.getAttribute('data-command');
-  if (!command) {
-    command = element.querySelector('*[data-command]')
-                     .getAttribute('data-command');
-  }
-  return command;
-}
-
-/**
- * Designed to be called from context.update(). Acts just like update() except
- * that it also calls onExternalUpdate() to inform the UI of an unexpected
- * change to the current command.
- */
-Requisition.prototype._contextUpdate = function(typed) {
-  return this.update(typed).then(reply => {
-    this.onExternalUpdate({ typed: typed });
-    return reply;
-  });
-};
-
-/**
- * Called by the UI when ever the user interacts with a command line input
- * @param typed The contents of the input field OR an HTML element (or an event
- * that targets an HTML element) which has a data-command attribute or a child
- * with the same that contains the command to update with
- */
-Requisition.prototype.update = function(typed) {
-  // Should be "if (typed instanceof HTMLElement)" except Gecko
-  if (typeof typed.querySelector === 'function') {
-    typed = getDataCommandAttribute(typed);
-  }
-  // Should be "if (typed instanceof Event)" except Gecko
-  if (typeof typed.currentTarget === 'object') {
-    typed = getDataCommandAttribute(typed.currentTarget);
-  }
-
-  var updateId = this._beginChange();
-
-  this._args = exports.tokenize(typed);
-  var args = this._args.slice(0); // i.e. clone
-
-  this._split(args);
-
-  return this._assign(args).then(() => {
-    return this._endChangeCheckOrder(updateId);
-  });
-};
-
-/**
- * Similar to update('') except that it's guaranteed to execute synchronously
- */
-Requisition.prototype.clear = function() {
-  var arg = new Argument('', '', '');
-  this._args = [ arg ];
-
-  var conversion = commandModule.parse(this.executionContext, arg, false);
-  this.setAssignment(this.commandAssignment, conversion, { internal: true });
-};
-
-/**
- * tokenize() is a state machine. These are the states.
- */
-var In = {
-  /**
-   * The last character was ' '.
-   * Typing a ' ' character will not change the mode
-   * Typing one of '"{ will change mode to SINGLE_Q, DOUBLE_Q or SCRIPT.
-   * Anything else goes into SIMPLE mode.
-   */
-  WHITESPACE: 1,
-
-  /**
-   * The last character was part of a parameter.
-   * Typing ' ' returns to WHITESPACE mode. Any other character
-   * (including '"{} which are otherwise special) does not change the mode.
-   */
-  SIMPLE: 2,
-
-  /**
-   * We're inside single quotes: '
-   * Typing ' returns to WHITESPACE mode. Other characters do not change mode.
-   */
-  SINGLE_Q: 3,
-
-  /**
-   * We're inside double quotes: "
-   * Typing " returns to WHITESPACE mode. Other characters do not change mode.
-   */
-  DOUBLE_Q: 4,
-
-  /**
-   * We're inside { and }
-   * Typing } returns to WHITESPACE mode. Other characters do not change mode.
-   * SCRIPT mode is slightly different from other modes in that spaces between
-   * the {/} delimiters and the actual input are not considered significant.
-   * e.g: " x " is a 3 character string, delimited by double quotes, however
-   * { x } is a 1 character JavaScript surrounded by whitespace and {}
-   * delimiters.
-   * In the short term we assume that the JS routines can make sense of the
-   * extra whitespace, however at some stage we may need to move the space into
-   * the Argument prefix/suffix.
-   * Also we don't attempt to handle nested {}. See bug 678961
-   */
-  SCRIPT: 5
-};
-
-/**
- * Split up the input taking into account ', " and {.
- * We don't consider \t or other classical whitespace characters to split
- * arguments apart. For one thing these characters are hard to type, but also
- * if the user has gone to the trouble of pasting a TAB character into the
- * input field (or whatever it takes), they probably mean it.
- */
-exports.tokenize = function(typed) {
-  // For blank input, place a dummy empty argument into the list
-  if (typed == null || typed.length === 0) {
-    return [ new Argument('', '', '') ];
-  }
-
-  if (isSimple(typed)) {
-    return [ new Argument(typed, '', '') ];
-  }
-
-  var mode = In.WHITESPACE;
-
-  // First we swap out escaped characters that are special to the tokenizer.
-  // So a backslash followed by any of ['"{} ] is turned into a unicode private
-  // char so we can swap back later
-  typed = typed
-      .replace(/\\\\/g, '\uF000')
-      .replace(/\\ /g, '\uF001')
-      .replace(/\\'/g, '\uF002')
-      .replace(/\\"/g, '\uF003')
-      .replace(/\\{/g, '\uF004')
-      .replace(/\\}/g, '\uF005');
-
-  function unescape2(escaped) {
-    return escaped
-        .replace(/\uF000/g, '\\\\')
-        .replace(/\uF001/g, '\\ ')
-        .replace(/\uF002/g, '\\\'')
-        .replace(/\uF003/g, '\\\"')
-        .replace(/\uF004/g, '\\{')
-        .replace(/\uF005/g, '\\}');
-  }
-
-  var i = 0;          // The index of the current character
-  var start = 0;      // Where did this section start?
-  var prefix = '';    // Stuff that comes before the current argument
-  var args = [];      // The array that we're creating
-  var blockDepth = 0; // For JS with nested {}
-
-  // This is just a state machine. We're going through the string char by char
-  // The 'mode' is one of the 'In' states. As we go, we're adding Arguments
-  // to the 'args' array.
-
-  while (true) {
-    var c = typed[i];
-    var str;
-    switch (mode) {
-      case In.WHITESPACE:
-        if (c === '\'') {
-          prefix = typed.substring(start, i + 1);
-          mode = In.SINGLE_Q;
-          start = i + 1;
-        }
-        else if (c === '"') {
-          prefix = typed.substring(start, i + 1);
-          mode = In.DOUBLE_Q;
-          start = i + 1;
-        }
-        else if (c === '{') {
-          prefix = typed.substring(start, i + 1);
-          mode = In.SCRIPT;
-          blockDepth++;
-          start = i + 1;
-        }
-        else if (/ /.test(c)) {
-          // Still whitespace, do nothing
-        }
-        else {
-          prefix = typed.substring(start, i);
-          mode = In.SIMPLE;
-          start = i;
-        }
-        break;
-
-      case In.SIMPLE:
-        // There is an edge case of xx'xx which we are assuming to
-        // be a single parameter (and same with ")
-        if (c === ' ') {
-          str = unescape2(typed.substring(start, i));
-          args.push(new Argument(str, prefix, ''));
-          mode = In.WHITESPACE;
-          start = i;
-          prefix = '';
-        }
-        break;
-
-      case In.SINGLE_Q:
-        if (c === '\'') {
-          str = unescape2(typed.substring(start, i));
-          args.push(new Argument(str, prefix, c));
-          mode = In.WHITESPACE;
-          start = i + 1;
-          prefix = '';
-        }
-        break;
-
-      case In.DOUBLE_Q:
-        if (c === '"') {
-          str = unescape2(typed.substring(start, i));
-          args.push(new Argument(str, prefix, c));
-          mode = In.WHITESPACE;
-          start = i + 1;
-          prefix = '';
-        }
-        break;
-
-      case In.SCRIPT:
-        if (c === '{') {
-          blockDepth++;
-        }
-        else if (c === '}') {
-          blockDepth--;
-          if (blockDepth === 0) {
-            str = unescape2(typed.substring(start, i));
-            args.push(new ScriptArgument(str, prefix, c));
-            mode = In.WHITESPACE;
-            start = i + 1;
-            prefix = '';
-          }
-        }
-        break;
-    }
-
-    i++;
-
-    if (i >= typed.length) {
-      // There is nothing else to read - tidy up
-      if (mode === In.WHITESPACE) {
-        if (i !== start) {
-          // There's whitespace at the end of the typed string. Add it to the
-          // last argument's suffix, creating an empty argument if needed.
-          var extra = typed.substring(start, i);
-          var lastArg = args[args.length - 1];
-          if (!lastArg) {
-            args.push(new Argument('', extra, ''));
-          }
-          else {
-            lastArg.suffix += extra;
-          }
-        }
-      }
-      else if (mode === In.SCRIPT) {
-        str = unescape2(typed.substring(start, i + 1));
-        args.push(new ScriptArgument(str, prefix, ''));
-      }
-      else {
-        str = unescape2(typed.substring(start, i + 1));
-        args.push(new Argument(str, prefix, ''));
-      }
-      break;
-    }
-  }
-
-  return args;
-};
-
-/**
- * If the input has no spaces, quotes, braces or escapes,
- * we can take the fast track.
- */
-function isSimple(typed) {
-   for (var i = 0; i < typed.length; i++) {
-     var c = typed.charAt(i);
-     if (c === ' ' || c === '"' || c === '\'' ||
-         c === '{' || c === '}' || c === '\\') {
-       return false;
-     }
-   }
-   return true;
-}
-
-/**
- * Looks in the commands for a command extension that matches what has been
- * typed at the command line.
- */
-Requisition.prototype._split = function(args) {
-  // Handle the special case of the user typing { javascript(); }
-  // We use the hidden 'eval' command directly rather than shift()ing one of
-  // the parameters, and parse()ing it.
-  var conversion;
-  if (args[0].type === 'ScriptArgument') {
-    // Special case: if the user enters { console.log('foo'); } then we need to
-    // use the hidden 'eval' command
-    var command = this.system.commands.get(evalCmd.name);
-    conversion = new Conversion(command, new ScriptArgument());
-    this._setAssignmentInternal(this.commandAssignment, conversion);
-    return;
-  }
-
-  var argsUsed = 1;
-
-  while (argsUsed <= args.length) {
-    var arg = (argsUsed === 1) ?
-              args[0] :
-              new MergedArgument(args, 0, argsUsed);
-
-    if (this.prefix != null && this.prefix !== '') {
-      var prefixArg = new Argument(this.prefix, '', ' ');
-      var prefixedArg = new MergedArgument([ prefixArg, arg ]);
-
-      conversion = commandModule.parse(this.executionContext, prefixedArg, false);
-      if (conversion.value == null) {
-        conversion = commandModule.parse(this.executionContext, arg, false);
-      }
-    }
-    else {
-      conversion = commandModule.parse(this.executionContext, arg, false);
-    }
-
-    // We only want to carry on if this command is a parent command,
-    // which means that there is a commandAssignment, but not one with
-    // an exec function.
-    if (!conversion.value || conversion.value.exec) {
-      break;
-    }
-
-    // Previously we needed a way to hide commands depending context.
-    // We have not resurrected that feature yet, but if we do we should
-    // insert code here to ignore certain commands depending on the
-    // context/environment
-
-    argsUsed++;
-  }
-
-  // This could probably be re-written to consume args as we go
-  for (var i = 0; i < argsUsed; i++) {
-    args.shift();
-  }
-
-  this._setAssignmentInternal(this.commandAssignment, conversion);
-};
-
-/**
- * Add all the passed args to the list of unassigned assignments.
- */
-Requisition.prototype._addUnassignedArgs = function(args) {
-  args.forEach(arg => {
-    this._unassigned.push(new UnassignedAssignment(this, arg));
-  });
-
-  return RESOLVED;
-};
-
-/**
- * Work out which arguments are applicable to which parameters.
- */
-Requisition.prototype._assign = function(args) {
-  // See comment in _split. Avoid multiple updates
-  var noArgUp = { internal: true };
-
-  this._unassigned = [];
-
-  if (!this.commandAssignment.value) {
-    return this._addUnassignedArgs(args);
-  }
-
-  if (args.length === 0) {
-    this._setBlankArguments();
-    return RESOLVED;
-  }
-
-  // Create an error if the command does not take parameters, but we have
-  // been given them ...
-  if (this.assignmentCount === 0) {
-    return this._addUnassignedArgs(args);
-  }
-
-  // Special case: if there is only 1 parameter, and that's of type
-  // text, then we put all the params into the first param
-  if (this.assignmentCount === 1) {
-    var assignment = this.getAssignment(0);
-    if (assignment.param.type.name === 'string') {
-      var arg = (args.length === 1) ? args[0] : new MergedArgument(args);
-      return this.setAssignment(assignment, arg, noArgUp);
-    }
-  }
-
-  // Positional arguments can still be specified by name, but if they are
-  // then we need to ignore them when working them out positionally
-  var unassignedParams = this.getParameterNames();
-
-  // We collect the arguments used in arrays here before assigning
-  var arrayArgs = {};
-
-  // Extract all the named parameters
-  var assignments = this.getAssignments(false);
-  var namedDone = util.promiseEach(assignments, function(assignment) {
-    // Loop over the arguments
-    // Using while rather than loop because we remove args as we go
-    var i = 0;
-    while (i < args.length) {
-      if (!assignment.param.isKnownAs(args[i].text)) {
-        // Skip this parameter and handle as a positional parameter
-        i++;
-        continue;
-      }
-
-      var arg = args.splice(i, 1)[0];
-      /* jshint loopfunc:true */
-      unassignedParams = unassignedParams.filter(function(test) {
-        return test !== assignment.param.name;
-      });
-
-      // boolean parameters don't have values, default to false
-      if (assignment.param.type.name === 'boolean') {
-        arg = new TrueNamedArgument(arg);
-      }
-      else {
-        var valueArg = null;
-        if (i + 1 <= args.length) {
-          valueArg = args.splice(i, 1)[0];
-        }
-        arg = new NamedArgument(arg, valueArg);
-      }
-
-      if (assignment.param.type.name === 'array') {
-        var arrayArg = arrayArgs[assignment.param.name];
-        if (!arrayArg) {
-          arrayArg = new ArrayArgument();
-          arrayArgs[assignment.param.name] = arrayArg;
-        }
-        arrayArg.addArgument(arg);
-        return RESOLVED;
-      }
-      else {
-        if (assignment.arg.type === 'BlankArgument') {
-          return this.setAssignment(assignment, arg, noArgUp);
-        }
-        else {
-          return this._addUnassignedArgs(arg.getArgs());
-        }
-      }
-    }
-  }, this);
-
-  // What's left are positional parameters: assign in order
-  var positionalDone = namedDone.then(() => {
-    return util.promiseEach(unassignedParams, function(name) {
-      var assignment = this.getAssignment(name);
-
-      // If not set positionally, and we can't set it non-positionally,
-      // we have to default it to prevent previous values surviving
-      if (!assignment.param.isPositionalAllowed) {
-        this._setBlankAssignment(assignment);
-        return RESOLVED;
-      }
-
-      // If this is a positional array argument, then it swallows the
-      // rest of the arguments.
-      if (assignment.param.type.name === 'array') {
-        var arrayArg = arrayArgs[assignment.param.name];
-        if (!arrayArg) {
-          arrayArg = new ArrayArgument();
-          arrayArgs[assignment.param.name] = arrayArg;
-        }
-        arrayArg.addArguments(args);
-        args = [];
-        // The actual assignment to the array parameter is done below
-        return RESOLVED;
-      }
-
-      // Set assignment to defaults if there are no more arguments
-      if (args.length === 0) {
-        this._setBlankAssignment(assignment);
-        return RESOLVED;
-      }
-
-      var arg = args.splice(0, 1)[0];
-      // --foo and -f are named parameters, -4 is a number. So '-' is either
-      // the start of a named parameter or a number depending on the context
-      var isIncompleteName = assignment.param.type.name === 'number' ?
-          /-[-a-zA-Z_]/.test(arg.text) :
-          arg.text.charAt(0) === '-';
-
-      if (isIncompleteName) {
-        this._unassigned.push(new UnassignedAssignment(this, arg));
-        return RESOLVED;
-      }
-      else {
-        return this.setAssignment(assignment, arg, noArgUp);
-      }
-    }, this);
-  });
-
-  // Now we need to assign the array argument (if any)
-  var arrayDone = positionalDone.then(() => {
-    return util.promiseEach(Object.keys(arrayArgs), function(name) {
-      var assignment = this.getAssignment(name);
-      return this.setAssignment(assignment, arrayArgs[name], noArgUp);
-    }, this);
-  });
-
-  // What's left is can't be assigned, but we need to officially unassign them
-  return arrayDone.then(() => {
-    return this._addUnassignedArgs(args);
-  });
-};
-
-/**
- * Entry point for keyboard accelerators or anything else that wants to execute
- * a command.
- * @param options Object describing how the execution should be handled.
- * (optional). Contains some of the following properties:
- * - hidden (boolean, default=false) Should the output be hidden from the
- *   commandOutputManager for this requisition
- * - command/args A fast shortcut to executing a known command with a known
- *   set of parsed arguments.
- */
-Requisition.prototype.exec = function(options) {
-  var command = null;
-  var args = null;
-  var hidden = false;
-
-  if (options) {
-    if (options.hidden) {
-      hidden = true;
-    }
-
-    if (options.command != null) {
-      // Fast track by looking up the command directly since passed args
-      // means there is no command line to parse.
-      command = this.system.commands.get(options.command);
-      if (!command) {
-        console.error('Command not found: ' + options.command);
-      }
-      args = options.args;
-    }
-  }
-
-  if (!command) {
-    command = this.commandAssignment.value;
-    args = this.getArgsObject();
-  }
-
-  // Display JavaScript input without the initial { or closing }
-  var typed = this.toString();
-  if (evalCmd.isCommandRegexp.test(typed)) {
-    typed = typed.replace(evalCmd.isCommandRegexp, '');
-    // Bug 717763: What if the JavaScript naturally ends with a }?
-    typed = typed.replace(/\s*}\s*$/, '');
-  }
-
-  var output = new Output({
-    command: command,
-    args: args,
-    typed: typed,
-    canonical: this.toCanonicalString(),
-    hidden: hidden
-  });
-
-  this.commandOutputManager.onOutput({ output: output });
-
-  var onDone = function(data) {
-    output.complete(data, false);
-    return output;
-  };
-
-  var onError = function(data, ex) {
-    if (logErrors) {
-      if (ex != null) {
-        util.errorHandler(ex);
-      }
-      else {
-        console.error(data);
-      }
-    }
-
-    if (data != null && typeof data === 'string') {
-      data = data.replace(/^Protocol error: /, ''); // Temp fix for bug 1035296
-    }
-
-    data = (data != null && data.isTypedData) ? data : {
-      isTypedData: true,
-      data: data,
-      type: 'error'
-    };
-    output.complete(data, true);
-    return output;
-  };
-
-  if (this.status !== Status.VALID) {
-    var ex = new Error(this.getStatusMessage());
-    // We only reject a call to exec if GCLI breaks. Errors with commands are
-    // exposed in the 'error' status of the Output object
-    return Promise.resolve(onError(ex)).then(output => {
-      this.clear();
-      return output;
-    });
-  }
-  else {
-    try {
-      return host.exec(() => {
-        return command.exec(args, this.executionContext);
-      }).then(onDone, onError);
-    }
-    catch (ex) {
-      var data = (typeof ex.message === 'string' && ex.stack != null) ?
-                 ex.message : ex;
-      return Promise.resolve(onError(data, ex));
-    }
-    finally {
-      this.clear();
-    }
-  }
-};
-
-/**
- * Designed to be called from context.updateExec(). Acts just like updateExec()
- * except that it also calls onExternalUpdate() to inform the UI of an
- * unexpected change to the current command.
- */
-Requisition.prototype._contextUpdateExec = function(typed, options) {
-  var reqOpts = {
-    document: this.document,
-    environment: this.environment
-  };
-  var child = new Requisition(this.system, reqOpts);
-  return child.updateExec(typed, options).then(function(reply) {
-    child.destroy();
-    return reply;
-  }.bind(child));
-};
-
-/**
- * A shortcut for calling update, resolving the promise and then exec.
- * @param input The string to execute
- * @param options Passed to exec
- * @return A promise of an output object
- */
-Requisition.prototype.updateExec = function(input, options) {
-  return this.update(input).then(() => {
-    return this.exec(options);
-  });
-};
-
-exports.Requisition = Requisition;
-
-/**
- * A simple object to hold information about the output of a command
- */
-function Output(options) {
-  options = options || {};
-  this.command = options.command || '';
-  this.args = options.args || {};
-  this.typed = options.typed || '';
-  this.canonical = options.canonical || '';
-  this.hidden = options.hidden === true ? true : false;
-
-  this.type = undefined;
-  this.data = undefined;
-  this.completed = false;
-  this.error = false;
-  this.start = new Date();
-
-  this.promise = new Promise((resolve, reject) => {
-    this._resolve = resolve;
-  });
-}
-
-/**
- * Called when there is data to display, and the command has finished executing
- * See changed() for details on parameters.
- */
-Output.prototype.complete = function(data, error) {
-  this.end = new Date();
-  this.completed = true;
-  this.error = error;
-
-  if (data != null && data.isTypedData) {
-    this.data = data.data;
-    this.type = data.type;
-  }
-  else {
-    this.data = data;
-    this.type = this.command.returnType;
-    if (this.type == null) {
-      this.type = (this.data == null) ? 'undefined' : typeof this.data;
-    }
-  }
-
-  if (this.type === 'object') {
-    throw new Error('No type from output of ' + this.typed);
-  }
-
-  this._resolve();
-};
-
-/**
- * Call converters.convert using the data in this Output object
- */
-Output.prototype.convert = function(type, conversionContext) {
-  var converters = conversionContext.system.converters;
-  return converters.convert(this.data, this.type, type, conversionContext);
-};
-
-Output.prototype.toJson = function() {
-  // Exceptions don't stringify, so we try a bit harder
-  var data = this.data;
-  if (this.error && JSON.stringify(this.data) === '{}') {
-    data = {
-      columnNumber: data.columnNumber,
-      fileName: data.fileName,
-      lineNumber: data.lineNumber,
-      message: data.message,
-      stack: data.stack
-    };
-  }
-
-  return {
-    typed: this.typed,
-    type: this.type,
-    data: data,
-    isError: this.error
-  };
-};
-
-exports.Output = Output;
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/commands/clear.js
+++ /dev/null
@@ -1,59 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var l10n = require('../util/l10n');
-
-exports.items = [
-  {
-    // A command to clear the output area
-    item: 'command',
-    runAt: 'client',
-    name: 'clear',
-    description: l10n.lookup('clearDesc'),
-    returnType: 'clearoutput',
-    exec: function(args, context) { }
-  },
-  {
-    item: 'converter',
-    from: 'clearoutput',
-    to: 'view',
-    exec: function(ignore, conversionContext) {
-      return {
-        html: '<span onload="${onload}"></span>',
-        data: {
-          onload: function(ev) {
-            // element starts off being the span above, and we walk up the
-            // tree looking for the terminal
-            var element = ev.target;
-            while (element != null && element.terminal == null) {
-              element = element.parentElement;
-            }
-
-            if (element == null) {
-              // This is only an event handler on a completed command
-              // So we're relying on this showing up in the console
-              throw new Error('Failed to find clear');
-            }
-
-            element.terminal.clear();
-          }
-        }
-      };
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/commands/commands.js
+++ /dev/null
@@ -1,570 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var util = require('../util/util');
-var l10n = require('../util/l10n');
-
-/**
- * Implement the localization algorithm for any documentation objects (i.e.
- * description and manual) in a command.
- * @param data The data assigned to a description or manual property
- * @param onUndefined If data == null, should we return the data untouched or
- * lookup a 'we don't know' key in it's place.
- */
-function lookup(data, onUndefined) {
-  if (data == null) {
-    if (onUndefined) {
-      return l10n.lookup(onUndefined);
-    }
-
-    return data;
-  }
-
-  if (typeof data === 'string') {
-    return data;
-  }
-
-  if (typeof data === 'object') {
-    if (data.key) {
-      return l10n.lookup(data.key);
-    }
-
-    var locales = l10n.getPreferredLocales();
-    var translated;
-    locales.some(function(locale) {
-      translated = data[locale];
-      return translated != null;
-    });
-    if (translated != null) {
-      return translated;
-    }
-
-    console.error('Can\'t find locale in descriptions: ' +
-            'locales=' + JSON.stringify(locales) + ', ' +
-            'description=' + JSON.stringify(data));
-    return '(No description)';
-  }
-
-  return l10n.lookup(onUndefined);
-}
-
-
-/**
- * The command object is mostly just setup around a commandSpec (as passed to
- * Commands.add()).
- */
-function Command(types, commandSpec) {
-  Object.keys(commandSpec).forEach(function(key) {
-    this[key] = commandSpec[key];
-  }, this);
-
-  if (!this.name) {
-    throw new Error('All registered commands must have a name');
-  }
-
-  if (this.params == null) {
-    this.params = [];
-  }
-  if (!Array.isArray(this.params)) {
-    throw new Error('command.params must be an array in ' + this.name);
-  }
-
-  this.hasNamedParameters = false;
-  this.description = 'description' in this ? this.description : undefined;
-  this.description = lookup(this.description, 'canonDescNone');
-  this.manual = 'manual' in this ? this.manual : undefined;
-  this.manual = lookup(this.manual);
-
-  // At this point this.params has nested param groups. We want to flatten it
-  // out and replace the param object literals with Parameter objects
-  var paramSpecs = this.params;
-  this.params = [];
-  this.paramGroups = {};
-  this._shortParams = {};
-
-  var addParam = param => {
-    var groupName = param.groupName || l10n.lookup('canonDefaultGroupName');
-    this.params.push(param);
-    if (!this.paramGroups.hasOwnProperty(groupName)) {
-      this.paramGroups[groupName] = [];
-    }
-    this.paramGroups[groupName].push(param);
-  };
-
-  // Track if the user is trying to mix default params and param groups.
-  // All the non-grouped parameters must come before all the param groups
-  // because non-grouped parameters can be assigned positionally, so their
-  // index is important. We don't want 'holes' in the order caused by
-  // parameter groups.
-  var usingGroups = false;
-
-  // In theory this could easily be made recursive, so param groups could
-  // contain nested param groups. Current thinking is that the added
-  // complexity for the UI probably isn't worth it, so this implementation
-  // prevents nesting.
-  paramSpecs.forEach(function(spec) {
-    if (!spec.group) {
-      var param = new Parameter(types, spec, this, null);
-      addParam(param);
-
-      if (!param.isPositionalAllowed) {
-        this.hasNamedParameters = true;
-      }
-
-      if (usingGroups && param.groupName == null) {
-        throw new Error('Parameters can\'t come after param groups.' +
-                        ' Ignoring ' + this.name + '/' + spec.name);
-      }
-
-      if (param.groupName != null) {
-        usingGroups = true;
-      }
-    }
-    else {
-      spec.params.forEach(function(ispec) {
-        var param = new Parameter(types, ispec, this, spec.group);
-        addParam(param);
-
-        if (!param.isPositionalAllowed) {
-          this.hasNamedParameters = true;
-        }
-      }, this);
-
-      usingGroups = true;
-    }
-  }, this);
-
-  this.params.forEach(function(param) {
-    if (param.short != null) {
-      if (this._shortParams[param.short] != null) {
-        throw new Error('Multiple params using short name ' + param.short);
-      }
-      this._shortParams[param.short] = param;
-    }
-  }, this);
-}
-
-/**
- * JSON serializer that avoids non-serializable data
- * @param customProps Array of strings containing additional properties which,
- * if specified in the command spec, will be included in the JSON. Normally we
- * transfer only the properties required for GCLI to function.
- */
-Command.prototype.toJson = function(customProps) {
-  var json = {
-    item: 'command',
-    name: this.name,
-    params: this.params.map(function(param) { return param.toJson(); }),
-    returnType: this.returnType,
-    isParent: (this.exec == null)
-  };
-
-  if (this.description !==  l10n.lookup('canonDescNone')) {
-    json.description = this.description;
-  }
-  if (this.manual != null) {
-    json.manual = this.manual;
-  }
-  if (this.hidden != null) {
-    json.hidden = this.hidden;
-  }
-
-  if (Array.isArray(customProps)) {
-    customProps.forEach(prop => {
-      if (this[prop] != null) {
-        json[prop] = this[prop];
-      }
-    });
-  }
-
-  return json;
-};
-
-/**
- * Easy way to lookup parameters by full name
- */
-Command.prototype.getParameterByName = function(name) {
-  var reply;
-  this.params.forEach(function(param) {
-    if (param.name === name) {
-      reply = param;
-    }
-  });
-  return reply;
-};
-
-/**
- * Easy way to lookup parameters by short name
- */
-Command.prototype.getParameterByShortName = function(short) {
-  return this._shortParams[short];
-};
-
-exports.Command = Command;
-
-
-/**
- * A wrapper for a paramSpec so we can sort out shortened versions names for
- * option switches
- */
-function Parameter(types, paramSpec, command, groupName) {
-  this.command = command || { name: 'unnamed' };
-  this.paramSpec = paramSpec;
-  this.name = this.paramSpec.name;
-  this.type = this.paramSpec.type;
-  this.short = this.paramSpec.short;
-
-  if (this.short != null && !/[0-9A-Za-z]/.test(this.short)) {
-    throw new Error('\'short\' value must be a single alphanumeric digit.');
-  }
-
-  this.groupName = groupName;
-  if (this.groupName != null) {
-    if (this.paramSpec.option != null) {
-      throw new Error('Can\'t have a "option" property in a nested parameter');
-    }
-  }
-  else {
-    if (this.paramSpec.option != null) {
-      this.groupName = (this.paramSpec.option === true) ?
-                       l10n.lookup('canonDefaultGroupName') :
-                       '' + this.paramSpec.option;
-    }
-  }
-
-  if (!this.name) {
-    throw new Error('In ' + this.command.name +
-                    ': all params must have a name');
-  }
-
-  var typeSpec = this.type;
-  this.type = types.createType(typeSpec);
-  if (this.type == null) {
-    console.error('Known types: ' + types.getTypeNames().join(', '));
-    throw new Error('In ' + this.command.name + '/' + this.name +
-                    ': can\'t find type for: ' + JSON.stringify(typeSpec));
-  }
-
-  // boolean parameters have an implicit defaultValue:false, which should
-  // not be changed. See the docs.
-  if (this.type.name === 'boolean' &&
-      this.paramSpec.defaultValue !== undefined) {
-    throw new Error('In ' + this.command.name + '/' + this.name +
-                    ': boolean parameters can not have a defaultValue.' +
-                    ' Ignoring');
-  }
-
-  // All parameters that can only be set via a named parameter must have a
-  // non-undefined default value
-  if (!this.isPositionalAllowed && this.paramSpec.defaultValue === undefined &&
-      this.type.getBlank == null && this.type.name !== 'boolean') {
-    throw new Error('In ' + this.command.name + '/' + this.name +
-                    ': Missing defaultValue for optional parameter.');
-  }
-
-  if (this.paramSpec.defaultValue !== undefined) {
-    this.defaultValue = this.paramSpec.defaultValue;
-  }
-  else {
-    Object.defineProperty(this, 'defaultValue', {
-      get: function() {
-        return this.type.getBlank().value;
-      },
-      enumerable: true
-    });
-  }
-
-  // Resolve the documentation
-  this.manual = lookup(this.paramSpec.manual);
-  this.description = lookup(this.paramSpec.description, 'canonDescNone');
-
-  // Is the user required to enter data for this parameter? (i.e. has
-  // defaultValue been set to something other than undefined)
-  // TODO: When the defaultValue comes from type.getBlank().value (see above)
-  // then perhaps we should set using something like
-  //   isDataRequired = (type.getBlank().status !== VALID)
-  this.isDataRequired = (this.defaultValue === undefined);
-
-  // Are we allowed to assign data to this parameter using positional
-  // parameters?
-  this.isPositionalAllowed = this.groupName == null;
-}
-
-/**
- * Does the given name uniquely identify this param (among the other params
- * in this command)
- * @param name The name to check
- */
-Parameter.prototype.isKnownAs = function(name) {
-  return (name === '--' + this.name) || (name === '-' + this.short);
-};
-
-/**
- * Reflect the paramSpec 'hidden' property (dynamically so it can change)
- */
-Object.defineProperty(Parameter.prototype, 'hidden', {
-  get: function() {
-    return this.paramSpec.hidden;
-  },
-  enumerable: true
-});
-
-/**
- * JSON serializer that avoids non-serializable data
- */
-Parameter.prototype.toJson = function() {
-  var json = {
-    name: this.name,
-    type: this.type.getSpec(this.command.name, this.name),
-    short: this.short
-  };
-
-  // Values do not need to be serializable, so we don't try. For the client
-  // side (which doesn't do any executing) we only care whether default value is
-  // undefined, null, or something else.
-  if (this.paramSpec.defaultValue !== undefined) {
-    json.defaultValue = (this.paramSpec.defaultValue === null) ? null : {};
-  }
-  if (this.paramSpec.description != null) {
-    json.description = this.paramSpec.description;
-  }
-  if (this.paramSpec.manual != null) {
-    json.manual = this.paramSpec.manual;
-  }
-  if (this.paramSpec.hidden != null) {
-    json.hidden = this.paramSpec.hidden;
-  }
-
-  // groupName can be set outside a paramSpec, (e.g. in grouped parameters)
-  // but it works like 'option' does so we use 'option' for groupNames
-  if (this.groupName != null || this.paramSpec.option != null) {
-    json.option = this.groupName || this.paramSpec.option;
-  }
-
-  return json;
-};
-
-exports.Parameter = Parameter;
-
-
-/**
- * A store for a list of commands
- * @param types Each command uses a set of Types to parse its parameters so the
- * Commands container needs access to the list of available types.
- * @param location String that, if set will force all commands to have a
- * matching runAt property to be accepted
- */
-function Commands(types, location) {
-  this.types = types;
-  this.location = location;
-
-  // A lookup hash of our registered commands
-  this._commands = {};
-  // A sorted list of command names, we regularly want them in order, so pre-sort
-  this._commandNames = [];
-  // A lookup of the original commandSpecs by command name
-  this._commandSpecs = {};
-
-  // Enable people to be notified of changes to the list of commands
-  this.onCommandsChange = util.createEvent('commands.onCommandsChange');
-}
-
-/**
- * Add a command to the list of known commands.
- * @param commandSpec The command and its metadata.
- * @return The new command, or null if a location property has been set and the
- * commandSpec doesn't have a matching runAt property.
- */
-Commands.prototype.add = function(commandSpec) {
-  if (this.location != null && commandSpec.runAt != null &&
-      commandSpec.runAt !== this.location) {
-    return;
-  }
-
-  if (this._commands[commandSpec.name] != null) {
-    // Roughly commands.remove() without the event call, which we do later
-    delete this._commands[commandSpec.name];
-    this._commandNames = this._commandNames.filter(function(test) {
-      return test !== commandSpec.name;
-    });
-  }
-
-  var command = new Command(this.types, commandSpec);
-  this._commands[commandSpec.name] = command;
-  this._commandNames.push(commandSpec.name);
-  this._commandNames.sort();
-
-  this._commandSpecs[commandSpec.name] = commandSpec;
-
-  this.onCommandsChange();
-  return command;
-};
-
-/**
- * Remove an individual command. The opposite of Commands.add().
- * Removing a non-existent command is a no-op.
- * @param commandOrName Either a command name or the command itself.
- * @return true if a command was removed, false otherwise.
- */
-Commands.prototype.remove = function(commandOrName) {
-  var name = typeof commandOrName === 'string' ?
-          commandOrName :
-          commandOrName.name;
-
-  if (!this._commands[name]) {
-    return false;
-  }
-
-  // See start of commands.add if changing this code
-  delete this._commands[name];
-  delete this._commandSpecs[name];
-  this._commandNames = this._commandNames.filter(function(test) {
-    return test !== name;
-  });
-
-  this.onCommandsChange();
-  return true;
-};
-
-/**
- * Retrieve a command by name
- * @param name The name of the command to retrieve
- */
-Commands.prototype.get = function(name) {
-  // '|| undefined' is to silence 'reference to undefined property' warnings
-  return this._commands[name] || undefined;
-};
-
-/**
- * Get an array of all the registered commands.
- */
-Commands.prototype.getAll = function() {
-  return Object.keys(this._commands).map(function(name) {
-    return this._commands[name];
-  }, this);
-};
-
-/**
- * Get access to the stored commandMetaDatas (i.e. before they were made into
- * instances of Command/Parameters) so we can remote them.
- * @param customProps Array of strings containing additional properties which,
- * if specified in the command spec, will be included in the JSON. Normally we
- * transfer only the properties required for GCLI to function.
- */
-Commands.prototype.getCommandSpecs = function(customProps) {
-  var commandSpecs = [];
-
-  Object.keys(this._commands).forEach(name => {
-    var command = this._commands[name];
-    if (!command.noRemote) {
-      commandSpecs.push(command.toJson(customProps));
-    }
-  });
-
-  return commandSpecs;
-};
-
-/**
- * Add a set of commands that are executed somewhere else, optionally with a
- * command prefix to distinguish these commands from a local set of commands.
- * @param commandSpecs Presumably as obtained from getCommandSpecs
- * @param remoter Function to call on exec of a new remote command. This is
- * defined just like an exec function (i.e. that takes args/context as params
- * and returns a promise) with one extra feature, that the context includes a
- * 'commandName' property that contains the original command name.
- * @param prefix The name prefix that we assign to all command names
- * @param to URL-like string that describes where the commands are executed.
- * This is to complete the parent command description.
- */
-Commands.prototype.addProxyCommands = function(commandSpecs, remoter, prefix, to) {
-  if (prefix != null) {
-    if (this._commands[prefix] != null) {
-      throw new Error(l10n.lookupFormat('canonProxyExists', [ prefix ]));
-    }
-
-    // We need to add the parent command so all the commands from the other
-    // system have a parent
-    this.add({
-      name: prefix,
-      isProxy: true,
-      description: l10n.lookupFormat('canonProxyDesc', [ to ]),
-      manual: l10n.lookupFormat('canonProxyManual', [ to ])
-    });
-  }
-
-  commandSpecs.forEach(commandSpec => {
-    var originalName = commandSpec.name;
-    if (!commandSpec.isParent) {
-      commandSpec.exec = (args, context) => {
-        context.commandName = originalName;
-        return remoter(args, context);
-      };
-    }
-
-    if (prefix != null) {
-      commandSpec.name = prefix + ' ' + commandSpec.name;
-    }
-    commandSpec.isProxy = true;
-    this.add(commandSpec);
-  });
-};
-
-/**
- * Remove a set of commands added with addProxyCommands.
- * @param prefix The name prefix that we assign to all command names
- */
-Commands.prototype.removeProxyCommands = function(prefix) {
-  var toRemove = [];
-  Object.keys(this._commandSpecs).forEach(name => {
-    if (name.indexOf(prefix) === 0) {
-      toRemove.push(name);
-    }
-  });
-
-  var removed = [];
-  toRemove.forEach(name => {
-    var command = this.get(name);
-    if (command.isProxy) {
-      this.remove(name);
-      removed.push(name);
-    }
-    else {
-      console.error('Skipping removal of \'' + name +
-                    '\' because it is not a proxy command.');
-    }
-  });
-
-  return removed;
-};
-
-exports.Commands = Commands;
-
-/**
- * CommandOutputManager stores the output objects generated by executed
- * commands.
- *
- * CommandOutputManager is exposed to the the outside world and could (but
- * shouldn't) be used before gcli.startup() has been called.
- * This could should be defensive to that where possible, and we should
- * certainly document if the use of it or similar will fail if used too soon.
- */
-function CommandOutputManager() {
-  this.onOutput = util.createEvent('CommandOutputManager.onOutput');
-}
-
-exports.CommandOutputManager = CommandOutputManager;
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/commands/context.js
+++ /dev/null
@@ -1,62 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var l10n = require('../util/l10n');
-var cli = require('../cli');
-
-/**
- * 'context' command
- */
-var context = {
-  item: 'command',
-  name: 'context',
-  description: l10n.lookup('contextDesc'),
-  manual: l10n.lookup('contextManual'),
-  params: [
-   {
-     name: 'prefix',
-     type: 'command',
-     description: l10n.lookup('contextPrefixDesc'),
-     defaultValue: null
-   }
-  ],
-  returnType: 'string',
-  // The context command is client only because it's essentially sugar for
-  // typing commands. When there is a command prefix in action, it is the job
-  // of the remoter to add the prefix to the typed strings that are sent for
-  // remote execution
-  noRemote: true,
-  exec: function echo(args, context) {
-    var requisition = cli.getMapping(context).requisition;
-
-    if (args.prefix == null) {
-      requisition.prefix = null;
-      return l10n.lookup('contextEmptyReply');
-    }
-
-    if (args.prefix.exec != null) {
-      throw new Error(l10n.lookupFormat('contextNotParentError',
-                                        [ args.prefix.name ]));
-    }
-
-    requisition.prefix = args.prefix.name;
-    return l10n.lookupFormat('contextReply', [ args.prefix.name ]);
-  }
-};
-
-exports.items = [ context ];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/commands/help.js
+++ /dev/null
@@ -1,387 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var l10n = require('../util/l10n');
-var cli = require('../cli');
-
-/**
- * Add an 'paramGroups' accessor to a command metadata object to sort the
- * params into groups according to the option of the param.
- */
-function addParamGroups(command) {
-  Object.defineProperty(command, 'paramGroups', {
-    get: function() {
-      var paramGroups = {};
-      this.params.forEach(function(param) {
-        var groupName = param.option || l10n.lookup('canonDefaultGroupName');
-        if (paramGroups[groupName] == null) {
-          paramGroups[groupName] = [];
-        }
-        paramGroups[groupName].push(param);
-      });
-      return paramGroups;
-    },
-    enumerable: true
-  });
-}
-
-/**
- * Get a data block for the help_man.html/help_man.txt templates
- */
-function getHelpManData(commandData, context) {
-  // Filter out hidden parameters
-  commandData.command.params = commandData.command.params.filter(
-    param => !param.hidden
-  );
-
-  addParamGroups(commandData.command);
-  commandData.subcommands.forEach(addParamGroups);
-
-  return {
-    l10n: l10n.propertyLookup,
-    onclick: context.update,
-    ondblclick: context.updateExec,
-    describe: function(item) {
-      return item.manual || item.description;
-    },
-    getTypeDescription: function(param) {
-      var input = '';
-      if (param.defaultValue === undefined) {
-        input = l10n.lookup('helpManRequired');
-      }
-      else if (param.defaultValue === null) {
-        input = l10n.lookup('helpManOptional');
-      }
-      else {
-        // We need defaultText to work the text version of defaultValue
-        input = l10n.lookup('helpManOptional');
-        /*
-        var val = param.type.stringify(param.defaultValue);
-        input = Promise.resolve(val).then(function(defaultValue) {
-          return l10n.lookupFormat('helpManDefault', [ defaultValue ]);
-        }.bind(this));
-        */
-      }
-
-      return Promise.resolve(input).then(defaultDescr => {
-        return '(' + (param.type.name || param.type) + ', ' + defaultDescr + ')';
-      });
-    },
-    getSynopsis: function(param) {
-      var name = param.name + (param.short ? '|-' + param.short : '');
-      if (param.option == null) {
-        return param.defaultValue !== undefined ?
-            '[' + name + ']' :
-            '<' + name + '>';
-      }
-      else {
-        return param.type === 'boolean' || param.type.name === 'boolean' ?
-            '[--' + name + ']' :
-            '[--' + name + ' ...]';
-      }
-    },
-    command: commandData.command,
-    subcommands: commandData.subcommands
-  };
-}
-
-/**
- * Get a data block for the help_list.html/help_list.txt templates
- */
-function getHelpListData(commandsData, context) {
-  commandsData.commands.forEach(addParamGroups);
-
-  var heading;
-  if (commandsData.commands.length === 0) {
-    heading = l10n.lookupFormat('helpListNone', [ commandsData.prefix ]);
-  }
-  else if (commandsData.prefix == null) {
-    heading = l10n.lookup('helpListAll');
-  }
-  else {
-    heading = l10n.lookupFormat('helpListPrefix', [ commandsData.prefix ]);
-  }
-
-  return {
-    l10n: l10n.propertyLookup,
-    includeIntro: commandsData.prefix == null,
-    heading: heading,
-    onclick: context.update,
-    ondblclick: context.updateExec,
-    matchingCommands: commandsData.commands
-  };
-}
-
-/**
- * Create a block of data suitable to be passed to the help_list.html template
- */
-function getMatchingCommands(context, prefix) {
-  var commands = cli.getMapping(context).requisition.system.commands;
-  var reply = commands.getAll().filter(function(command) {
-    if (command.hidden) {
-      return false;
-    }
-
-    if (prefix && command.name.indexOf(prefix) !== 0) {
-      // Filtered out because they don't match the search
-      return false;
-    }
-    if (!prefix && command.name.includes(' ')) {
-      // We don't show sub commands with plain 'help'
-      return false;
-    }
-    return true;
-  });
-
-  reply.sort(function(c1, c2) {
-    return c1.name.localeCompare(c2.name);
-  });
-
-  reply = reply.map(function(command) {
-    return command.toJson();
-  });
-
-  return reply;
-}
-
-/**
- * Find all the sub commands of the given command
- */
-function getSubCommands(context, command) {
-  var commands = cli.getMapping(context).requisition.system.commands;
-  var subcommands = commands.getAll().filter(function(subcommand) {
-    return subcommand.name.indexOf(command.name) === 0 &&
-           subcommand.name !== command.name &&
-           !subcommand.hidden;
-  });
-
-  subcommands.sort(function(c1, c2) {
-    return c1.name.localeCompare(c2.name);
-  });
-
-  subcommands = subcommands.map(function(subcommand) {
-    return subcommand.toJson();
-  });
-
-  return subcommands;
-}
-
-var helpCss = '' +
-  '.gcli-help-name {\n' +
-  '  text-align: end;\n' +
-  '}\n' +
-  '\n' +
-  '.gcli-help-arrow {\n' +
-  '  color: #AAA;\n' +
-  '}\n' +
-  '\n' +
-  '.gcli-help-description {\n' +
-  '  margin: 0 20px;\n' +
-  '  padding: 0;\n' +
-  '}\n' +
-  '\n' +
-  '.gcli-help-parameter {\n' +
-  '  margin: 0 30px;\n' +
-  '  padding: 0;\n' +
-  '}\n' +
-  '\n' +
-  '.gcli-help-header {\n' +
-  '  margin: 10px 0 6px;\n' +
-  '}\n';
-
-exports.items = [
-  {
-    // 'help' command
-    item: 'command',
-    name: 'help',
-    runAt: 'client',
-    description: l10n.lookup('helpDesc'),
-    manual: l10n.lookup('helpManual'),
-    params: [
-      {
-        name: 'search',
-        type: 'string',
-        description: l10n.lookup('helpSearchDesc'),
-        manual: l10n.lookup('helpSearchManual3'),
-        defaultValue: null
-      }
-    ],
-
-    exec: function(args, context) {
-      var commands = cli.getMapping(context).requisition.system.commands;
-      var command = commands.get(args.search);
-      if (command) {
-        return context.typedData('commandData', {
-          command: command.toJson(),
-          subcommands: getSubCommands(context, command)
-        });
-      }
-
-      return context.typedData('commandsData', {
-        prefix: args.search,
-        commands: getMatchingCommands(context, args.search)
-      });
-    }
-  },
-  {
-    // Convert a command into an HTML man page
-    item: 'converter',
-    from: 'commandData',
-    to: 'view',
-    exec: function(commandData, context) {
-      return {
-        html:
-          '<div>\n' +
-          '  <p class="gcli-help-header">\n' +
-          '    ${l10n.helpManSynopsis}:\n' +
-          '    <span class="gcli-out-shortcut" data-command="${command.name}"\n' +
-          '        onclick="${onclick}" ondblclick="${ondblclick}">\n' +
-          '      ${command.name}\n' +
-          '      <span foreach="param in ${command.params}">${getSynopsis(param)} </span>\n' +
-          '    </span>\n' +
-          '  </p>\n' +
-          '\n' +
-          '  <p class="gcli-help-description">${describe(command)}</p>\n' +
-          '\n' +
-          '  <div if="${!command.isParent}">\n' +
-          '    <div foreach="groupName in ${command.paramGroups}">\n' +
-          '      <p class="gcli-help-header">${groupName}:</p>\n' +
-          '      <ul class="gcli-help-parameter">\n' +
-          '        <li if="${command.params.length === 0}">${l10n.helpManNone}</li>\n' +
-          '        <li foreach="param in ${command.paramGroups[groupName]}">\n' +
-          '          <code>${getSynopsis(param)}</code> <em>${getTypeDescription(param)}</em>\n' +
-          '          <br/>\n' +
-          '          ${describe(param)}\n' +
-          '        </li>\n' +
-          '      </ul>\n' +
-          '    </div>\n' +
-          '  </div>\n' +
-          '\n' +
-          '  <div if="${command.isParent}">\n' +
-          '    <p class="gcli-help-header">${l10n.subCommands}:</p>\n' +
-          '    <ul class="gcli-help-${subcommands}">\n' +
-          '      <li if="${subcommands.length === 0}">${l10n.subcommandsNone}</li>\n' +
-          '      <li foreach="subcommand in ${subcommands}">\n' +
-          '        ${subcommand.name}: ${subcommand.description}\n' +
-          '        <span class="gcli-out-shortcut" data-command="help ${subcommand.name}"\n' +
-          '            onclick="${onclick}" ondblclick="${ondblclick}">\n' +
-          '          help ${subcommand.name}\n' +
-          '        </span>\n' +
-          '      </li>\n' +
-          '    </ul>\n' +
-          '  </div>\n' +
-          '\n' +
-          '</div>\n',
-        options: { allowEval: true, stack: 'commandData->view' },
-        data: getHelpManData(commandData, context),
-        css: helpCss,
-        cssId: 'gcli-help'
-      };
-    }
-  },
-  {
-    // Convert a command into a string based man page
-    item: 'converter',
-    from: 'commandData',
-    to: 'stringView',
-    exec: function(commandData, context) {
-      return {
-        html:
-          '<div>## ${command.name}\n' +
-          '\n' +
-          '# ${l10n.helpManSynopsis}: ${command.name} <loop foreach="param in ${command.params}">${getSynopsis(param)} </loop>\n' +
-          '\n' +
-          '# ${l10n.helpManDescription}:\n' +
-          '\n' +
-          '${command.manual || command.description}\n' +
-          '\n' +
-          '<loop foreach="groupName in ${command.paramGroups}">\n' +
-          '<span if="${!command.isParent}"># ${groupName}:\n' +
-          '\n' +
-          '<span if="${command.params.length === 0}">${l10n.helpManNone}</span><loop foreach="param in ${command.paramGroups[groupName]}">* ${param.name}: ${getTypeDescription(param)}\n' +
-          '  ${param.manual || param.description}\n' +
-          '</loop>\n' +
-          '</span>\n' +
-          '</loop>\n' +
-          '\n' +
-          '<span if="${command.isParent}"># ${l10n.subCommands}:</span>\n' +
-          '\n' +
-          '<span if="${subcommands.length === 0}">${l10n.subcommandsNone}</span>\n' +
-          '<loop foreach="subcommand in ${subcommands}">* ${subcommand.name}: ${subcommand.description}\n' +
-          '</loop>\n' +
-          '</div>\n',
-        options: { allowEval: true, stack: 'commandData->stringView' },
-        data: getHelpManData(commandData, context)
-      };
-    }
-  },
-  {
-    // Convert a list of commands into a formatted list
-    item: 'converter',
-    from: 'commandsData',
-    to: 'view',
-    exec: function(commandsData, context) {
-      return {
-        html:
-          '<div>\n' +
-          '  <div if="${includeIntro}">\n' +
-          '    <p>${l10n.helpIntro}</p>\n' +
-          '  </div>\n' +
-          '\n' +
-          '  <p>${heading}</p>\n' +
-          '\n' +
-          '  <table>\n' +
-          '    <tr foreach="command in ${matchingCommands}">\n' +
-          '      <td class="gcli-help-name">${command.name}</td>\n' +
-          '      <td class="gcli-help-arrow">-</td>\n' +
-          '      <td>\n' +
-          '        ${command.description}\n' +
-          '        <span class="gcli-out-shortcut"\n' +
-          '            onclick="${onclick}" ondblclick="${ondblclick}"\n' +
-          '            data-command="help ${command.name}">help ${command.name}</span>\n' +
-          '      </td>\n' +
-          '    </tr>\n' +
-          '  </table>\n' +
-          '</div>\n',
-        options: { allowEval: true, stack: 'commandsData->view' },
-        data: getHelpListData(commandsData, context),
-        css: helpCss,
-        cssId: 'gcli-help'
-      };
-    }
-  },
-  {
-    // Convert a list of commands into a formatted list
-    item: 'converter',
-    from: 'commandsData',
-    to: 'stringView',
-    exec: function(commandsData, context) {
-      return {
-        html:
-          '<pre><span if="${includeIntro}">## ${l10n.helpIntro}</span>\n' +
-          '\n' +
-          '# ${heading}\n' +
-          '\n' +
-          '<loop foreach="command in ${matchingCommands}">${command.name} &#x2192; ${command.description}\n' +
-          '</loop></pre>',
-        options: { allowEval: true, stack: 'commandsData->stringView' },
-        data: getHelpListData(commandsData, context)
-      };
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/commands/mocks.js
+++ /dev/null
@@ -1,68 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var cli = require('../cli');
-var mockCommands = require('../test/mockCommands');
-var mockFileCommands = require('../test/mockFileCommands');
-var mockSettings = require('../test/mockSettings');
-
-var isNode = (typeof(process) !== 'undefined' &&
-             process.title.includes('node'));
-
-exports.items = [
-  {
-    item: 'command',
-    name: 'mocks',
-    description: 'Add/remove mock commands',
-    params: [
-      {
-        name: 'included',
-        type: {
-          name: 'selection',
-          data: [ 'on', 'off' ]
-        },
-        description: 'Turn mock commands on or off',
-      }
-    ],
-    returnType: 'string',
-
-    exec: function(args, context) {
-      var requisition = cli.getMapping(context).requisition;
-      this[args.included](requisition);
-      return 'Mock commands are now ' + args.included;
-    },
-
-    on: function(requisition) {
-      mockCommands.setup(requisition);
-      mockSettings.setup(requisition.system);
-
-      if (isNode) {
-        mockFileCommands.setup(requisition);
-      }
-    },
-
-    off: function(requisition) {
-      mockCommands.shutdown(requisition);
-      mockSettings.shutdown(requisition.system);
-
-      if (isNode) {
-        mockFileCommands.shutdown(requisition);
-      }
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/commands/moz.build
+++ /dev/null
@@ -1,16 +0,0 @@
-# -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
-# vim: set filetype=python:
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-DevToolsModules(
-    'clear.js',
-    'commands.js',
-    'context.js',
-    'help.js',
-    'mocks.js',
-    'pref.js',
-    'preflist.js',
-    'test.js',
-)
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/commands/pref.js
+++ /dev/null
@@ -1,93 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var l10n = require('../util/l10n');
-
-exports.items = [
-  {
-    // 'pref' command
-    item: 'command',
-    name: 'pref',
-    description: l10n.lookup('prefDesc'),
-    manual: l10n.lookup('prefManual')
-  },
-  {
-    // 'pref show' command
-    item: 'command',
-    name: 'pref show',
-    runAt: 'client',
-    description: l10n.lookup('prefShowDesc'),
-    manual: l10n.lookup('prefShowManual'),
-    params: [
-      {
-        name: 'setting',
-        type: 'setting',
-        description: l10n.lookup('prefShowSettingDesc'),
-        manual: l10n.lookup('prefShowSettingManual')
-      }
-    ],
-    exec: function(args, context) {
-      return l10n.lookupFormat('prefShowSettingValue',
-                               [ args.setting.name, args.setting.value ]);
-    }
-  },
-  {
-    // 'pref set' command
-    item: 'command',
-    name: 'pref set',
-    runAt: 'client',
-    description: l10n.lookup('prefSetDesc'),
-    manual: l10n.lookup('prefSetManual'),
-    params: [
-      {
-        name: 'setting',
-        type: 'setting',
-        description: l10n.lookup('prefSetSettingDesc'),
-        manual: l10n.lookup('prefSetSettingManual')
-      },
-      {
-        name: 'value',
-        type: 'settingValue',
-        description: l10n.lookup('prefSetValueDesc'),
-        manual: l10n.lookup('prefSetValueManual')
-      }
-    ],
-    exec: function(args, context) {
-      args.setting.value = args.value;
-    }
-  },
-  {
-    // 'pref reset' command
-    item: 'command',
-    name: 'pref reset',
-    runAt: 'client',
-    description: l10n.lookup('prefResetDesc'),
-    manual: l10n.lookup('prefResetManual'),
-    params: [
-      {
-        name: 'setting',
-        type: 'setting',
-        description: l10n.lookup('prefResetSettingDesc'),
-        manual: l10n.lookup('prefResetSettingManual')
-      }
-    ],
-    exec: function(args, context) {
-      args.setting.setDefault();
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/commands/preflist.js
+++ /dev/null
@@ -1,214 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var l10n = require('../util/l10n');
-
-/**
- * Format a list of settings for display
- */
-var prefsViewConverter = {
-  item: 'converter',
-  from: 'prefsData',
-  to: 'view',
-  exec: function(prefsData, conversionContext) {
-    var prefList = new PrefList(prefsData, conversionContext);
-    return {
-      html:
-        '<div ignore="${onLoad(__element)}">\n' +
-        '  <!-- This is broken, and unimportant. Comment out for now\n' +
-        '  <div class="gcli-pref-list-filter">\n' +
-        '    ${l10n.prefOutputFilter}:\n' +
-        '    <input onKeyUp="${onFilterChange}" value="${search}"/>\n' +
-        '  </div>\n' +
-        '  -->\n' +
-        '  <table class="gcli-pref-list-table">\n' +
-        '    <colgroup>\n' +
-        '      <col class="gcli-pref-list-name"/>\n' +
-        '      <col class="gcli-pref-list-value"/>\n' +
-        '    </colgroup>\n' +
-        '    <tr>\n' +
-        '      <th>${l10n.prefOutputName}</th>\n' +
-        '      <th>${l10n.prefOutputValue}</th>\n' +
-        '    </tr>\n' +
-        '  </table>\n' +
-        '  <div class="gcli-pref-list-scroller">\n' +
-        '    <table class="gcli-pref-list-table" save="${table}">\n' +
-        '    </table>\n' +
-        '  </div>\n' +
-        '</div>\n',
-      data: prefList,
-      options: {
-        blankNullUndefined: true,
-        allowEval: true,
-        stack: 'prefsData->view'
-      },
-      css:
-        '.gcli-pref-list-scroller {\n' +
-        '  max-height: 200px;\n' +
-        '  overflow-y: auto;\n' +
-        '  overflow-x: hidden;\n' +
-        '  display: inline-block;\n' +
-        '}\n' +
-        '\n' +
-        '.gcli-pref-list-table {\n' +
-        '  width: 500px;\n' +
-        '  table-layout: fixed;\n' +
-        '}\n' +
-        '\n' +
-        '.gcli-pref-list-table tr > th {\n' +
-        '  text-align: left;\n' +
-        '}\n' +
-        '\n' +
-        '.gcli-pref-list-table tr > td {\n' +
-        '  text-overflow: elipsis;\n' +
-        '  word-wrap: break-word;\n' +
-        '}\n' +
-        '\n' +
-        '.gcli-pref-list-name {\n' +
-        '  width: 70%;\n' +
-        '}\n' +
-        '\n' +
-        '.gcli-pref-list-command {\n' +
-        '  display: none;\n' +
-        '}\n' +
-        '\n' +
-        '.gcli-pref-list-row:hover .gcli-pref-list-command {\n' +
-        '  /* \'pref list\' is a bit broken and unimportant. Band-aid follows */\n' +
-        '  /* display: inline-block; */\n' +
-        '}\n',
-      cssId: 'gcli-pref-list'
-    };
-  }
-};
-
-/**
- * Format a list of settings for display
- */
-var prefsStringConverter = {
-  item: 'converter',
-  from: 'prefsData',
-  to: 'string',
-  exec: function(prefsData, conversionContext) {
-    var reply = '';
-    prefsData.settings.forEach(function(setting) {
-      reply += setting.name + ' -> ' + setting.value + '\n';
-    });
-    return reply;
-  }
-};
-
-/**
- * 'pref list' command
- */
-var prefList = {
-  item: 'command',
-  name: 'pref list',
-  description: l10n.lookup('prefListDesc'),
-  manual: l10n.lookup('prefListManual'),
-  params: [
-    {
-      name: 'search',
-      type: 'string',
-      defaultValue: null,
-      description: l10n.lookup('prefListSearchDesc'),
-      manual: l10n.lookup('prefListSearchManual')
-    }
-  ],
-  returnType: 'prefsData',
-  exec: function(args, context) {
-    return new Promise(function(resolve, reject) {
-      // This can be slow, get out of the way of the main thread
-      setTimeout(() => {
-        var prefsData = {
-          settings: context.system.settings.getAll(args.search),
-          search: args.search
-        };
-        resolve(prefsData);
-      }, 10);
-    });
-  }
-};
-
-/**
- * A manager for our version of about:config
- */
-function PrefList(prefsData, conversionContext) {
-  this.search = prefsData.search;
-  this.settings = prefsData.settings;
-  this.conversionContext = conversionContext;
-
-  this.onLoad = this.onLoad.bind(this);
-}
-
-/**
- * A load event handler registered by the template engine so we can load the
- * inner document
- */
-PrefList.prototype.onLoad = function(element) {
-  var table = element.querySelector('.gcli-pref-list-table');
-  this.updateTable(table);
-  return '';
-};
-
-/**
- * Forward localization lookups
- */
-PrefList.prototype.l10n = l10n.propertyLookup;
-
-/**
- * Called from the template onkeyup for the filter element
- */
-PrefList.prototype.updateTable = function(table) {
-  var view = this.conversionContext.createView({
-    html:
-      '<table>\n' +
-      '  <colgroup>\n' +
-      '    <col class="gcli-pref-list-name"/>\n' +
-      '    <col class="gcli-pref-list-value"/>\n' +
-      '  </colgroup>\n' +
-      '  <tr class="gcli-pref-list-row" foreach="setting in ${settings}">\n' +
-      '    <td>${setting.name}</td>\n' +
-      '    <td onclick="${onSetClick}" data-command="pref set ${setting.name} ">\n' +
-      '      ${setting.value}\n' +
-      '      [Edit]\n' +
-      '    </td>\n' +
-      '  </tr>\n' +
-      '</table>\n',
-    options: { blankNullUndefined: true, stack: 'prefsData#inner' },
-    data: this
-  });
-
-  view.appendTo(table, true);
-};
-
-PrefList.prototype.onFilterChange = function(ev) {
-  if (ev.target.value !== this.search) {
-    this.search = ev.target.value;
-
-    var root = ev.target.parentNode.parentNode;
-    var table = root.querySelector('.gcli-pref-list-table');
-    this.updateTable(table);
-  }
-};
-
-PrefList.prototype.onSetClick = function(ev) {
-  var typed = ev.currentTarget.getAttribute('data-command');
-  this.conversionContext.update(typed);
-};
-
-exports.items = [ prefsViewConverter, prefsStringConverter, prefList ];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/commands/test.js
+++ /dev/null
@@ -1,215 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var examiner = require('../testharness/examiner');
-var stati = require('../testharness/status').stati;
-var helpers = require('../test/helpers');
-var suite = require('../test/suite');
-var cli = require('../cli');
-var Requisition = require('../cli').Requisition;
-var createRequisitionAutomator = require('../test/automators/requisition').createRequisitionAutomator;
-
-var isNode = (typeof(process) !== 'undefined' &&
-             process.title.includes('node'));
-
-suite.init(isNode);
-
-exports.optionsContainer = [];
-
-exports.items = [
-  {
-    item: 'type',
-    name: 'suite',
-    parent: 'selection',
-    cacheable: true,
-    lookup: function() {
-      return Object.keys(examiner.suites).map(function(name) {
-        return { name: name, value: examiner.suites[name] };
-      });
-    }
-  },
-  {
-    item: 'command',
-    name: 'test',
-    description: 'Run GCLI unit tests',
-    params: [
-      {
-        name: 'suite',
-        type: 'suite',
-        description: 'Test suite to run.',
-        defaultValue: examiner
-      },
-      {
-        name: 'usehost',
-        type: 'boolean',
-        description: 'Run the unit tests in the host window',
-        option: true
-      }
-    ],
-    returnType: 'examiner-output',
-    noRemote: true,
-    exec: function(args, context) {
-      if (args.usehost && exports.optionsContainer.length === 0) {
-        throw new Error('Can\'t use --usehost without injected options');
-      }
-
-      var options;
-      if (args.usehost) {
-        options = exports.optionsContainer[0];
-      }
-      else {
-        var env = {
-          document: document,
-          window: window
-        };
-        options = {
-          isNode: isNode,
-          isFirefox: false,
-          isPhantomjs: false,
-          requisition: new Requisition(context.system, { environment: env })
-        };
-        options.automator = createRequisitionAutomator(options.requisition);
-      }
-
-      var requisition = options.requisition;
-      requisition.system.commands.get('mocks').on(requisition);
-      helpers.resetResponseTimes();
-      examiner.reset();
-
-      return args.suite.run(options).then(function() {
-        requisition.system.commands.get('mocks').off(requisition);
-        var output = context.typedData('examiner-output', examiner.toRemote());
-
-        if (output.data.summary.status === stati.pass) {
-          return output;
-        }
-        else {
-          cli.logErrors = false;
-          throw output;
-        }
-      });
-    }
-  },
-  {
-    item: 'converter',
-    from: 'examiner-output',
-    to: 'string',
-    exec: function(output, conversionContext) {
-      return '\n' + examiner.detailedResultLog('NodeJS/NoDom') +
-             '\n' + helpers.timingSummary;
-    }
-  },
-  {
-    item: 'converter',
-    from: 'examiner-output',
-    to: 'view',
-    exec: function(output, conversionContext) {
-      return {
-        html:
-          '<div>\n' +
-          '  <table class="gcliTestResults">\n' +
-          '    <thead>\n' +
-          '      <tr>\n' +
-          '        <th class="gcliTestSuite">Suite</th>\n' +
-          '        <th>Test</th>\n' +
-          '        <th>Results</th>\n' +
-          '        <th>Checks</th>\n' +
-          '        <th>Notes</th>\n' +
-          '      </tr>\n' +
-          '    </thead>\n' +
-          '    <tbody foreach="suite in ${suites}">\n' +
-          '      <tr foreach="test in ${suite.tests}" title="${suite.name}.${test.name}()">\n' +
-          '        <td class="gcliTestSuite">${suite.name}</td>\n' +
-          '        <td class="gcliTestTitle">${test.title}</td>\n' +
-          '        <td class="gcliTest${test.status.name}">${test.status.name}</td>\n' +
-          '        <td class="gcliTestChecks">${test.checks}</td>\n' +
-          '        <td class="gcliTestMessages">\n' +
-          '          <div foreach="failure in ${test.failures}">\n' +
-          '            ${failure.message}\n' +
-          '            <ul if="${failure.params}">\n' +
-          '              <li>P1: ${failure.p1}</li>\n' +
-          '              <li>P2: ${failure.p2}</li>\n' +
-          '            </ul>\n' +
-          '          </div>\n' +
-          '        </td>\n' +
-          '      </tr>\n' +
-          '    </tbody>\n' +
-          '    <tfoot>\n' +
-          '      <tr>\n' +
-          '        <th></th>\n' +
-          '        <th>Total</th>\n' +
-          '        <th>${summary.status.name}</th>\n' +
-          '        <th class="gcliTestChecks">${summary.checks}</th>\n' +
-          '        <th></th>\n' +
-          '      </tr>\n' +
-          '    </tfoot>\n' +
-          '  </table>\n' +
-          '</div>',
-        css:
-          '.gcliTestSkipped {\n' +
-          '  background-color: #EEE;\n' +
-          '  color: #000;\n' +
-          '}\n' +
-          '\n' +
-          '.gcliTestExecuting {\n' +
-          '  background-color: #888;\n' +
-          '  color: #FFF;\n' +
-          '}\n' +
-          '\n' +
-          '.gcliTestWaiting {\n' +
-          '  background-color: #FFA;\n' +
-          '  color: #000;\n' +
-          '}\n' +
-          '\n' +
-          '.gcliTestPass {\n' +
-          '  background-color: #8F8;\n' +
-          '  color: #000;\n' +
-          '}\n' +
-          '\n' +
-          '.gcliTestFail {\n' +
-          '  background-color: #F00;\n' +
-          '  color: #FFF;\n' +
-          '}\n' +
-          '\n' +
-          'td.gcliTestSuite {\n' +
-          '  font-family: monospace;\n' +
-          '  font-size: 90%;\n' +
-          '  text-align: right;\n' +
-          '}\n' +
-          '\n' +
-          '.gcliTestResults th.gcliTestSuite,\n' +
-          '.gcliTestResults .gcliTestChecks {\n' +
-          '  text-align: right;\n' +
-          '}\n' +
-          '\n' +
-          '.gcliTestResults th {\n' +
-          '  text-align: left;\n' +
-          '}\n' +
-          '\n' +
-          '.gcliTestMessages ul {\n' +
-          '  margin: 0 0 10px;\n' +
-          '  padding-left: 20px;\n' +
-          '  list-style-type: square;\n' +
-          '}\n',
-        cssId: 'gcli-test',
-        data: output,
-        options: { allowEval: true, stack: 'test.html' }
-      };
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/connectors/connectors.js
+++ /dev/null
@@ -1,157 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-/**
- * This is how to implement a connector
- *  var baseConnector = {
- *    item: 'connector',
- *    name: 'foo',
- *
- *    connect: function(url) {
- *      return Promise.resolve(new FooConnection(url));
- *    }
- *  };
- */
-
-/**
- * A prototype base for Connectors
- */
-function Connection() {
-}
-
-/**
- * Add an event listener
- */
-Connection.prototype.on = function(event, action) {
-  if (!this._listeners) {
-    this._listeners = {};
-  }
-  if (!this._listeners[event]) {
-    this._listeners[event] = [];
-  }
-  this._listeners[event].push(action);
-};
-
-/**
- * Remove an event listener
- */
-Connection.prototype.off = function(event, action) {
-  if (!this._listeners) {
-    return;
-  }
-  var actions = this._listeners[event];
-  if (actions) {
-    this._listeners[event] = actions.filter(li => {
-      return li !== action;
-    });
-  }
-};
-
-/**
- * Emit an event. For internal use only
- */
-Connection.prototype._emit = function(event, data) {
-  if (this._listeners == null || this._listeners[event] == null) {
-    return;
-  }
-
-  var listeners = this._listeners[event];
-  listeners.forEach(listener => {
-    // Fail fast if we mutate the list of listeners while emitting
-    if (listeners !== this._listeners[event]) {
-      throw new Error('Listener list changed while emitting');
-    }
-
-    try {
-      listener.call(null, data);
-    }
-    catch (ex) {
-      console.log('Error calling listeners to ' + event);
-      console.error(ex);
-    }
-  });
-};
-
-/**
- * Send a message to the other side of the connection
- */
-Connection.prototype.call = function(feature, data) {
-  throw new Error('Not implemented');
-};
-
-/**
- * Disconnecting a Connection destroys the resources it holds. There is no
- * common route back to being connected once this has been called
- */
-Connection.prototype.disconnect = function() {
-  return Promise.resolve();
-};
-
-exports.Connection = Connection;
-
-/**
- * A manager for the registered Connectors
- */
-function Connectors() {
-  // This is where we cache the connectors that we know about
-  this._registered = {};
-}
-
-/**
- * Add a new connector to the cache
- */
-Connectors.prototype.add = function(connector) {
-  this._registered[connector.name] = connector;
-};
-
-/**
- * Remove an existing connector from the cache
- */
-Connectors.prototype.remove = function(connector) {
-  var name = typeof connector === 'string' ? connector : connector.name;
-  delete this._registered[name];
-};
-
-/**
- * Get access to the list of known connectors
- */
-Connectors.prototype.getAll = function() {
-  return Object.keys(this._registered).map(name => {
-    return this._registered[name];
-  });
-};
-
-var defaultConnectorName;
-
-/**
- * Get access to a connector by name. If name is undefined then first try to
- * use the same connector that we used last time, and if there was no last
- * time, then just use the first registered connector as a default.
- */
-Connectors.prototype.get = function(name) {
-  if (name == null) {
-    name = (defaultConnectorName == null) ?
-        Object.keys(this._registered)[0] :
-        defaultConnectorName;
-  }
-
-  defaultConnectorName = name;
-  return this._registered[name];
-};
-
-exports.Connectors = Connectors;
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/connectors/moz.build
+++ /dev/null
@@ -1,9 +0,0 @@
-# -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
-# vim: set filetype=python:
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-DevToolsModules(
-    'connectors.js',
-)
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/converters/basic.js
+++ /dev/null
@@ -1,94 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var util = require('../util/util');
-
-/**
- * Several converters are just data.toString inside a 'p' element
- */
-function nodeFromDataToString(data, conversionContext) {
-  var node = util.createElement(conversionContext.document, 'p');
-  node.textContent = data.toString();
-  return node;
-}
-
-exports.items = [
-  {
-    item: 'converter',
-    from: 'string',
-    to: 'dom',
-    exec: nodeFromDataToString
-  },
-  {
-    item: 'converter',
-    from: 'number',
-    to: 'dom',
-    exec: nodeFromDataToString
-  },
-  {
-    item: 'converter',
-    from: 'boolean',
-    to: 'dom',
-    exec: nodeFromDataToString
-  },
-  {
-    item: 'converter',
-    from: 'undefined',
-    to: 'dom',
-    exec: function(data, conversionContext) {
-      return util.createElement(conversionContext.document, 'span');
-    }
-  },
-  {
-    item: 'converter',
-    from: 'json',
-    to: 'view',
-    exec: function(json, context) {
-      var html = JSON.stringify(json, null, '&#160;').replace(/\n/g, '<br/>');
-      return {
-        html: '<pre>' + html + '</pre>'
-      };
-    }
-  },
-  {
-    item: 'converter',
-    from: 'number',
-    to: 'string',
-    exec: function(data) { return '' + data; }
-  },
-  {
-    item: 'converter',
-    from: 'boolean',
-    to: 'string',
-    exec: function(data) { return '' + data; }
-  },
-  {
-    item: 'converter',
-    from: 'undefined',
-    to: 'string',
-    exec: function(data) { return ''; }
-  },
-  {
-    item: 'converter',
-    from: 'json',
-    to: 'string',
-    exec: function(json, conversionContext) {
-      return JSON.stringify(json, null, '  ');
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/converters/converters.js
+++ /dev/null
@@ -1,280 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var util = require('../util/util');
-var host = require('../util/host');
-
-// It's probably easiest to read this bottom to top
-
-/**
- * Best guess at creating a DOM element from random data
- */
-var fallbackDomConverter = {
-  from: '*',
-  to: 'dom',
-  exec: function(data, conversionContext) {
-    return conversionContext.document.createTextNode(data || '');
-  }
-};
-
-/**
- * Best guess at creating a string from random data
- */
-var fallbackStringConverter = {
-  from: '*',
-  to: 'string',
-  exec: function(data, conversionContext) {
-    return data == null ? '' : data.toString();
-  }
-};
-
-/**
- * Convert a view object to a DOM element
- */
-var viewDomConverter = {
-  item: 'converter',
-  from: 'view',
-  to: 'dom',
-  exec: function(view, conversionContext) {
-    if (!view.isView) {
-      view = conversionContext.createView(view);
-    }
-    return view.toDom(conversionContext.document);
-  }
-};
-
-/**
- * Convert a view object to a string
- */
-var viewStringConverter = {
-  item: 'converter',
-  from: 'view',
-  to: 'string',
-  exec: function(view, conversionContext) {
-    if (!view.isView) {
-      view = conversionContext.createView(view);
-    }
-    return view.toDom(conversionContext.document).textContent;
-  }
-};
-
-/**
- * Convert a view object to a string
- */
-var stringViewStringConverter = {
-  item: 'converter',
-  from: 'stringView',
-  to: 'string',
-  exec: function(view, conversionContext) {
-    if (!view.isView) {
-      view = conversionContext.createView(view);
-    }
-    return view.toDom(conversionContext.document).textContent;
-  }
-};
-
-/**
- * Convert an exception to a DOM element
- */
-var errorDomConverter = {
-  item: 'converter',
-  from: 'error',
-  to: 'dom',
-  exec: function(ex, conversionContext) {
-    var node = util.createElement(conversionContext.document, 'p');
-    node.className = 'gcli-error';
-    node.textContent = errorStringConverter.exec(ex, conversionContext);
-    return node;
-  }
-};
-
-/**
- * Convert an exception to a string
- */
-var errorStringConverter = {
-  item: 'converter',
-  from: 'error',
-  to: 'string',
-  exec: function(ex, conversionContext) {
-    if (typeof ex === 'string') {
-      return ex;
-    }
-    if (ex instanceof Error) {
-      return '' + ex;
-    }
-    if (typeof ex.message === 'string') {
-      return ex.message;
-    }
-    return '' + ex;
-  }
-};
-
-/**
- * Create a new converter by using 2 converters, one after the other
- */
-function getChainConverter(first, second) {
-  if (first.to !== second.from) {
-    throw new Error('Chain convert impossible: ' + first.to + '!=' + second.from);
-  }
-  return {
-    from: first.from,
-    to: second.to,
-    exec: function(data, conversionContext) {
-      var intermediate = first.exec(data, conversionContext);
-      return second.exec(intermediate, conversionContext);
-    }
-  };
-}
-
-/**
- * A manager for the registered Converters
- */
-function Converters() {
-  // This is where we cache the converters that we know about
-  this._registered = {
-    from: {}
-  };
-}
-
-/**
- * Add a new converter to the cache
- */
-Converters.prototype.add = function(converter) {
-  var fromMatch = this._registered.from[converter.from];
-  if (fromMatch == null) {
-    fromMatch = {};
-    this._registered.from[converter.from] = fromMatch;
-  }
-
-  fromMatch[converter.to] = converter;
-};
-
-/**
- * Remove an existing converter from the cache
- */
-Converters.prototype.remove = function(converter) {
-  var fromMatch = this._registered.from[converter.from];
-  if (fromMatch == null) {
-    return;
-  }
-
-  if (fromMatch[converter.to] === converter) {
-    fromMatch[converter.to] = null;
-  }
-};
-
-/**
- * Work out the best converter that we've got, for a given conversion.
- */
-Converters.prototype.get = function(from, to) {
-  var fromMatch = this._registered.from[from];
-  if (fromMatch == null) {
-    return this._getFallbackConverter(from, to);
-  }
-
-  var converter = fromMatch[to];
-  if (converter == null) {
-    // Someone is going to love writing a graph search algorithm to work out
-    // the smallest number of conversions, or perhaps the least 'lossy'
-    // conversion but for now the only 2 step conversions which we are going to
-    // special case are foo->view->dom and foo->stringView->string.
-    if (to === 'dom') {
-      converter = fromMatch.view;
-      if (converter != null) {
-        return getChainConverter(converter, viewDomConverter);
-      }
-    }
-
-    if (to === 'string') {
-      converter = fromMatch.stringView;
-      if (converter != null) {
-        return getChainConverter(converter, stringViewStringConverter);
-      }
-      converter = fromMatch.view;
-      if (converter != null) {
-        return getChainConverter(converter, viewStringConverter);
-      }
-    }
-
-    return this._getFallbackConverter(from, to);
-  }
-  return converter;
-};
-
-/**
- * Get all the registered converters. Most for debugging
- */
-Converters.prototype.getAll = function() {
-  return Object.keys(this._registered.from).map(name => {
-    return this._registered.from[name];
-  });
-};
-
-/**
- * Helper for get to pick the best fallback converter
- */
-Converters.prototype._getFallbackConverter = function(from, to) {
-  console.error('No converter from ' + from + ' to ' + to + '. Using fallback');
-
-  if (to === 'dom') {
-    return fallbackDomConverter;
-  }
-
-  if (to === 'string') {
-    return fallbackStringConverter;
-  }
-
-  throw new Error('No conversion possible from ' + from + ' to ' + to + '.');
-};
-
-/**
- * Convert some data from one type to another
- * @param data The object to convert
- * @param from The type of the data right now
- * @param to The type that we would like the data in
- * @param conversionContext An execution context (i.e. simplified requisition)
- * which is often required for access to a document, or createView function
- */
-Converters.prototype.convert = function(data, from, to, conversionContext) {
-  try {
-    if (from === to) {
-      return Promise.resolve(data);
-    }
-
-    var converter = this.get(from, to);
-    return host.exec(() => {
-      return converter.exec(data, conversionContext);
-    });
-  }
-  catch (ex) {
-    var converter = this.get('error', to);
-    return host.exec(() => {
-      return converter.exec(ex, conversionContext);
-    });
-  }
-};
-
-exports.Converters = Converters;
-
-/**
- * Items for export
- */
-exports.items = [
-  viewDomConverter, viewStringConverter, stringViewStringConverter,
-  errorDomConverter, errorStringConverter
-];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/converters/html.js
+++ /dev/null
@@ -1,47 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var util = require('../util/util');
-
-/**
- * 'html' means a string containing HTML markup. We use innerHTML to inject
- * this into a DOM which has security implications, so this module will not
- * be used in all implementations.
- */
-exports.items = [
-  {
-    item: 'converter',
-    from: 'html',
-    to: 'dom',
-    exec: function(html, conversionContext) {
-      var div = util.createElement(conversionContext.document, 'div');
-      div.innerHTML = html;
-      return div;
-    }
-  },
-  {
-    item: 'converter',
-    from: 'html',
-    to: 'string',
-    exec: function(html, conversionContext) {
-      var div = util.createElement(conversionContext.document, 'div');
-      div.innerHTML = html;
-      return div.textContent;
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/converters/moz.build
+++ /dev/null
@@ -1,12 +0,0 @@
-# -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
-# vim: set filetype=python:
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-DevToolsModules(
-    'basic.js',
-    'converters.js',
-    'html.js',
-    'terminal.js',
-)
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/converters/terminal.js
+++ /dev/null
@@ -1,56 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var util = require('../util/util');
-
-/**
- * A 'terminal' object is a string or an array of strings, which are typically
- * the output from a shell command
- */
-exports.items = [
-  {
-    item: 'converter',
-    from: 'terminal',
-    to: 'dom',
-    createTextArea: function(text, conversionContext) {
-      var node = util.createElement(conversionContext.document, 'textarea');
-      node.classList.add('gcli-row-subterminal');
-      node.readOnly = true;
-      node.textContent = text;
-      return node;
-    },
-    exec: function(data, conversionContext) {
-      if (Array.isArray(data)) {
-        var node = util.createElement(conversionContext.document, 'div');
-        data.forEach(function(member) {
-          node.appendChild(this.createTextArea(member, conversionContext));
-        });
-        return node;
-      }
-      return this.createTextArea(data);
-    }
-  },
-  {
-    item: 'converter',
-    from: 'terminal',
-    to: 'string',
-    exec: function(data, conversionContext) {
-      return Array.isArray(data) ? data.join('') : '' + data;
-    }
-  }
-];
deleted file mode 100644
--- a/devtools/shared/gcli/source/lib/gcli/fields/delegate.js
+++ /dev/null
@@ -1,96 +0,0 @@
-/*
- * Copyright 2012, Mozilla Foundation and contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-'use strict';
-
-var util = require('../util/util');
-var Field = require('./fields').Field;
-
-/**
- * A field that works with delegate types by delaying resolution until that
- * last possible time
- */
-function DelegateField(type, options) {
-  Field.call(this, type, options);
-  this.options = options;
-
-  this.element = util.createElement(this.document, 'div');
-  this.update();
-
-  this.onFieldChange = util.createEvent('DelegateField.onFieldChange');
-}
-
-DelegateField.prototype = Object.create(Field.prototype);