Bug 1518581 - Remove unused parameters from openAboutDebugging;r=daisuke a=lizzard
authorJulian Descottes <jdescottes@mozilla.com>
Thu, 14 Feb 2019 08:27:23 +0000
changeset 516183 cf46b5a6e3ed1f8db130127dca8d14b9fddb7cfa
parent 516182 7fc7867411b58196980f5f724f086b6250643750
child 516184 571ad2687c2ea8b846d9468e493fae909eb35e78
push id1953
push userffxbld-merge
push dateMon, 11 Mar 2019 12:10:20 +0000
treeherdermozilla-release@9c35dcbaa899 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdaisuke, lizzard
bugs1518581
milestone66.0
Bug 1518581 - Remove unused parameters from openAboutDebugging;r=daisuke a=lizzard Depends on D18235 The two parameters in the openAboutDebugging are a leftover from the original helper of the old about:debugging. Differential Revision: https://phabricator.services.mozilla.com/D18236
devtools/client/aboutdebugging-new/test/browser/head.js
--- a/devtools/client/aboutdebugging-new/test/browser/head.js
+++ b/devtools/client/aboutdebugging-new/test/browser/head.js
@@ -39,22 +39,22 @@ registerCleanupFunction(async function()
 /**
  * Enable the new about:debugging panel.
  */
 async function enableNewAboutDebugging() {
   await pushPref("devtools.aboutdebugging.new-enabled", true);
   await pushPref("devtools.aboutdebugging.network", true);
 }
 
-async function openAboutDebugging(page, win) {
+async function openAboutDebugging() {
   await enableNewAboutDebugging();
 
   info("opening about:debugging");
 
-  const tab = await addTab("about:debugging", { window: win });
+  const tab = await addTab("about:debugging");
   const browser = tab.linkedBrowser;
   const document = browser.contentDocument;
   const window = browser.contentWindow;
   info("wait for the initial about:debugging requests to be successful");
   await waitForRequestsSuccess(window);
 
   return { tab, document, window };
 }