1882096 - Remove CDP support from Remote Agent

Status: RESOLVED FIXED

(Remote Protocol :: CDP, task, P2)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Later this year we want to drop support for CDP from our code base on trunk and only keep support on the 128 ESR branch.

This bug should remove all the CDP related code and tests that still live in tree.

Comment 1

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

The severity field is not set for this bug.
:whimboo, could you have a look please?

For more information, please visit BugBot documentation.

Comment 2

[Julian Descottes [:jdescottes]]

Henrik, do you think we need another bug to remove the remote.active-protocols preference, or should we handle it here as well?

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

We probably can remove the preference at the same time when removing CDP support given that it is all bundled together. It's basically no real extra work. And in case some other protocol comes up in the future we can re-add this preference or something similar.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Note that we are still blocked on removing CDP until both Selenium and Cypress have released their new major versions that do no longer need CDP for Firefox.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

We actually decided to not remove our CDP as of this time given that folks still using the protocol might be affected and no longer able to support Firefox. As such we are waiting a little bit longer until the next ESR release of Firefox (140) is out. Once development on 141 started on May 26th we will remove CDP. That means it will fit into our milestone 16.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1882096 - [remote] Remove CDP support from Remote Agent.

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1882096 - [remote] Remove CDP support from Remote Agent.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f587a8c46fc3
[remote] Remove CDP support from Remote Agent. r=webdriver-reviewers,jdescottes

Comment 10

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/f587a8c46fc3

Comment 11

[Pascal Chevrel:pascalc]

Henrik, should that be mentioned in our release notes?

Comment 12

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to Pascal Chevrel:pascalc from comment #11)

Henrik, should that be mentioned in our release notes?

Thanks for asking! I don't think that we have to mention the removal of our experimental CDP implementation in the official release notes of Firefox. It's only affecting developers and testers and as such is better suited to be added to the release notes on MDN, which we will take care of.