Closed Bug 1381175 Opened 3 years ago Closed 3 years ago

Display count of web content processes in about:support

Categories

(Firefox :: General, defect)

56 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 56
Tracking Status
firefox55 - fixed
firefox56 --- fixed

People

(Reporter: Felipe, Assigned: Felipe)

Details

Attachments

(1 file)

This should count the number of processes dedicated to web content, and not include the other types of child processes (plugins, gfx, etc).
[Tracking Requested - why for this release]:
This information will be useful to let users easily see the current number of processes they are running with e10s multi
Comment on attachment 8886721 [details]
Bug 1381175 - Display count of web content processes in about:support.

https://reviewboard.mozilla.org/r/157516/#review163210
Attachment #8886721 - Flags: review?(mrbkap) → review+
Pushed by felipc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/af62be2909cf
Display count of web content processes in about:support. r=mrbkap
Comment on attachment 8886721 [details]
Bug 1381175 - Display count of web content processes in about:support.

Approval Request Comment
[Feature/Bug causing the regression]: Display count of processes for e10s-multi
[User impact if declined]: No easy way to tell from within Firefox how many processes are being used
[Is this code covered by automated tests?]: Yes
[Has the fix been verified in Nightly?]: No
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: No
[Why is the change risky/not risky?]: just change some information displayed in about:support
[String changes made/needed]: There is one new string ("Web Content Processes"), but we think we should uplift a hardcoded version of it (english-only) to have this included in about:support for 55. There's already a lot of text in about:support that is not localizable, and I think the benefits of this outweighs the untranslated string for one version.
Attachment #8886721 - Flags: approval-mozilla-beta?
https://hg.mozilla.org/mozilla-central/rev/af62be2909cf
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
I don't think I need to track this.
Comment on attachment 8886721 [details]
Bug 1381175 - Display count of web content processes in about:support.

add info about web content processes in about:support, beta55+

This needs a separate patch for beta with the english string hardcoded.
Attachment #8886721 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
(In reply to Julien Cristau [:jcristau] from comment #8)
> This needs a separate patch for beta with the english string hardcoded.

Needs rebasing anyway.
Flags: needinfo?(felipc)
(In reply to :Felipe Gomes (needinfo me!) from comment #5)
> [Is this code covered by automated tests?]: Yes
> [Has the fix been verified in Nightly?]: No
> [Needs manual test from QE? If yes, steps to reproduce]: No

Setting qe-verify- based on Felipe's assessment on manual testing needs and the fact that this fix has automated coverage.
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.