Bug 1386550 - remove usage of sdk/system/events in DevTools test;r=sole draft
authorJulian Descottes <jdescottes@mozilla.com>
Wed, 02 Aug 2017 12:01:24 +0200
changeset 619639 6ed2763285c1a32c60771d473c4a0613b542c142
parent 619581 52285ea5e54c73d3ed824544cef2ee3f195f05e6
child 640468 19be3296afb03c2daba87b259204efd10ac60c12
push id71756
push userjdescottes@mozilla.com
push dateWed, 02 Aug 2017 10:42:49 +0000
reviewerssole
bugs1386550
milestone57.0a1
Bug 1386550 - remove usage of sdk/system/events in DevTools test;r=sole MozReview-Commit-ID: FM1AjShYmkj
devtools/client/debugger/test/mochitest/addon-source/browser_dbg_addon3/lib/main.js
--- a/devtools/client/debugger/test/mochitest/addon-source/browser_dbg_addon3/lib/main.js
+++ b/devtools/client/debugger/test/mochitest/addon-source/browser_dbg_addon3/lib/main.js
@@ -1,13 +1,20 @@
 var { Cc, Ci } = require("chrome");
-var { once } = require("sdk/system/events");
+var observerService = Cc["@mozilla.org/observer-service;1"].getService(Ci.nsIObserverService);
 
-var observerService = Cc["@mozilla.org/observer-service;1"].getService(Ci.nsIObserverService);
 var observer = {
   observe: function () {
     debugger;
   }
 };
 
-once("sdk:loader:destroy", () => observerService.removeObserver(observer, "debuggerAttached"));
+observerService.addObserver(observer, "debuggerAttached");
 
-observerService.addObserver(observer, "debuggerAttached");
+var sdkLoaderDestroyObserver = {
+  observe: function () {
+    // Remove all observers on sdk:loader:destroy
+    observerService.removeObserver(observer, "debuggerAttached");
+    observerService.removeObserver(sdkLoaderDestroyObserver, "sdk:loader:destroy");
+  }
+};
+
+observerService.addObserver(sdkLoaderDestroyObserver, "sdk:loader:destroy");