660718 - Add link from about:support to about:memory

Status: RESOLVED FIXED

(Toolkit :: General, enhancement)

Description

[Jesse Ruderman]

Maybe just a simple line in the first table, near the link to about:plugins:

Memory use      about:memory

Comment 1

[Ed Morley [:emorley]]

Attachment: Patch v1

Adds "Memory Use     about:memory" as an extra row to the end of the 'Application Basics' block in about:support, after the about:crashes entry.

I'm open to suggestions as to the placement/naming.

My reasons for adding to the end of the application basics block were:
- Avoids breaking muscle memory for the other entries above it
- The addition will hopefully be more noticeable for people who are used to about:support already, who may otherwise miss it if added halfway through the existing list
- about:plugins and about:buildconfig are related to the current install rather than the current session, so it seems to make more sense to put about:memory with about:crashes (which is at least about recent sessions, even if not the current one)

Also removes some trailing whitespace in aboutSupport.xhtml.

Comment 2

[Ed Morley [:emorley]]

Attachment: Screenshot v1

Comment 3

[Ed Morley [:emorley]]

Quick sanity check, seeing as I was pushing a bunch of things to try anyway:
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=39b4ffb18fd5

Comment 4

[(no longer active)]

Comment on attachment 559985 [details] [diff] [review]
Patch v1

I'm going to break the rules, and r+ this!

Comment 5

[Ed Morley [:emorley]]

Thanks Ehsan :-)

https://hg.mozilla.org/integration/mozilla-inbound/rev/9d0c5f3e3392

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/9d0c5f3e3392