Commit 2379fd70 authored by Julian Descottes's avatar Julian Descottes
Browse files

Bug 1747041 - [cdp] Check if message manager exists in TabSession destructor... 

Bug 1747041 - [cdp] Check if message manager exists in TabSession destructor r=webdriver-reviewers,whimboo

Differential Revision: https://phabricator.services.mozilla.com/D134507
parent c310de99

remote/cdp/sessions/TabSession.jsm

+6 −4
Original line number Diff line number Diff line
@@ -50,13 +50,15 @@ class TabSession extends Session { 


    this.requestPromises.clear();



    this.mm.sendAsyncMessage("remote:destroy", {

    // this.mm might be null if the browser of the TabTarget was already closed.

    // See Bug 1747301.

    this.mm?.sendAsyncMessage("remote:destroy", {

      browsingContextId: this.browsingContext.id,

    });



    this.mm.removeMessageListener("remote:event", this);

    this.mm.removeMessageListener("remote:result", this);

    this.mm.removeMessageListener("remote:error", this);

    this.mm?.removeMessageListener("remote:event", this);

    this.mm?.removeMessageListener("remote:result", this);

    this.mm?.removeMessageListener("remote:error", this);

  }



  execute(id, domain, command, params) {