Bug 1550651 - Remove AboutDebuggingNew component registration r=daisuke
authorJulian Descottes <jdescottes@mozilla.com>
Tue, 28 May 2019 01:20:08 +0000
changeset 476326 8c0b0448646b22219fb13d460b7eba9e63be0b83
parent 476325 08cc281fa75920202790232d39cf452b1893e209
child 476327 0d2958471c0885b7acdfbeba557b109f1b403da4
push id36092
push userarchaeopteryx@coole-files.de
push dateFri, 31 May 2019 17:03:46 +0000
treeherdermozilla-central@8384972e1f6a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdaisuke
bugs1550651
milestone69.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 1550651 - Remove AboutDebuggingNew component registration r=daisuke The component is no longer needed since we enabled the new aboutdebugging by default on Nightly Differential Revision: https://phabricator.services.mozilla.com/D32756
devtools/startup/AboutDebuggingNewRegistration.jsm
devtools/startup/components.conf
devtools/startup/moz.build
deleted file mode 100644
--- a/devtools/startup/AboutDebuggingNewRegistration.jsm
+++ /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";
-
-// This component is only registered and packaged for local and nightly builds in order to
-// open the new about:debugging when going to about:debugging-new, without having to flip
-// the preference. This allows running both versions of about:debugging side by side to
-// compare them.
-const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
-
-const { nsIAboutModule } = Ci;
-
-function AboutDebuggingNew() {}
-
-AboutDebuggingNew.prototype = {
-  classDescription: "about:debugging-new",
-  classID: Components.ID("610e4e26-26bd-4a7d-aebc-69211d5a3be0"),
-  contractID: "@mozilla.org/network/protocol/about;1?what=debugging-new",
-
-  QueryInterface: ChromeUtils.generateQI([nsIAboutModule]),
-
-  newChannel: function(_, loadInfo) {
-    const uri = "chrome://devtools/content/aboutdebugging-new/index.html";
-
-    const chan = Services.io.newChannelFromURIWithLoadInfo(
-      Services.io.newURI(uri),
-      loadInfo
-    );
-    chan.owner = Services.scriptSecurityManager.getSystemPrincipal();
-    return chan;
-  },
-
-  getURIFlags: function(uri) {
-    return nsIAboutModule.ALLOW_SCRIPT;
-  },
-};
-
-var EXPORTED_SYMBOLS = ["AboutDebuggingNew"];
--- a/devtools/startup/components.conf
+++ b/devtools/startup/components.conf
@@ -23,18 +23,8 @@ if buildconfig.substs['MOZ_DEVTOOLS'] ==
         },
         {
             'cid': '{11342911-3135-45a8-8d71-737a2b0ad469}',
             'contract_ids': ['@mozilla.org/network/protocol/about;1?what=devtools-toolbox'],
             'jsm': 'resource:///modules/AboutDevToolsToolboxRegistration.jsm',
             'constructor': 'AboutDevtoolsToolbox',
         },
     ]
-
-if defined('NIGHTLY_BUILD'):
-    Classes += [
-        {
-            'cid': '{610e4e26-26bd-4a7d-aebc-69211d5a3be0}',
-            'contract_ids': ['@mozilla.org/network/protocol/about;1?what=debugging-new'],
-            'jsm': 'resource:///modules/AboutDebuggingNewRegistration.jsm',
-            'constructor': 'AboutDebuggingNew',
-        },
-    ]
--- a/devtools/startup/moz.build
+++ b/devtools/startup/moz.build
@@ -18,21 +18,16 @@ if CONFIG['MOZ_DEVTOOLS'] == 'all':
         'DevToolsStartup.jsm',
     ]
 
     DIRS += [
       'aboutdevtools',
       'locales',
     ]
 
-if CONFIG['NIGHTLY_BUILD']:
-    EXTRA_JS_MODULES += [
-        'AboutDebuggingNewRegistration.jsm',
-    ]
-
 XPCOM_MANIFESTS += [
     'components.conf',
 ]
 
 XPCSHELL_TESTS_MANIFESTS += ['tests/unit/xpcshell.ini']
 
 if CONFIG['MOZ_BUILD_APP'] != 'mobile/android':
     BROWSER_CHROME_MANIFESTS += ['tests/browser/browser.ini']