1870919 - DevTools Release Tasks - Cycle 123

Status: RESOLVED FIXED

(DevTools :: General, task, P1)

Description

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

Soft Freeze / Merge Date: 2024-01-11 / 2024-01-22 (from the Release train page)
More details about any of those tasks can be found at: https://firefox-source-docs.mozilla.org/devtools/release.html

Anytime during the release:

• [x] outdated backward compatibility code removed (doc, searchfox)
• [x] outdated telemetry probes removed (bz query)
• [x] in devtools-qa bugs, needinfo the triage owner (or the assignee if they are a team member) to test the bug in beta channel (bz query)
• [x] Update MDN data for the Compatibility panel follow devtools/shared/compatibility/README.md
• [x] Check the browser information in the Compatibility Panel is up to date (example: version number for Firefox Nightly should be 123):
  • Steps: Open the Inspector, then select the Compatibility side panel, and click on Settings to see the list of all browsers

Close to the end of the release

• [x] smoke test for remote debugging against version 122 (doc)
• [x] smoke test for remote debugging against the same version (version 123) (same doc)
• [x] check xpcshell debugging (doc)

Marionette and BiDi:

• [ ] close to the end of release (~1 week), based on the list of bugs fixed during the release, update Marionette release notes on MDN for 122: https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/122#webdriver_conformance_marionette.
• [ ] right after the release of Firefox 122, send a mail to the dev-webdriver mailing list.

For reference: full list of Marionette and BiDi bugs fixed during the release.

Template suggestion for the Marionette email (adjust wording if relevant):

Firefox 122 has just been released, containing updates for our WebDriver (BiDi, Marionette) support.

You can find all the details in the change log for web developers:

https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/122#webdriver_conformance_webdriver_bidi_marionette

See https://docs.google.com/document/d/1t8pR_lcTkjZjX1ddOf6tyvPY8dtE519Vlu8wXX58x_E for detailed explanations about Marionette tasks, or contact :whimboo / :jdescottes for help

Nice to have:

• [ ] Generate webidl-pure-allowlist.js and webidl-deprecated-list.js (doc)

Comment 1

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

I had a look at the removal of the backward compatible code but I have troubles with the following:
https://searchfox.org/mozilla-central/rev/a46c21ec5559b6d1ac96fcb95bff6885a10bbf99/devtools/server/actors/root.js#138

Julian or NIcholas, could you please explain what needs to get done here?

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

(In reply to Henrik Skupin [:whimboo][⌚️UTC+1] from comment #1)

I had a look at the removal of the backward compatible code but I have troubles with the following:
https://searchfox.org/mozilla-central/rev/a46c21ec5559b6d1ac96fcb95bff6885a10bbf99/devtools/server/actors/root.js#138

Julian or NIcholas, could you please explain what needs to get done here?

So here, you can remove this line, and we can assume on the client that this is always true.
Looking at how it's used (https://searchfox.org/mozilla-central/search?q=supportsDebuggerStatementIgnore&path=&case=true&regexp=false), it's a bit involved, but we should be able to remove all the conditions where it's used, and remove the intermediary bits where we put it into the debugger redux state. I'm happy to do it if you don't have time for it :)

Comment 3

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

The PR for the release notes on MDN can be found at: https://github.com/mdn/content/pull/31792

Comment 4

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

This is all done including the blog post and email to the webdriver mailing list.