684339 - about:memory should link to about:support

Status: RESOLVED FIXED

(Toolkit :: about:memory, defect)

Description

[(no longer active)]

This information is usually very valuable when filing memory bugs.  We should just add it to about:memory IMO.

Comment 1

[Justin Lebar (not reading bugmail)]

Standard operating procedure should be including about:memory *and* about:support.  But seeing as it's not, maybe ehsan is right.

Comment 2

[(no longer active)]

Attachment: Patch (v1)

Comment 3

[Nicholas Nethercote [inactive]]

Comment on attachment 558581 [details] [diff] [review]
Patch (v1)

Review of attachment 558581 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/components/aboutmemory/content/aboutMemory.js
@@ +259,5 @@
>          : "<span class='option'><a href='about:memory?verbose'>More verbose</a></span>";
>    text += "</div>";
>  
>    text += "<div>" +
> +          "<span class='option'><a href='about:support'>Useful troubleshooting infromation</a></span>" +

Drop the "Useful", and spell "information" correctly :)  Then it'll just be "Troubleshooting information" which matches the menu item.

Comment 4

[(no longer active)]

Come on, "information" is sooooo 1990!  :P

http://hg.mozilla.org/integration/mozilla-inbound/rev/fd50c3e8aad3

Comment 5

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/fd50c3e8aad3