Closed
Bug 1191860
Opened 9 years ago
Closed 6 years ago
Errors in Firefox AutoConfig no longer show an error popup
Categories
(Core :: Preferences: Backend, defect)
Core
Preferences: Backend
Tracking
()
People
(Reporter: mkaply, Assigned: mkaply)
References
Details
(Keywords: regression)
When Autoconfig parsing fails, there used to be an error popup. Now it just fails with no error.
Assignee | ||
Comment 2•9 years ago
|
||
Oddly it works in a build that I build myself, but doesn't work in any shipping builds (nightly, etc.). My build is NOT debug.
Assignee | ||
Comment 3•9 years ago
|
||
The code is here: http://mxr.mozilla.org/mozilla-central/source/extensions/pref/autoconfig/src/nsReadConfig.cpp#68 My guess is that using the promptService this early is failing on a release build for some reason...
Assignee | ||
Comment 4•9 years ago
|
||
Problem started in Firefox 34. Last release that worked https://hg.mozilla.org/mozilla-central/rev/83f519eb1a3a First release that failes https://hg.mozilla.org/mozilla-central/rev/70be728521e3 http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=83f519eb1a3a&tochange=70be728521e3
Comment 5•9 years ago
|
||
Tracking for 43+ and marking all current releases as affected.
status-firefox41:
--- → affected
status-firefox43:
--- → affected
status-firefox44:
--- → affected
tracking-firefox43:
--- → +
tracking-firefox44:
--- → +
tracking-firefox-esr38:
--- → ?
Assignee | ||
Comment 6•9 years ago
|
||
I have been unable to find anything that would break this and I have been unable to debug since if I build a local build (release or debug), it works. It only seems to fail on official builds. (I haven't tried a try server build yet).
Comment 7•9 years ago
|
||
Wontfix for 43, maybe this is not worth tracking if it's been a regression since 34 and no one else is complaining. We can still fix it of course, maybe aiming for the next major ESR (45).
Given that this issue has been wontfix'd for so long and that we are halfway through Beta44 cycle, wontfix'ing for FF44. I don't see the need to track this one any further. Release management team will be happy to uplift a fix when ready.
Comment 9•8 years ago
|
||
This is in 45esr already, dropping the 38 flag.
Assignee | ||
Comment 10•6 years ago
|
||
We're getting rid of the popup anyway in Firefox 62.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•