Closed Bug 248280 Opened 16 years ago Closed 10 years ago
.allow .mailnews" is obsolete and should be removed
currently it's in: http://lxr.mozilla.org/mozilla/source/modules/libpref/src/init/all.js#422 it should be moved to a mailnews specific .js file
Assignee: general → nobody
Component: General → Backend
Product: SeaMonkey → MailNews Core
QA Contact: general → backend
The pref is now obsolete, and should be removed - see bug 504965 comment 8.
Hardware: x86 → All
Target Milestone: --- → Thunderbird 3.0b4
Removing the dead ui from seamonkey. You may want to figure out how much sense it makes to have the single [ ] Browser section there and such, but let's leave that to another bug.
dmose, want to review the m-c patch? dunno who would be a good reviewer for something like this, but as you're the one who implemented the removal of the pref...
Attachment #396141 - Flags: review?(dmose)
Attachment #396141 - Flags: review?(dmose) → review+
Comment on attachment 396141 [details] [diff] [review] mozilla-central proposed fix r=dmose; thanks for the patch!
Attachment #396140 - Flags: superreview?(neil) → superreview+
mozilla-central part checked in changeset: 31791:8e21c74914f7 http://hg.mozilla.org/mozilla-central/rev/8e21c74914f7
Comment on attachment 396140 [details] [diff] [review] comm-central proposed fix Thanks for caring!
Attachment #396140 - Flags: review?(mnyromyr) → review+
Attachment #396140 - Flags: approval-seamonkey2.0?
Attachment #396140 - Flags: approval-seamonkey2.0? → approval-seamonkey2.0+
changeset: 3545:6169983ce2ac http://hg.mozilla.org/comm-central/rev/6169983ce2ac ->FIXED
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Adding fixed-seamonkey2.0 keyword so the queries for approved but not fixed bugs don't catch that bug.
(In reply to comment #3) > Created an attachment (id=396141) [details] > mozilla-central proposed fix Should this be landed on 1.9.1/.2 as well? If so, I guess you should request approval. I saw that some users are confused by finding the pref in about:config in SeaMonkey 2.0, thinking it would be any good to set it to true.
Comment on attachment 396141 [details] [diff] [review] mozilla-central proposed fix Dead pref removal to avoid confusion.
Comment on attachment 396141 [details] [diff] [review] mozilla-central proposed fix approval1.9.2 requests aren't currently being monitored, since we're nearing RC freeze and there are too many outstanding requests, so I'm clearing this request. Feel free to re-request approval if you are confident that it's worth drivers' time to consider whether this non-blocker needs to land for 1.9.2 at this stage.
You need to log in before you can comment on or make changes to this bug.