Closed
Bug 286380
Opened 16 years ago
Closed 16 years ago
chrome.manifest not removed when an extension is uninstalled
Categories
(Toolkit :: Add-ons Manager, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
References
Details
Attachments
(2 obsolete files)
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050315 Firefox/1.0+ Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050315 Firefox/1.0+ It appears that the chrome.manifest file that was introduced by bug 278534 and bug 283352 didn't include code to remove the chrome.manifest when an extension is uninstalled (it is removed during an upgrade and then replaced with a new one) which will then prevent the removal of the directory used by the uninstalled directory. Reproducible: Always Steps to Reproduce: 1. Install, restart, uninstall an extension Actual Results: The chrome.manifest remains Expected Results: The chrome.manifest is removed and thereby allows the code in nsExtensionManager.js to remove the no longer used directory.
![]() |
Assignee | |
Comment 1•16 years ago
|
||
This patch removes the chrome.manifest file when a theme or extension is uninstalled. There are still issues with uninstalling themes as noted in the patch due to bug 251026 which prevents a theme from being uninstalled when it has a preview or icon in its jar.
![]() |
Assignee | |
Updated•16 years ago
|
Attachment #177594 -
Attachment is obsolete: true
Attachment #177594 -
Flags: review?(benjamin)
![]() |
Assignee | |
Comment 2•16 years ago
|
||
Attachment #177597 -
Flags: review?(benjamin)
Comment 3•16 years ago
|
||
Comment on attachment 177597 [details] [diff] [review] patch (fixed indent) r=me, however please hold off landing this patch until Ben lands some semi-significant extension manager reworking
Attachment #177597 -
Flags: review?(benjamin) → review+
![]() |
Assignee | |
Comment 4•16 years ago
|
||
Not a problem. I suspect that the changes with Ben's patch from bug 286034 will change the structure as well as flow significantly enough to make this patch as well as the patch to bug 251026 and bug 252418 at best to not be need since the functionality can be included in the rewrite and at worse require additional rewrite of the code if these steps aren't taken into account in the flow.
Comment 5•16 years ago
|
||
Should this bug be set as a blocker for Bug 248298 ?
![]() |
Assignee | |
Comment 6•16 years ago
|
||
I don't believe it is necessary to do so. I think we will know better after the patch in bug 286034 is checked in and it does appear that it will fix bug 248298 and this one.
Depends on: eminstall
![]() |
Assignee | |
Comment 7•16 years ago
|
||
Fixed by the checkin for bug 286034
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
![]() |
Assignee | |
Updated•16 years ago
|
Attachment #177597 -
Attachment is obsolete: true
Attachment #177597 -
Flags: review+
Updated•13 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•