AddonInternal.hasBinaryComponents doesn't affect AddonInternal.appDisabled until after initial install

VERIFIED FIXED in Firefox 10

Status

()

Toolkit
Add-ons Manager
P1
normal
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: Unfocused, Assigned: Unfocused)

Tracking

({verified-aurora})

10 Branch
mozilla11
verified-aurora
Points:
---
Bug Flags:
in-testsuite +
in-litmus -

Firefox Tracking Flags

(firefox10 fixed)

Details

(Whiteboard: [qa!])

Attachments

(1 attachment)

Addons with binary components that aren't strictly compatible are still able to get installed.

Cause: loadManifestFromRDF() sets appDisabled based on isUsableAddon(). However, hasBinaryComponents isn't determined until after loadManifestFromRDF() returns, so it's not taken into account until after the addon is installed.



(Thanks to Virgil in QA for catching this.)
Created attachment 574855 [details] [diff] [review]
Patch v1
Attachment #574855 - Flags: review?(dtownsend)
Attachment #574855 - Flags: review?(dtownsend) → review+
https://hg.mozilla.org/integration/fx-team/rev/6b2a7c8e58de
Flags: in-testsuite+
Flags: in-litmus-
Target Milestone: --- → mozilla11
Comment on attachment 574855 [details] [diff] [review]
Patch v1

Same deal as bug 700201 - part of the big compatible-by-default push, which isn't enabled on Aurora yet (though it will soon be on central!), but would like to get this in so QA can go over it and to avoid bitrot.
Attachment #574855 - Flags: approval-mozilla-aurora?
Keywords: qawanted
Comment on attachment 574855 [details] [diff] [review]
Patch v1

Please land on M-C (currently on fx-team) as well as Aurora.
Attachment #574855 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/mozilla-central/rev/6b2a7c8e58de
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
https://hg.mozilla.org/releases/mozilla-aurora/rev/b621aa998435
status-firefox10: affected → fixed
Mozilla/5.0 (Windows NT 5.1; rv:11.0a1) Gecko/20111127 Firefox/11.0a1
Mozilla/5.0 (Windows NT 5.1; rv:10.0a2) Gecko/20111127 Firefox/10.0a2

https://addons.mozilla.org/en-US/firefox/addon/foxytunes/?src=ss
https://addons.mozilla.org/en-US/firefox/addon/grafx-bot/?src=ss

Verified Aurora and Nightly on Ubuntu 11.10, Windows XP, 7, Mac OS 10.6.

Binary add-ons can no longer be installed with strictCompatibility enabled.
Status: RESOLVED → VERIFIED
Keywords: qawanted → verified-aurora
Whiteboard: [qa!]
You need to log in before you can comment on or make changes to this bug.