1389195 - Unport bug 1388946 from C-C: Set extensions.legacy.enabled to true for TB

Status: RESOLVED FIXED

(Thunderbird :: General, enhancement)

Comment 1

[Jorg K (CEST = GMT+2)]

Attachment: 1389195-lecagy-extensions.patch - TB patch

One review will do.

I noticed that |pref("extensions.logging.enabled", false);| was in the file twice, so I took the second copy out. You can see the first copy in the patch.

Comment 2

[Jorg K (CEST = GMT+2)]

Sorry, I didn't to the SM part since I don't know how you structure/group your preferences.

Comment 3

[Jorg K (CEST = GMT+2)]

Let's make this TB only since SM already dealt with this in bug 1373375 (as FRG told me in a PM):
https://hg.mozilla.org/comm-central/rev/ccbd9c80a4ddd761cc7e7c24c3886cf8412bda35#l1.13

I'm not sure what the other stuff is they landed there. FRG, do we need it and/or the confvars.sh change?

Comment 4

[Jorg K (CEST = GMT+2)]

Oh, I forgot to mention, the thing I'm undoing is this:
https://hg.mozilla.org/integration/autoland/rev/45d8844f0124#l1.12

Comment 5

[Frank-Rainer Grahl (:frg)]

Thanks Jorg for the heads up. SeaMonkey is covered in bug 1373375 which we spoke or better mailed about a few weeks ago. You might want to take the build time variable too from it. This will probably be the next step if it is not altogether removed. I read yesterday that the pref will become a compile time variable and I think this one will be it if not renamed.

As far as I see it bug 1373375 is for firefox prefs only. Shouldn'r concern the c-c apps right now. The default is still true for them so technically you won't need to change anything right now.

You already have the "pref("extensions.startupScanScopes", 4)". "extensions.legacy.exceptions" is a legacy exception list. Any add-on in it will not be flagged as legacy. You dealt with it with css but this is low level access and for now honored by toolkit. Not sure if you want it. 

All of this might change till Fx 57 GA but I doubt they will be able to remove things like the default theme from the exception list.

Comment 6

[Jorg K (CEST = GMT+2)]

Attachment: 1389195-lecagy-extensions.patch - TB patch (v2)

Thanks, FRG. I moved the new extension further down just after extensions.startupScanScopes. I also added the confvars.sh.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/df01b55915fa
Allow legacy extensions in TB. r=mkmelin