1648378 - Remove virtual memory from about:processes

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, enhancement)

Description

[David Teller [:Yoric] - still alive but not very active]

So, we can either remove it entirely or hide it behind a pref.
Any preference, nika, kmag?

Comment 1

[Nika Layzell [:nika] (ni? for response)]

I don't really see much of a reason to keep it around, so I'd be fine with just completely removing it.

Comment 2

[David Teller [:Yoric] - still alive but not very active]

Marking this bug as mentored.

The objective of this bug is to patch aboutProcesses.js, aboutProcesses.html, aboutProcesses.css and aboutProcesses.ftl to get rid of the column called "Virtual memory" (column-memory-virtual).

Comment 3

[Daniel Zhang]

Attachment: Correction to aboutProcesses.js

Comment 4

[Daniel Zhang]

Attachment: Patch to aboutProcesses.html

Comment 5

[Daniel Zhang]

Attachment: Patch to aboutProcesses.ftl

Comment 6

[David Teller [:Yoric] - still alive but not very active]

Hey, Daniel Zhang, thanks for the patches!

Unfortunately, we have stopped using splinter a few years ago. Any chance you could use phabricator instead? I realize that the documentation of phabricator can be arid, so don't hesitate to ask questions on #introduction!

Comment 7

[David Teller [:Yoric] - still alive but not very active]

Note that you'll need to use moz-phab submit to send the patches, instead of sending the modified files.

Comment 8

[David Teller [:Yoric] - still alive but not very active]

Note: We have deadline issues here, so if this bug isn't completed next week, I'll pick it up.

Comment 9

[Sonia Singla :soniasingla]

Attachment: Bug 1648378 - Removed Virtual Memory from about:processes

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/47bf6da3e008
Removed Virtual Memory from about:processes r=Yoric,fluent-reviewers,flod

Comment 11

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/47bf6da3e008