1495657 - Remove DebuggerClient.addListener calls related to workerListChanged once all related client are converted to fronts

Status: RESOLVED WORKSFORME

(DevTools :: Framework, enhancement, P1)

Description

[Alexandre Poirot [:ochameau]]

Bug 1485660 is introducing a temporary code in order to support either old fashion client, emitting events on DebuggerClient or fronts, emitting events on the front itself.

We should remove that and only listen on fronts once all the related clients are converted to fronts. i.e. bug 1485660, bug 1485661, bug 1494632.

This only impact two files:
 * devtools/client/debugger/debugger-controller.js
 * devtools/client/debugger/new/src/client/firefox/events.js

And will concists in doing:
-    // `_tabClient` an be BrowsingContextTargetFront (protocol.js front) or WorkerClient
-    // (old fashion client)
-    if (typeof(this._tabClient.on) == "function") {
-      this._tabClient.on("workerListChanged", this._onWorkerListChanged);
-    } else {
-      this._tabClient.addListener("workerListChanged", this._onWorkerListChanged);
-    }
+    this._tabClient.on("workerListChanged", this._onWorkerListChanged);

Comment 1

[Yulia Startsev [:yulia]]

It looks like this was done: https://searchfox.org/mozilla-central/search?q=addListener%28%22workerListChanged%22&path=