Closed Bug 1412949 Opened 5 years ago Closed 5 years ago
Some code in System
Memory Reporter might be dead code
I've noticed LinuxUtils.cpp is completely uncovered by tests (https://codecov.io/gh/marco-c/gecko-dev/src/1ebd2eff44617df3b82eea7d2f3ca1b60cc591a0/xpcom/base/LinuxUtils.cpp). The portions of SystemMemoryReporter.cpp which are supposed to call LinuxUtils.cpp are not covered too (https://codecov.io/gh/marco-c/gecko-dev/src/1ebd2eff44617df3b82eea7d2f3ca1b60cc591a0/xpcom/base/SystemMemoryReporter.cpp). Should we remove these portions now that Firefox OS is no longer actively developed and Firefox probably doesn't have access to those system directories?
It's not completely dead... that code is behind the memory.system_memory_reporter pref, which is false by default. You can turn it on and it works. But it's probably not useful, so I'll write a patch to remove it.
jld, can you see any reason not to remove this?
Assignee: nobody → n.nethercote
Status: NEW → ASSIGNED
Comment on attachment 8923646 [details] [diff] [review] Remove SystemMemoryReporter Review of attachment 8923646 [details] [diff] [review]: ----------------------------------------------------------------- Kind of sad to see these go, but I can't think of a good reason to keep them. Can you update the about:memory docs  as well?  https://developer.mozilla.org/en-US/docs/Mozilla/Performance/about:memory#System
Attachment #8923646 - Flags: review?(erahm) → review+
Comment on attachment 8923646 [details] [diff] [review] Remove SystemMemoryReporter I can't think of a good reason to keep this either.
Attachment #8923646 - Flags: feedback?(jld) → feedback+
https://hg.mozilla.org/integration/mozilla-inbound/rev/d76af6611465cfccaec40098493eedcd0ba1298b Bug 1412949 - Remove SystemMemoryReporter. r=erahm.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/d76af6611465 Remove SystemMemoryReporter. r=erahm.
You need to log in before you can comment on or make changes to this bug.