Page MenuHomePhabricator

Bug 1818465 - Add JSOracle+WindowsUtils actors name to about:processes r?florian!
ClosedPublic

Authored by gerard-majax on Feb 23 2023, 2:23 PM.
Referenced Files
Unknown Object (File)
Thu, May 1, 7:13 AM
Unknown Object (File)
Thu, Apr 17, 10:03 AM
Unknown Object (File)
Wed, Apr 16, 11:03 PM
Unknown Object (File)
Apr 9 2025, 10:35 PM
Unknown Object (File)
Apr 4 2025, 2:21 PM
Unknown Object (File)
Mar 6 2025, 2:35 PM
Unknown Object (File)
Mar 1 2025, 7:13 AM
Unknown Object (File)
Feb 16 2025, 7:39 AM

Diff Detail

Repository
rMOZILLACENTRAL mozilla-central
Lint
Lint Not Applicable
Unit
Tests Not Applicable
Build Status
Buildable 511448
Build 607875: arc lint + arc unit

Event Timeline

phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: secure-revision.
florian added inline comments.
toolkit/components/aboutprocesses/content/aboutProcesses.js
892

Are you confident that with the test added in D170753 we will never display this "Unknown actor" string? If not, should we change this string to include the actorName as a parameter? (ie. "Unknown actor (${actorName})")

gerard-majax added inline comments.
toolkit/components/aboutprocesses/content/aboutProcesses.js
892

Since we iterate on all member in the test, i can't see a case where we could get tricked

toolkit/locales/en-US/toolkit/about/aboutProcesses.ftl
133

Should these two strings be kept in English (and have a comment to explain it)?

Out of curiosity: Oracle as in "Oracle DB" or something else?

gerard-majax edited the summary of this revision. (Show Details)
gerard-majax marked an inline comment as done.
gerard-majax added inline comments.
toolkit/locales/en-US/toolkit/about/aboutProcesses.ftl
133

This is a question for @handyman and @farre

toolkit/locales/en-US/toolkit/about/aboutProcesses.ftl
133

Oracle as in something that answers questions, so not anything tied to the company.

toolkit/locales/en-US/toolkit/about/aboutProcesses.ftl
133

And about localization. I really don't know. In that case we should probably expand JS to JavaScript and only localize Oracle. I can see it being "JavaScript Orakel" in Swedish. But it is also really only an internal name of the process that handles JavaScript validation for Opaque Response Blocking.

toolkit/locales/en-US/toolkit/about/aboutProcesses.ftl
133

Given this seems to be internal (couldn't find anything with a search on the web), I would add a comment to keep it in English.

If space is not a concern, I would also expand JS.

# "Oracle" refers to an internal Firefox process and should be kept in English
about-processes-utility-actor-js-oracle = JavaScript Oracle
about-processes-utility-actor-windows-utils = Windows Utils

I assume it's OK if "Utils" gets translated by some as "Utilities". If not, we should add a similar comment.

toolkit/locales/en-US/toolkit/about/aboutProcesses.ftl
133

Yeah, JavaScript Oracle as an internal name, only in English, is good for the js oracle.

This revision is now accepted and ready to land.Feb 23 2023, 4:31 PM
This revision is now accepted and ready to land.Feb 24 2023, 8:37 PM