1405793 - Port bug 1401946 to TB [Show profile path next to the open button]

Status: RESOLVED FIXED

(Thunderbird :: General, enhancement)

Description

[Richard Marti (:Paenglab)]

Firefox shows now the path to the profile folder.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: aboutSupportPath.patch

FX change: https://hg.mozilla.org/mozilla-central/rev/b5da04ce9e4372d20784eb49721210e268e91d42

We have already the show profile folder in about:support implemented with the ability to hide it and be a link. I added only the change for the xhtml file to be still in sync and changed slightly the js part to work with the added ID.

Comment 2

[:aceman]

Comment on attachment 8915256 [details] [diff] [review]
aboutSupportPath.patch

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

Thanks, seems to work.
But please update also the test (searching the tree for the id you are changing is useful):

diff --git a/mail/test/mozmill/content-tabs/test-about-support.js b/mail/test/mozmill/content-tabs/test-about-support.js
--- a/mail/test/mozmill/content-tabs/test-about-support.js
+++ b/mail/test/mozmill/content-tabs/test-about-support.js
@@ -202,17 +202,17 @@ function test_modified_pref_on_blacklist
  */
 function test_private_data() {
   let tab = open_about_support();
   let checkbox = content_tab_eid(tab, "check-show-private-data");
.
   // We use the profile path and some other element as an example
   // of a private-only element.
   let privateElem1 = find_private_element(tab);
-  let privateElem2 = content_tab_e(tab, "profile-dir-path");;
+  let privateElem2 = content_tab_e(tab, "profile-dir-box");

Comment 3

[Richard Marti (:Paenglab)]

Attachment: aboutSupportPath.patch

Updated the test with the correct ID.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/38273b6b8bb0
Port bug 1401946 to TB [Show profile path next to the open button]. r=aceman