1578361 - Remove unused localTransport getter on debugger-client

Status: RESOLVED FIXED

(DevTools :: General, task, P3)

Description

[Julian Descottes [:jdescottes]]

In debugger-client's constructor we set

this.localTransport = this._transport.onOutputStreamReady === undefined;

https://searchfox.org/mozilla-central/rev/e04021f29e6d8a37753ba2b510432315ce05a8d7/devtools/shared/client/debugger-client.js#61

While this works, it's very weird to rely on a random callback being defined or not on the transport to know if we are on using a localtransport.

This is not used so it should be removed.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1578361 - removed unused localTransport getter on debugger-client

This was initially added in Bug 907755 for old telemetry probes.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/872fcf1f5fa6
removed unused localTransport getter on debugger-client r=ladybenko

Comment 4

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/872fcf1f5fa6