343601 - Remove old preferences

Status: RESOLVED FIXED

(Calendar :: Preferences, defect)

Description

[Matthew (lilmatt) Willis]

Nuprefs (bug 333923) has baked for a while. Now that we've announced the end of xpfe support, we can remove the old preferences.

Comment 1

[Matthew (lilmatt) Willis]

Attachment: rev0 - removes old preferences

Comment 2

[Matthew (lilmatt) Willis]

Attachment: rev1 - adds overlays I missed

Comment 3

[Joey Minta]

Comment on attachment 228144 [details] [diff] [review]
rev1 - adds overlays I missed

minused for breaking lightning.

Comment 4

[Matthew (lilmatt) Willis]

Attachment: rev2 - doesn't break lightning

Comment 5

[Matthew (lilmatt) Willis]

Attachment: rev3 - fixes stuff jminta found

Comment 6

[Matthew (lilmatt) Willis]

(In reply to comment #5)
> Created an attachment (id=228826) [edit]
> rev3 - fixes stuff jminta found

jminta found:
 - editCategories.xul was still using prefs.dtd
 - lightning-preferences.xul was defining %preferencesDTD which mail's prefwindow already had

Comment 7

[Joey Minta]

Comment on attachment 228826 [details] [diff] [review]
rev3 - fixes stuff jminta found

 calendar.jar:
 %   content calendar %content/calendar/
 #ifndef MOZ_SUNBIRD
 %   overlay chrome://browser/content/browser.xul chrome://calendar/content/calExtOverlay.xul
 %   overlay chrome://communicator/content/tasksOverlay.xul chrome://calendar/content/calendarOverlay.xul
-%   overlay chrome://communicator/content/pref/preftree.xul chrome://calendar/content/pref/prefOverlay.xul
-%   overlay chrome://communicator/content/pref/pref-appearance.xul chrome://calendar/content/pref/prefOverlay.xul
 %   overlay chrome://messenger/content/mailWindowOverlay.xul chrome://calendar/content/calExtOverlay.xul
 %   overlay chrome://messenger/content/messengercompose/messengercompose.xul chrome://calendar/content/calExtOverlay.xul
 *   content/calendar/contents.rdf (content/contents.rdf)
 *   skin/classic/calendar/contents.rdf (skin/classic/contents.rdf)
 *   skin/modern/calendar/contents.rdf (skin/modern/contents.rdf)
+%   skin calendar modern/1.0 %skin/modern/calendar/
 #endif
 %   skin calendar classic/1.0 %skin/classic/calendar/
-%   skin calendar modern/1.0 %skin/modern/calendar/
 *   content/calendar/applicationUtil.js (content/applicationUtil.js)
The modern stuff definitely isn't relevant to this patch.  We need a followup to deal with the whole #ifndef MOZ_SUNBIRD thing.

r=jminta with that

Comment 8

[Matthew (lilmatt) Willis]

(In reply to comment #7)
> (From update of attachment 228826 [details] [diff] [review] [edit])
> The modern stuff definitely isn't relevant to this patch.
No worries. It was removed from cvs in bug 343990

> We need a follow up to deal with the whole #ifndef MOZ_SUNBIRD thing.
Spun off as bug 344447

-> FIXED