Closed
Bug 818468
Opened 12 years ago
Closed 11 years ago
Langpacks bundled in distribution/extensions are registered but disabled even if shown enabled
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
VERIFIED
FIXED
mozilla21
People
(Reporter: glazou, Assigned: mossop)
References
Details
Attachments
(2 files)
7.72 KB,
patch
|
Unfocused
:
review+
lsblakk
:
approval-mozilla-aurora-
|
Details | Diff | Splinter Review |
11.44 KB,
patch
|
lsblakk
:
approval-mozilla-aurora+
bajaj
:
approval-mozilla-beta+
bajaj
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
This seems to be a regression that started with restartless langpacks. Before that change, langpacks added in XPI form into (appdir)/extensions were registered, listed by the add-ons manager and enabled after restart. After the change, they're registered, listed by the add-ons manager as enabled but they are in fact disabled... Opening the add-ons manager, clicking on "Disable" and then "Enable" enables the add-on. I verified the change is the culprit forcing bootstrapping in my tree turning the 'false' into a 'true' in http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/extensions/XPIProvider.jsm#766
Reporter | ||
Comment 1•12 years ago
|
||
Correction, this is (appdir)/distribution/extensions. Sorry for that.
Summary: Langpacks bundled in /extensions are registered but disabled even if shown enabled → Langpacks bundled in distribution/extensions are registered but disabled even if shown enabled
Comment 2•12 years ago
|
||
Do dictionaries work there?
Reporter | ||
Comment 3•12 years ago
|
||
(In reply to Axel Hecht [:Pike] from comment #2) > Do dictionaries work there? I just tried. The behaviour is exactly the same.
Comment 4•11 years ago
|
||
This bug (together with bug 677092) appears to have caused https://launchpad.net/bugs/1094376. We'll need to find an owner for FF19.
tracking-firefox19:
--- → ?
Comment 5•11 years ago
|
||
Can someone please confirm if that is a dupe to bug 806198?
Assignee | ||
Comment 6•11 years ago
|
||
(In reply to Wolfgang Rosenauer [:wolfiR] from comment #5) > Can someone please confirm if that is a dupe to bug 806198? Maybe, probably no way to tell for sure unless you still have your profile files from before you got it working in bug 806198 or have a way to reproduce
Assignee: nobody → dtownsend+bugmail
Updated•11 years ago
|
tracking-firefox18:
--- → ?
Comment 8•11 years ago
|
||
QA - can you confirm the steps to reproduce in bug 806198?
Keywords: qawanted
Assignee | ||
Comment 9•11 years ago
|
||
My testing is suggesting that this is worse than first thought and that language packs don't survive a restart right now. Can anyone else confirm/deny that?
Assignee | ||
Comment 10•11 years ago
|
||
(In reply to Dave Townsend (:Mossop) from comment #9) > My testing is suggesting that this is worse than first thought and that > language packs don't survive a restart right now. Can anyone else > confirm/deny that? Ok, I've verified that I was wrong here, some failsafe code from bug 619607 was stepping in and making sure that as long as langpacks were installed through the UI they work correctly.
Assignee | ||
Comment 11•11 years ago
|
||
It's important to always call loadBootstrapScope for a restartless add-on as that takes care of registering it in bootstrappedAddons which is what we use to load these add-ons on startup. Bug 677092 skipped that for langpacks though it wasn't obvious because the code in 619607 was forcing the add-on into bootstrappedAddons after a restart but only if the langpack was installed through the UI. Installing another add-on later probably causes Firefox to pick up the faulty langpacks too. Note I'm also setting bootstrapScope for the add-on just to stop us calling back into loadBootstrapScope later, it doesn't matter much what we set it to. Added tests verifying that langpacks are properly unloaded on the first shutdown (they weren't before) and that those detected in the profile or distribution direction (the same really but lets be thorough) get started correctly and survive a restart.
Attachment #699533 -
Flags: review?(bmcbride)
Assignee | ||
Comment 12•11 years ago
|
||
Also spinning this through try but I don't expect there to be any issues.
Comment 13•11 years ago
|
||
Does anyone have an example from AMO I use to reproduce this problem and see if it is happening in Win/Mac? If do this from AMO will I not be able to reproduce the problem (according to comment #10)?
Assignee | ||
Comment 14•11 years ago
|
||
(In reply to juan becerra [:juanb] from comment #13) > Does anyone have an example from AMO I use to reproduce this problem and see > if it is happening in Win/Mac? If do this from AMO will I not be able to > reproduce the problem (according to comment #10)? The way to reproduce this is to take a langpack xpi, drop it into <profile>/extensions/<id>.xpi and then start Firefox. It should show in the add-ons manager as enabled but locale switcher etc. won't list it.
Comment 15•11 years ago
|
||
I've been trying to reproduce this with several language packs using the instructions in comment #14, but when I start the browser I'm prompted to "modify Firefox with the following add-on" and in about:addons the language pack appears disabled.
Assignee | ||
Comment 16•11 years ago
|
||
(In reply to juan becerra [:juanb] from comment #15) > I've been trying to reproduce this with several language packs using the > instructions in comment #14, but when I start the browser I'm prompted to > "modify Firefox with the following add-on" and in about:addons the language > pack appears disabled. Bah, forgot about that sorry. Ok instead put them into <appdir>/distribution/extensions/<id>.xpi and then delete your profile and start Firefox.
Comment 17•11 years ago
|
||
After that last suggestion I see the language pack in about:addons, and it looks enabled, but when I install the Locale Switcher addon and restart to see if the language pack is enabled, the language pack is listed and it works if I select it and restart. I tried doing something similar with a dictionary, but that never got listed in about:addons. I am not sure how easy it would be for an average user to get into this situation.
Comment 18•11 years ago
|
||
Considering QA feedback regarding the uncertainty of this user scenario being uncommon and given the lack of early feedback on support, we are planning to unthrottle tomorrow as scheduled. We will react accordingly if the landscape changes based on user feedback in the coming few days.
Comment 19•11 years ago
|
||
Just to explain my usecase where all of my openSUSE users are affected: We are shipping language packs (self created from the repos along with Firefox) in $(appdir)/extensions (not xpi but flat files what shouldn't make a difference). In addition we have intl.locale.matchOS set to true so Firefox comes up with the language the user uses in his environment automatically. With 18.0 this is currently broken until the users disables/enables the language in the addon manager. Then it works until the next update.
Comment 20•11 years ago
|
||
Same here. All users of Firefox langpacks on Arch Linux are affected as well. We ship the xpi files in (appdir)/extensions.
Comment 21•11 years ago
|
||
Same on Debian.
Assignee | ||
Comment 22•11 years ago
|
||
To be clear, we're planning to have the fix for this in Firefox 19 and the recommendation for Linux distros that are affected by this is to back out bug 677092 from their Firefox 18 builds.
Updated•11 years ago
|
Comment 23•11 years ago
|
||
If we do spin an 18.0.1, we'll back out bug 677092.
Comment 24•11 years ago
|
||
That may even be the best idea for FF19, if this patch carries any risk.
tracking-firefox20:
--- → +
Updated•11 years ago
|
Attachment #699533 -
Flags: review?(bmcbride) → review+
Updated•11 years ago
|
Status: NEW → ASSIGNED
Comment 25•11 years ago
|
||
(In reply to Dave Townsend (:Mossop) from comment #14) > The way to reproduce this is to take a langpack xpi, drop it into > <profile>/extensions/<id>.xpi and then start Firefox. It should show in the > add-ons manager as enabled but locale switcher etc. won't list it. (In reply to Dave Townsend (:Mossop) from comment #16) > Bah, forgot about that sorry. Ok instead put them into > <appdir>/distribution/extensions/<id>.xpi and then delete your profile and > start Firefox. I don't see the language pack in addons manager after copying to <appdir>/distribution/extensions/<id>.xpi, both on FF 17 and 18. What am I doing wrong? Should I enabled something in about:config ?
Comment 26•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c5c4cff29edd
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Assignee | ||
Comment 27•11 years ago
|
||
(In reply to Paul Silaghi [QA] from comment #25) > (In reply to Dave Townsend (:Mossop) from comment #14) > > The way to reproduce this is to take a langpack xpi, drop it into > > <profile>/extensions/<id>.xpi and then start Firefox. It should show in the > > add-ons manager as enabled but locale switcher etc. won't list it. > > (In reply to Dave Townsend (:Mossop) from comment #16) > > Bah, forgot about that sorry. Ok instead put them into > > <appdir>/distribution/extensions/<id>.xpi and then delete your profile and > > start Firefox. > > I don't see the language pack in addons manager after copying to > <appdir>/distribution/extensions/<id>.xpi, both on FF 17 and 18. What am I > doing wrong? Should I enabled something in about:config ? Did you delete your profile before starting Firefox?
Comment 28•11 years ago
|
||
Yes, I removed all the profiles in %appdata%. Why there isn't by default an "<appdir>/distribution/" folder in my FF 18 release dir, but there is one in the FF 18b6 dir ?
Assignee | ||
Comment 29•11 years ago
|
||
(In reply to Paul Silaghi [QA] from comment #28) > Yes, I removed all the profiles in %appdata%. > > Why there isn't by default an "<appdir>/distribution/" folder in my FF 18 > release dir, but there is one in the FF 18b6 dir ? We ship testpilot with beta and aurora builds but not release builds.
Comment 30•11 years ago
|
||
Dave, comment# 23 seems to be a safer approach to get this resolved for a possible Fx18.0.1 in terms of avoiding any surprising regressions which may come up if we consider the fwd fix given the limited testing we have had . Can you please help prepare a backout patch in preparation and nominate for for approval-mozilla-release ?
Assignee | ||
Comment 31•11 years ago
|
||
(In reply to bhavana bajaj [:bajaj] from comment #30) > Dave, comment# 23 seems to be a safer approach to get this resolved for a > possible Fx18.0.1 in terms of avoiding any surprising regressions which may > come up if we consider the fwd fix given the limited testing we have had . > > Can you please help prepare a backout patch in preparation and nominate for > for approval-mozilla-release ? Running through try now: https://tbpl.mozilla.org/?tree=Try&rev=b595c1c3589f
Comment 32•11 years ago
|
||
(In reply to Dave Townsend (:Mossop) from comment #31) > (In reply to bhavana bajaj [:bajaj] from comment #30) > > Dave, comment# 23 seems to be a safer approach to get this resolved for a > > possible Fx18.0.1 in terms of avoiding any surprising regressions which may > > come up if we consider the fwd fix given the limited testing we have had . > > > > Can you please help prepare a backout patch in preparation and nominate for > > for approval-mozilla-release ? > > Running through try now: https://tbpl.mozilla.org/?tree=Try&rev=b595c1c3589f Dave, can we please have the patch landed & nominated for approval-mozilla-release and approval-mozilla-beta as per comment# 24 ?
Assignee | ||
Comment 33•11 years ago
|
||
Straight backout, applied cleanly. [Approval Request Comment] Regression caused by (bug #): 818468 User impact if declined: Broken langpacks on linux distros Testing completed (on m-c, etc.): It's a backout so this was the status quo for Fx4-17 Risk to taking this patch (and alternatives if risky): None
Attachment #702450 -
Flags: approval-mozilla-release?
Attachment #702450 -
Flags: approval-mozilla-beta?
Assignee | ||
Comment 34•11 years ago
|
||
Comment on attachment 699533 [details] [diff] [review] patch rev 1 I think we should take the real fix on aurora once it's been through a few more days on nightly. It's likely safe enough for beta too but I'm in no rush to push it there unless someone else thinks it is urgent.
Attachment #699533 -
Flags: approval-mozilla-aurora?
Comment 35•11 years ago
|
||
Comment on attachment 702450 [details] [diff] [review] backout patch Please land as soon as possible as we are going to build with FF19 beta & 18.0.1 very soon . Thanks !
Attachment #702450 -
Flags: approval-mozilla-release?
Attachment #702450 -
Flags: approval-mozilla-release+
Attachment #702450 -
Flags: approval-mozilla-beta?
Attachment #702450 -
Flags: approval-mozilla-beta+
Assignee | ||
Comment 36•11 years ago
|
||
Backed out: https://hg.mozilla.org/releases/mozilla-beta/rev/65ec01c62495 https://hg.mozilla.org/releases/mozilla-release/rev/4fe6923afe8a
status-firefox18:
--- → fixed
status-firefox19:
--- → fixed
Comment 37•11 years ago
|
||
Comment on attachment 699533 [details] [diff] [review] patch rev 1 Given the backouts on beta/release I think this should ride the trains, seeing it's not a regression. I'll approve the backout patch for Aurora as well.
Attachment #699533 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora-
Updated•11 years ago
|
Attachment #702450 -
Flags: approval-mozilla-aurora+
Reporter | ||
Comment 38•11 years ago
|
||
Just a big thank you for fixing this, the issue was a nightmare for me working on BlueGriffon!
Comment 39•11 years ago
|
||
(In reply to juan becerra [:juanb] from comment #17) > After that last suggestion I see the language pack in about:addons, and it > looks enabled, but when I install the Locale Switcher addon and restart to > see if the language pack is enabled, the language pack is listed and it > works if I select it and restart. I tested this with FF en-US, german language pack xpi, Ubuntu 12.04. I confirm the same behavior on FF 18, 18.0.1 and 19b2. I think the initial problem couldn't be reproduced using the STR in comment 14, 16 because the Locale Switcher, Quick Locale Switcher addons require browser restart. Anyway, I think this bug can be marked as verified considering reporter's comment 38.
Comment 40•11 years ago
|
||
Daniel, can you confirm this bug no longer reproduces for you in both Firefox 18.0.1 and Firefox 19.0b2?
Comment 41•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/08c0832fbb6e (I think disabled is more accurate since this was backed out on the branches)
status-firefox20:
--- → disabled
status-firefox21:
--- → fixed
Comment 42•11 years ago
|
||
Marking verified fixed based on comment 38 and 39. Please reopen if you can reproduce this bug in a current build.
You need to log in
before you can comment on or make changes to this bug.
Description
•