1709266 - [DevTools Release 90 Tasks] Remove backward compatibility code for emitDescriptorDestroyed

Status: RESOLVED FIXED

(DevTools :: General, task, P3)

Description

[Julian Descottes [:jdescottes]]

See https://searchfox.org/mozilla-central/search?path=&q=emitDescriptorDestroyed

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1709266 - [devtools] Remove backward compatibility code for emitDescriptorDestroyed

We no longer support connecting to FF87. emitDescriptorDestroyed was added to all tab descriptors in FF88 so all the code checking it can be removed.
Rephrased the comment around the about:debugging use case.

Comment 2

[Julian Descottes [:jdescottes]]

Attachment: Bug 1709266 - [devtools] Remove unused isDevToolsExtensionContext check in tabDescriptor

Depends on D114198

The isDevToolsExtensionContext flag was removed in Bug 1699493, we should be able to remove this check

Comment 3

[Julian Descottes [:jdescottes]]

Attachment: Bug 1709266 - [devtools] Stop listening for worker-close in worker descriptor front

Depends on D114199

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b70ec1cbca94
[devtools] Remove backward compatibility code for emitDescriptorDestroyed r=ochameau
https://hg.mozilla.org/integration/autoland/rev/c5690af9defb
[devtools] Remove unused isDevToolsExtensionContext check in tabDescriptor r=ochameau
https://hg.mozilla.org/integration/autoland/rev/facd310ec191
[devtools] Stop listening for worker-close in worker descriptor front r=ochameau,devtools-backward-compat-reviewers

Comment 5

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/b70ec1cbca94
https://hg.mozilla.org/mozilla-central/rev/c5690af9defb
https://hg.mozilla.org/mozilla-central/rev/facd310ec191