Closed Bug 1591025 Opened 5 years ago Closed 3 years ago

Show the pause reason in the Pause Overlay when remote debugging mobile Firefox

Categories

(DevTools :: Debugger, task, P3)

task

Tracking

(firefox96 fixed)

RESOLVED FIXED
96 Branch
Tracking Status
firefox96 --- fixed

People

(Reporter: jdescottes, Assigned: nchevobbe)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 2 obsolete files)

Followup to Bug 1590932.

The current implementation of the pause overlay is relying on a client side properties file to show the "pause" reason (whyPaused. strings from https://searchfox.org/mozilla-central/source/devtools/client/locales/en-US/debugger.properties#743)

This file is not available on mobile builds of Firefox and crashes the debugger. Since we need to uplift a fix we will provide a temporary solution in Bug 1590932 that doesn't show the pause reason in mobile builds.

However we should find a solution to provide the same feature to users debugging mobile Firefox. Either move the strings to a shared file, or let the client pass the string to the highlighter?

Thanks for the report Julian!
Honza

Priority: -- → P3

I guess we can use the highlighter properties file we already have: https://searchfox.org/mozilla-central/source/devtools/shared/locales/en-US/highlighters.properties

I'll give this a try

Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED

The next patch of this queue will add a shared debugger.properties file for
actual debugger related strings.
The file we're renaming doesn't have much to do with the debugger, so we
can rename it to something more specific.

Attachment #9252604 - Attachment is obsolete: true
Blocks: 1743155

Since those strings can be consumed by the PausedDebuggerOverlay, on the server,
we need to put them in devtools/shared.
As this will create some work for the l10n team, we migrate them to Fluent so we're
feeding 2 birds with 1 scone.
The <LocalizationProvider> is added directly in <WhyPaused> (and not in <App>,
as it's messing up with the React context (See Bug 1743155).
Some snapshots are updated. It's unfortunate that we don't see the translated
strings there anymore, but we can't have nice things in Jest.

Depends on: 1743347

Comment on attachment 9252666 [details]
Bug 1591025 - [devtools] Move whyPaused.* string to a ftl file in devtools/shared. r=bomsy!,jdescottes!

Revision D132260 was moved to bug 1743347. Setting attachment 9252666 [details] to obsolete.

Attachment #9252666 - Attachment is obsolete: true
Attachment #9252605 - Attachment description: Bug 1591025 - [devtools] Show PausedDebuggerOverlay in remote debugged Fenix. r=jdescottes. → Bug 1591025 - [devtools] Display PausedDebuggerOverlay images in remote debugged Fenix. r=jdescottes.
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/48c568e6d4d4 [devtools] Display PausedDebuggerOverlay images in remote debugged Fenix. r=jdescottes.

Backed out for causing mochitest failures on browser_metrics_debugger.js.

Backout link: https://hg.mozilla.org/integration/autoland/rev/c6b7e343048bd01cfd8f5df563795debcd558fdf

Push with failures : https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Crunnable&revision=48c568e6d4d4497f7886264efb3fe93346f4341f&selectedTaskRun=MHXJVi03SdWDvm5ABGftPQ.0

Link to failure log : https://treeherder.mozilla.org/logviewer?job_id=359928129&repo=autoland&lineNumber=2865

Failure message: TEST-UNEXPECTED-FAIL | devtools/client/framework/test/metrics/browser_metrics_debugger.js | Duplicated module load detected. List of duplicated modules

Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/90369a230cb4 [devtools] Display PausedDebuggerOverlay images in remote debugged Fenix. r=jdescottes.
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: