Closed Bug 702792 Opened 13 years ago Closed 13 years ago

Port |Bug 700201 - Addons that haven't been updated for an extremely long time should not be compatible by default|

Categories

(SeaMonkey :: Preferences, defect)

defect
Not set
normal

Tracking

(seamonkey2.7 fixed, seamonkey2.8 fixed)

RESOLVED FIXED
seamonkey2.8
Tracking Status
seamonkey2.7 --- fixed
seamonkey2.8 --- fixed

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

References

Details

Attachments

(2 files)

This copies two prefs recently introduced by FF in preparation for "compatible by default" extensions, namely:
* extensions.strictCompatibility
* extensions.minCompatibleAppVersion

I copied the former 1:1 which means that the feature is disabled for now and the comment points at the bug which will flip it for FF (which we should depend on, too). The pref was introduced for FF in bug 693901.

FF chose "4.0" for the latter, which matches "2.1" for SM. I think this choice is sensible since not much has changed for us since then, but much changed before that (SM 2.0 didn't make full use of Places etc.). Also, the global pref extensions.minCompatiblePlatformVersion is set to "2.0" which matches that choice.

Both prefs have already made it to Aurora (or, in case of the latter, have received a positive Aurora approval flag) which matches FF's desire to enable the feature for FF 10.
Attachment #574700 - Flags: review?(bugspam.Callek)
Attachment #574700 - Flags: approval-comm-aurora?
Heh, I even copied the typos. ;-)
Comment on attachment 574700 [details] [diff] [review]
patch [Checkin: comment 8]

Based on the approval-request comment, I approve this. don't have the time to actually review in the short-term, but the choices as to min Version, etc. sounds sane.
Attachment #574700 - Flags: review?(iann_bugzilla)
Attachment #574700 - Flags: approval-comm-aurora?
Attachment #574700 - Flags: approval-comm-aurora+
Actually, bug 698653 was just fixed (but not ported to or explicitly requested for Aurora yet), activating compatible-by-default on trunk. I'll provide an additional patch for the below unless bug 698653 gets ported to Aurora really quickly (in which case I'll replace the existing patch here).

-// Enables strict compatibility. To be toggled in bug 698653, to make addons
-// compatibile by default.
-pref("extensions.strictCompatibility", true);
+// Disables strict compatibility, making addons compatible-by-default.
+pref("extensions.strictCompatibility", false);
Comment on attachment 574700 [details] [diff] [review]
patch [Checkin: comment 8]

># HG changeset patch
># Parent 99d3461b94890e32d57a90d0b044322007f172fd
># User Jens Hatlak <jh@junetz.de>
>diff --git a/suite/browser/browser-prefs.js b/suite/browser/browser-prefs.js
>--- a/suite/browser/browser-prefs.js
>+++ b/suite/browser/browser-prefs.js
>@@ -543,16 +543,24 @@ pref("app.update.showInstalledUI", false
> //     versions.
> pref("app.update.incompatible.mode", 0);
> 
> // Extension preferences
> 
> // Enables some extra Extension System Logging (can reduce performance)
> pref("extensions.logging.enabled", false);
> 
>+// Enables strict compatibility. To be toggled in bug 698653, to make addons
Should we mention this bug number, our bug number or both?
>+// compatibile by default.
>+pref("extensions.strictCompatibility", true);
r=me with that answered/addressed
Attachment #574700 - Flags: review?(iann_bugzilla)
Attachment #574700 - Flags: review?(bugspam.Callek)
Attachment #574700 - Flags: review+
Comment on attachment 575544 [details] [diff] [review]
enable compatible-by-default [Checkin: comments 9 and 12]

As per previous comment.
Attachment #575544 - Flags: review?(iann_bugzilla) → review+
(In reply to Ian Neal from comment #5)
> >+// Enables strict compatibility. To be toggled in bug 698653, to make addons
> Should we mention this bug number, our bug number or both?

Effectively the bug number will only be on Aurora, since my second patch removes it for trunk right away. And on Aurora I think it's better to have the same bug number as in the corresponding FF prefs file, from where I copied it. And once this lands, this bug's number can easily be found using hg log or MXR/blame.

[Note that I decided to fix the typo upon checkin, so the second patch won't apply cleanly anymore.]

BTW note bug 698653 comments 12 and 14:
"If you had the setting "extensions.checkCompatibility.nightly" set false, you must remove this setting for the "extensions.strictCompatibility" set false to take effect."
"Apparently, the extensions.checkCompatibility.nightly;false is inserted and set to false when the Addon Compatibility Reporter is installed or enabled."
Comment on attachment 575544 [details] [diff] [review]
enable compatible-by-default [Checkin: comments 9 and 12]

http://hg.mozilla.org/comm-central/rev/3769878d4f39
Attachment #575544 - Attachment description: enable compatible-by-default → enable compatible-by-default [Checkin: comment 9]
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.8
Comment on attachment 575544 [details] [diff] [review]
enable compatible-by-default [Checkin: comments 9 and 12]

Requesting approval since bug 698653 just landed on Aurora (cf. comments 18 and 19 over there).
Attachment #575544 - Flags: approval-comm-aurora?
Comment on attachment 575544 [details] [diff] [review]
enable compatible-by-default [Checkin: comments 9 and 12]

a=me for comm-aurora.
Attachment #575544 - Flags: approval-comm-aurora? → approval-comm-aurora+
Comment on attachment 575544 [details] [diff] [review]
enable compatible-by-default [Checkin: comments 9 and 12]

http://hg.mozilla.org/releases/comm-aurora/rev/6f2ecdf14a0d
Attachment #575544 - Attachment description: enable compatible-by-default [Checkin: comment 9] → enable compatible-by-default [Checkin: comments 9 and 12]
I guess we need to add a line or two to relnotes for this feature. ;-)
FYI, I'm doing Bug 711275 for our 2.7 beta's as well, so we can be sure to TEST THIS!
Depends on: 700201
Keywords: relnote
You need to log in before you can comment on or make changes to this bug.