Bug 1317867 - Fixed window leak in browser_net_footer-summary.js mochitest r?Honza draft
authorJarda Snajdr <jsnajdr@gmail.com>
Fri, 18 Nov 2016 11:15:14 +0100
changeset 441070 a0638002a5e0aba7af2ff721b8263a46a45b1491
parent 441054 28e2a6dde76ab6ad4464a3662df1bd57af04398a
child 537486 db22ef7b770d9ed861b4f89cae7089533faf03ae
push id36350
push userbmo:jsnajdr@gmail.com
push dateFri, 18 Nov 2016 10:16:00 +0000
reviewersHonza
bugs1317867
milestone53.0a1
Bug 1317867 - Fixed window leak in browser_net_footer-summary.js mochitest r?Honza MozReview-Commit-ID: IwBPI0CAjXG
devtools/client/netmonitor/test/browser_net_footer-summary.js
--- a/devtools/client/netmonitor/test/browser_net_footer-summary.js
+++ b/devtools/client/netmonitor/test/browser_net_footer-summary.js
@@ -5,26 +5,28 @@
 
 /**
  * Test if the summary text displayed in the network requests menu footer
  * is correct.
  */
 
 add_task(function* () {
   requestLongerTimeout(2);
-  let { getSummary } = require("devtools/client/netmonitor/selectors/index");
-  let { L10N } = require("devtools/client/netmonitor/l10n");
-  let { PluralForm } = require("devtools/shared/plural-form");
 
   let { tab, monitor } = yield initNetMonitor(FILTERING_URL);
   info("Starting test... ");
 
   let { $, NetMonitorView, gStore } = monitor.panelWin;
   let { RequestsMenu } = NetMonitorView;
 
+  let winRequire = monitor.panelWin.require;
+  let { getSummary } = winRequire("devtools/client/netmonitor/selectors/index");
+  let { L10N } = winRequire("devtools/client/netmonitor/l10n");
+  let { PluralForm } = winRequire("devtools/shared/plural-form");
+
   RequestsMenu.lazyUpdate = false;
   testStatus();
 
   for (let i = 0; i < 2; i++) {
     info(`Performing requests in batch #${i}`);
     let wait = waitForNetworkEvents(monitor, 8);
     yield ContentTask.spawn(tab.linkedBrowser, {}, function* () {
       content.wrappedJSObject.performRequests('{ "getMedia": true, "getFlash": true }');