Closed
Bug 407008
Opened 17 years ago
Closed 17 years ago
Remove personal data option doesn't remove empty Mozilla/Firefox directories in appdata
Categories
(Firefox :: Installer, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3
People
(Reporter: andrewm715+bugzilla, Assigned: robert.strong.bugs)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
benjamin
:
review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b2pre) Gecko/2007120505 Minefield/3.0b2pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b2pre) Gecko/2007120505 Minefield/3.0b2pre
One of the things that annoys many Windows users is the useless files, folders and registry entries (aka 'gunk') that gets left behind after an application has been uninstalled. Firefox is much better than most programs at removing traces of its existence on your hard drive, but there are still a few leftovers. (The following applies to Windows Vista, although I'm pretty sure that it's the same on Windows XP and Windows 2000, but just with different paths; I have no idea about Mac and Linux :)
Reproducible: Always
Steps to Reproduce:
1. Download and install a recent nightly (but not the most recent; for example yesterday's).
2. Open Minefield, Help > Check for Updates... > Download & Install Now
3. After update has downloaded, click to restart Minefield.
4. After update has been installed and Minefield has restarted, close Minefield.
5. Uninstall Minefield through the Control Panel, making sure to select the checkbox 'Remove my Minefield personal data and customizations'.
6. Look in C:\Users\[USERNAME]\AppData\Roaming, C:\Users\[USERNAME]\AppData\Local and C:\Program Files.
Actual Results:
In C:\Users\[USERNAME]\AppData\Roaming, note that there is a folder called 'Mozilla' with just a sub-folder 'Firefox' which contains just an empty sub-folder 'Profiles'.
In C:\Users\[USERNAME]\AppData\Local, note that there is a folder called 'Mozilla' with just a sub-folder 'Firefox' which contains just a sub-folder 'Minefield', which in turn contains an 'updates' sub-folder (with an empty sub-folder '0' and the file 'last-update.log') and the files 'active-update.xml' and 'updates.xml'.
In C:\Program Files, note that there is the empty folder 'Minefield'.
Assuming Minefield was the only Mozilla application installed, that's 9 folders and 3 files left behind that are of no real value to the user; even if he decides to reinstall Minefield at a later date.
Expected Results:
When Minefield is uninstalled and the option to 'Remove my Minefield personal data and customizations' is selected, I expect that every trace of Firefox's existence should be deleted from my hard drive. Most traces are removed, but some gunk is still left behind. It is my understanding that the files listed above are recreated automatically when the need arises, and they certainly don't contain any personal information. I see no harm in deleting them when just Minefield is uninstalled, and IMHO they should certainly be deleted when I select the option to remove the profile data as well.
I realize of course that if you have other Mozilla products installed (e.g. Thunderbird) the folders themselves should not be deleted because they would not be empty, but if Minefield is the only Mozilla product installed on the system then I don't see why the empty folders should not removed as well when it is uninstalled.
Assignee | ||
Comment 1•17 years ago
|
||
That directory has nothing to do with profile removal so removing dependency on bug 398434.
No longer depends on: 398434
Assignee | ||
Comment 2•17 years ago
|
||
Relevant portions of comment #0
(In reply to comment #0)
>...
> In C:\Users\[USERNAME]\AppData\Roaming, note that there is a folder called
> 'Mozilla' with just a sub-folder 'Firefox' which contains just an empty
> sub-folder 'Profiles'.
We can remove these directories if they are empty.
> In C:\Users\[USERNAME]\AppData\Local, note that there is a folder called
> 'Mozilla' with just a sub-folder 'Firefox' which contains just a sub-folder
> 'Minefield', which in turn contains an 'updates' sub-folder (with an empty
> sub-folder '0' and the file 'last-update.log') and the files
> 'active-update.xml' and 'updates.xml'.
This is bug 392150
> In C:\Program Files, note that there is the empty folder 'Minefield'.
This worked for me last I checked a couple of weeks ago. Can you verify that the Minefield directory was in fact empty?
Status: UNCONFIRMED → NEW
Ever confirmed: true
(In reply to comment #2)
> > In C:\Program Files, note that there is the empty folder 'Minefield'.
> This worked for me last I checked a couple of weeks ago. Can you verify that
> the Minefield directory was in fact empty?
I just tried uninstalling Minefield, and on Vista, the empty Minefield folder is definitely left behind. I did notice something that seemed a bit strange, though. When I right-clicked the folder > Properties, the Minefield folder was marked as 'partially' Read-only (the solid blue square).
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b3pre) Gecko/2007121305 Minefield/3.0b3pre ID:2007121305
Depends on: 392150
Assignee | ||
Comment 4•17 years ago
|
||
There may be a hidden file in there. For example, if you opened that directory with explorer and changed the view a desktop.ini will be created. Try deleting the directory, install, and then uninstall without opening the directory with explorer until all of the steps are done.
Removing dependency... they are two different bugs and can be fixed separately.
No longer depends on: 392150
(In reply to comment #4)
> There may be a hidden file in there. For example, if you opened that directory
> with explorer and changed the view a desktop.ini will be created. Try deleting
> the directory, install, and then uninstall without opening the directory with
> explorer until all of the steps are done.
>
> Removing dependency... they are two different bugs and can be fixed separately.
Here's what I did:
Set Windows to display hidden and system files
Uninstalled Minefield
Looked in C:\Program Files\Minefield; nothing shows up there and in Properties, Windows reports Size: 0 bytes and 0 Files, 0 Folders
Deleted the Minefield folder
Installed Minefield
Uninstalled Minefield
Opened C:\Program Files
The Minefield folder was still there and it still looked empty (and Properties reported 0 bytes and 0 Files, 0 Folders, and Read-only still has the blue square). Hope that helps :)
Assignee | ||
Updated•17 years ago
|
Summary: Some files and folders are left behind after uninstall, even with profile data removal option enabled → Remove personal data option doesn't remove empty Mozilla/Firefox directories in appdata
Assignee | ||
Comment 6•17 years ago
|
||
Confirmed, filed bug 408314 to cover this specific issue and changed summary so this bug is about the removal of empty profile directories.
Just as an update, now that bug 392150 and bug 408314 have been fixed, there are now only two of the original empty directories that I mentioned in comment 0 left behind after uninstalling Minefield:
C:\Users\[USERNAME]\AppData\Local\Mozilla\Firefox
and
C:\Users\[USERNAME]\AppData\Roaming\Mozilla\Firefox\Profiles
However, I noticed that there is now another empty directory left behind:
C:\Users\[USERNAME]\AppData\Roaming\Mozilla\Extensions\{ec8030f7-c20a-464f-9b0e-13a3a9e97384}
Should I file a spin-off bug about this or can this bug cover this as well?
Assignee | ||
Comment 8•17 years ago
|
||
(In reply to comment #7)
>...
> C:\Users\[USERNAME]\AppData\Roaming\Mozilla\Extensions\{ec8030f7-c20a-464f-9b0e-13a3a9e97384}
>
> Should I file a spin-off bug about this or can this bug cover this as well?
No, we didn't create that directory and would never want to remove it... it is the responsibility of whatever added that to remove it.
(In reply to comment #8)
> (In reply to comment #7)
> >...
> > C:\Users\[USERNAME]\AppData\Roaming\Mozilla\Extensions\{ec8030f7-c20a-464f-9b0e-13a3a9e97384}
> >
> > Should I file a spin-off bug about this or can this bug cover this as well?
> No, we didn't create that directory and would never want to remove it... it is
> the responsibility of whatever added that to remove it.
Actually, that directory is created by Minefield. I'm not sure exactly when I first noticed it (my best guess is a few weeks ago) or what it's there for, but if you delete it it will just be recreated the next time you run Minefield. (I confirmed this by uninstalling and removing all empty directories related to Minefield, installing the 032304 nightly and then uninstalling it and looking at what got left behind.)
Assignee | ||
Comment 10•17 years ago
|
||
Ahhh... that's the new user level extension install location. Removing it should be fine and a separate bug should be filed.
Reporter | ||
Comment 11•17 years ago
|
||
(In reply to comment #10)
> Ahhh... that's the new user level extension install location. Removing it
> should be fine and a separate bug should be filed.
Filed bug 424999.
Assignee | ||
Comment 12•17 years ago
|
||
Remove %APPDATA%\Mozilla\Firefox\Profiles if empty and %APPDATA%\Mozilla\Firefox if empty when removing personal data
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
Attachment #315163 -
Flags: review?(benjamin)
Assignee | ||
Updated•17 years ago
|
Attachment #315163 -
Attachment is obsolete: true
Attachment #315163 -
Flags: review?(benjamin)
Assignee | ||
Comment 13•17 years ago
|
||
Remove %APPDATA%\Mozilla\Firefox\Profiles if empty and
%APPDATA%\Mozilla\Firefox if empty when removing personal data
This should be the last of the personal data removal fixes.
Attachment #315185 -
Flags: review?(benjamin)
Updated•17 years ago
|
Attachment #315185 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 14•17 years ago
|
||
Comment on attachment 315185 [details] [diff] [review]
patch rev2
Drivers, this is a simple / safe fix that removes empty directories when
personal data is removed during install. IIRC the removal of empty directories
is one of the requirements on the PRD so I am asking for a1.9.
Attachment #315185 -
Flags: approval1.9?
Comment 15•17 years ago
|
||
Comment on attachment 315185 [details] [diff] [review]
patch rev2
a1.9=beltzner
Attachment #315185 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Comment 16•17 years ago
|
||
Checked in to trunk
Checking in mozilla/toolkit/mozapps/installer/windows/nsis/common.nsh;
/cvsroot/mozilla/toolkit/mozapps/installer/windows/nsis/common.nsh,v <-- common.nsh
new revision: 1.41; previous revision: 1.40
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3
Comment 17•17 years ago
|
||
verified fixed using Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9pre)
Gecko/2008041506 Minefield/3.0pre.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•