1553029 - [remote-dbg-next] WebIDE deprecation message should only be displayed if new about:debugging is enabled

Status: VERIFIED FIXED

(DevTools :: about:debugging, task, P1)

Description

[Julian Descottes [:jdescottes]]

We started showing a deprecation message for WebIDE in Firefox 68. This deprecation message has a link that opens about:debugging. However this doesn't make any sense with the old about:debugging since it doesn't have any remote debugging features.

The new about:debugging was supposed to ride the trains with 68, but it turns out Release 68 will not have the new about:debugging.

The deprecation message should only be displayed if the new about:debugging is enabled.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1553029 - Show WebIDE deprecation message if the new about:debugging is enabled

Rather than having a separate preference for showing the deprecation message, we should reuse the aboutdebugging new pref

Comment 2

[Julian Descottes [:jdescottes]]

Attachment: Bug 1553029 - Update WebIDE deprecation message to stop mentioning release 69

There was a last minute schedule change for remote debugging release and we will not ride the trains with Firefox 68
The deprecation schedule is therefore shifting by one release. We update the deprecation message in WebIDE here and we should
uplift this change to beta if possible. The patch was written in order to be uplifted, we can go for a simpler version if the
uplift is a no go.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/92a16e67af5e
Update WebIDE deprecation message to stop mentioning release 69 r=flod,daisuke

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b2b3da42e524
Show WebIDE deprecation message if the new about:debugging is enabled r=daisuke

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/92a16e67af5e
https://hg.mozilla.org/mozilla-central/rev/b2b3da42e524

Comment 7

[Julian Descottes [:jdescottes]]

Comment on attachment 9066318 [details]
Bug 1553029 - Show WebIDE deprecation message if the new about:debugging is enabled

Beta/Release Uplift Approval Request

• User impact if declined: It was decided to hold remote debugging in DevEdition until 69, due to lack of QA resources to perform a final QA sign off in Beta 68.
  The first patch will prevent from showing the WebIDE deprecation message when the new remote debugging is not enabled.
  The second patch updates the deprecation message (which will still be displayed in DevEdition for instance) to say WebIDE will be removed in "an upcoming release" instead of "release 69"
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: STRs for first patch:

• open Firefox Beta with a new profile
• open WebIDE
  With the patch: No message should be displayed
  Without the patch: A deprecation is displayed with an "Open about:debugging" button

If you want to verify this on Nightly, you can just open Firefox Nightly and then set devtools.aboutdebugging.new-enabled to false in about:config.

STRs for second patch:

• open Firefox Nightly or DevEdition with a new profile
• open WebIDE
  With the patch: A deprecation message should be displayed in WebIDE and should not mention a specific release number
  Without the patch: A deprecation message is displayed in WebIDE and mentions "release 69"

• List of other uplifts needed: Bug 1553028, Bug 1553031
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): First patch is a small JS change that is already covered by automated test. Second change is a string change, but we added extra code to make sure the application would not crash in case the string was missing from language packs lagging behind.
• String changes made/needed: Removed error_webIDEDeprecated, added error_webIDEDeprecated2 in devtools/client/locales/en-US/webide.properties

Comment 8

[Hani Yacoub, Desktop QA]

Verified as fixed on Firefox Nightly 69.0a1 (2019-05-21) on Windows 10 x 64, Mac OS X 10.14 and on Ubuntu 18.04 x64.

Comment 9

[Julien Cristau [:jcristau] (back August 18)]

Comment on attachment 9066318 [details]
Bug 1553029 - Show WebIDE deprecation message if the new about:debugging is enabled

approved for 68.0b4

Comment 10

[Julien Cristau [:jcristau] (back August 18)]

woops. jumped the gun a little, sorry about that.

flod, is the string change here ok to uplift?

Comment 11

[Francesco Lodolo [:flod]]

(In reply to Julien Cristau [:jcristau] from comment #10)

woops. jumped the gun a little, sorry about that.

flod, is the string change here ok to uplift?

Yes, it's OK this early.

Comment 12

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-beta/rev/da60c46020d5
https://hg.mozilla.org/releases/mozilla-beta/rev/03dd32c084c5

Comment 13

[Hani Yacoub, Desktop QA]

Verified as fixed on Firefox 68.0b4 on Windows 10 x 64, Mac OS X 10.14 and on Ubuntu 18.04 x64.