Show the pause reason in the Pause Overlay when remote debugging mobile Firefox
Categories
(DevTools :: Debugger, task, P3)
Tracking
(firefox96 fixed)
Tracking | Status | |
---|---|---|
firefox96 | --- | fixed |
People
(Reporter: jdescottes, Assigned: nchevobbe)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
text/x-phabricator-request
|
Details |
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?
Assignee | ||
Comment 2•4 years ago
|
||
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 | ||
Comment 3•3 years ago
|
||
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.
Assignee | ||
Comment 4•3 years ago
|
||
Depends on D132223
Updated•3 years ago
|
Assignee | ||
Comment 5•3 years ago
|
||
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.
Comment 6•3 years ago
|
||
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.
Updated•3 years ago
|
Comment 8•3 years ago
|
||
Backed out for causing mochitest failures on browser_metrics_debugger.js.
Backout link: https://hg.mozilla.org/integration/autoland/rev/c6b7e343048bd01cfd8f5df563795debcd558fdf
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
Comment 10•3 years ago
|
||
bugherder |
Description
•