Closed Bug 693698 Opened 13 years ago Closed 13 years ago

3rd Party add-ons from the application folder are not disabled on upgrade

Categories

(Toolkit :: Add-ons Manager, defect)

8 Branch
x86
Windows 7
defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla10
Tracking Status
firefox8 + fixed
firefox9 + fixed

People

(Reporter: verdi, Assigned: mossop)

References

Details

(Keywords: verified-beta, Whiteboard: [qa!])

Attachments

(5 files)

1. I did a clean install of Firefox 7.0b6 in Windows 7. It had a fresh profile. 
2. I then downloaded and installed a variety of programs that bundle Firefox add-ons:
* Skype - http://skype.com
* Weatherbug - http://weatherbug.com
* Free FLV player for Windows - http://applian.com/flvplayer/
* Advanced SystemCare - http://www.iobit.com/advancedsystemcareper.html
* PDFCreator - http://www.pdfforge.org/pdfcreator
* Defraggler - http://www.piriform.com/defraggler/download

3. After restarting Firefox 7 I had a lot of third party add-ons (some were disabled because they weren't compatible).
4. I updated Firefox 7 to Firefox 8. During the update process my add-ons were checked and none of them were identified as third party. All were checked to keep and to be active (some were deactivated because they weren't compatible)
5. Firefox 8 still has a lot of third party add-ons.

I have the Program directory and the profile zipped up. I can upload them if they'd be helpful.
Can you attach a copy of extensions.sqlite from your profile.
Attachment #566286 - Attachment mime type: text/plain → application/octet-stream
nom'ing for tracking Fx8 -- we should make sure this isn't broken before advertising it as a Firefox 8 feature.
I updated my Ubuntu VM from Fx 7 to 8 and the Ubuntu add-on was flagged as third party on upgrade.
Thanks for the updates, the problems here are all because the third party add-ons are getting installed to the profile or application directory, I'm trying to work out whether it's possible to solve this problem.
As mentioned by Dave over on bug 693459 he wants to have it also tracked for Aurora.
(In reply to Dave Townsend (:Mossop) from comment #7)
> Thanks for the updates, the problems here are all because the third party
> add-ons are getting installed to the profile or application directory, I'm
> trying to work out whether it's possible to solve this problem.

If all else fails, can we ship a list of extensions that we know are being installed by third parties even though they look from the point of view of the add-on management code like extensions the user has installed?
For reference the following is the install location of all the add-ons listed in attachment 566286 [details]:

app-profile|Ask Toolbar
app-profile|Complitly - Speed up your search with your personal search suggestions tool
app-profile|Freecorder Toolbar
app-profile|YouTube Video Download Wizard
app-global|Default
app-global|Skype Click to Call
app-global|IObit Toolbar
app-global|pdfforge Toolbar
app-global|Widgi Toolbar Platform
Whiteboard: [qa+]
I'm splitting this up so this bug covers third-party add-ons in the application folder and bug 694575 covers those in the profile folder.
Summary: 3rd Party Add-ons are not disabled on upgrade → 3rd Party add-ons from the application folder are not disabled on upgrade
Assignee: nobody → dtownsend
Blocks: 693743
Status: NEW → ASSIGNED
No longer blocks: 693743
Attached patch patch rev 1Splinter Review
This makes the upgrade UI class any non-themes in the application directory as third party installs and updates the testcase to match. Distribution add-ons remain as they were.
Attachment #567775 - Flags: review?(bmcbride)
Attachment #567775 - Flags: review?(bmcbride) → review+
https://hg.mozilla.org/mozilla-central/rev/064189e08af2

adding [inbound] is no more needed (indeed it's actually discouraged)
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [qa+][inbound] → [qa+]
Target Milestone: --- → mozilla10
Comment on attachment 567775 [details] [diff] [review]
patch rev 1

I think we really want this for completeness in the feature. Yes, it's late. Without this change, we're far less effective when we help users disable unwanted add-ons.
Attachment #567775 - Flags: approval-mozilla-beta?
If it's not approved for Aurora, those who will update from 3.6 to 9 won't have this change.
Attachment #567775 - Flags: approval-mozilla-aurora?
Comment on attachment 567775 [details] [diff] [review]
patch rev 1

---------------------------------[ Triage Comment ]---------------------------------

Approving for 8beta and 9aurora as this is a good fix for a new feature and was approved via in person conversations.

Please land ASAP.
Attachment #567775 - Flags: approval-mozilla-beta?
Attachment #567775 - Flags: approval-mozilla-beta+
Attachment #567775 - Flags: approval-mozilla-aurora?
Attachment #567775 - Flags: approval-mozilla-aurora+
This is awesome!!
Mozilla/5.0 (Windows NT 6.1; rv:9.0) Gecko/20100101 Firefox/9.0
Mozilla/5.0 (Windows NT 5.1; rv:9.0) Gecko/20100101 Firefox/9.0

Verified in Firefox9Beta3, Firefox Aurora and Nightly on Windows XP, 7. Switched the shownselectionUI pref to enable the UI dialog after an upgrade. Used Pdfforce Tollbar, Widgi Toolbar Platform, AVG Safe search. All these are dsiplayed as Third Party in the selection dialog after an upgrade.
Status: RESOLVED → VERIFIED
Whiteboard: [qa+] → [qa!]
Keywords: verified-beta
Flags: in-litmus?(virgil.dicu)
Flags: in-litmus?(virgil.dicu) → in-litmus+
You need to log in before you can comment on or make changes to this bug.