Bug 1473513 - resolve memory leak

Summary: Depends on D6476

Reviewers: ochameau

Reviewed By: ochameau

Bug #: 1473513

Differential Revision: https://phabricator.services.mozilla.com/D6477

--HG--
extra : rebase_source : 61ccae905e3136ad3c293db692551786e83075c7

devtools/server/actors/targets/browsing-context.js

@@ -495,8 +495,8 @@ const browsingContextTargetPrototype = {
    * Called when the actor is removed from the connection.
    */
   destroy() {
-    Actor.prototype.destroy.call(this);
     this.exit();
+    Actor.prototype.destroy.call(this);
   },
 
   /**

devtools/server/actors/targets/worker.js

@@ -90,10 +90,10 @@ const WorkerTargetActor = protocol.ActorClassWithSpec(workerTargetSpec, {
   },
 
   destroy() {
-    protocol.Actor.prototype.destroy.call(this);
     if (this._attached) {
       this._detach();
     }
+    protocol.Actor.prototype.destroy.call(this);
   },
 
   connect(options) {

devtools/server/actors/worker/service-worker.js

@@ -50,8 +50,8 @@ const PushSubscriptionActor = protocol.ActorClassWithSpec(pushSubscriptionSpec,
   },
 
   destroy() {
-    protocol.Actor.prototype.destroy.call(this);
     this._subscription = null;
+    protocol.Actor.prototype.destroy.call(this);
   },
 });
 

devtools/shared/transport/tests/unit/testactors.js

@@ -7,7 +7,6 @@ const { RootActor } = require("devtools/server/actors/root");
 const { ThreadActor } = require("devtools/server/actors/thread");
 const { DebuggerServer } = require("devtools/server/main");
 const { ActorRegistry } = require("devtools/server/actor-registry");
-const promise = require("promise");
 
 var gTestGlobals = [];
 DebuggerServer.addTestGlobal = function(global) {