Closed Bug 343990 Opened 18 years ago Closed 18 years ago

Remove unused files

Categories

(Calendar :: Sunbird Only, defect)

Product:
Calendar
Component:
Sunbird Only
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: mattwillis, Assigned: mattwillis)

Details

Attachments

(2 files, 1 obsolete file)

rev1 - does a little more cleanup after mxr searches (CHECKED IN)
7.91 KB, patch
mvl
: first-review+
Details | Diff | Splinter Review
rev0 - stops packaging modern skin
4.30 KB, patch
mvl
: first-review+
Details | Diff | Splinter Review 
Let's remove some old files.
In addition to this patch, the following files will be removed entirely from the tree.

R calendar/resources/content/about.html
R calendar/resources/content/about.xul
R calendar/resources/content/calExtOverlay.js
R calendar/resources/content/calExtOverlay.xul
R calendar/resources/content/calendar.xul
R calendar/resources/content/contents.rdf
R calendar/resources/content/menuOverlay.xul
R calendar/resources/content/sunbird_small.png
R calendar/resources/skin/classic/contents.rdf
R calendar/resources/skin/modern/contents.rdf


Also please note that this bug does _NOT_ attempt to remove the old xpfe preferecnes. That is bug 343601
Attachment #228568 - Flags: first-review?(jminta)
lxr searches for the removed files.
Note that calendar.xul's result is misleading since we're _not_ removing Sunbird's calendar.xul


R calendar/resources/content/about.html
http://landfill.mozilla.org/mxr-test/mozilla/search?string=about.html&find=%2Fcalendar%2F&filter=

R calendar/resources/content/about.xul
http://landfill.mozilla.org/mxr-test/mozilla/search?string=about.xul&find=%2Fcalendar%2F&filter=

R calendar/resources/content/calExtOverlay.js
http://landfill.mozilla.org/mxr-test/mozilla/search?string=calExtOverlay.js&find=%2Fcalendar%2F&filter=

R calendar/resources/content/calExtOverlay.xul
http://landfill.mozilla.org/mxr-test/mozilla/search?find=%2Fcalendar%2F&string=calExtOverlay.xul

R calendar/resources/content/calendar.xul
http://landfill.mozilla.org/mxr-test/mozilla/search?string=calendar.xul&find=%2Fcalendar%2F&filter=

R calendar/resources/content/contents.rdf
R calendar/resources/skin/classic/contents.rdf
R calendar/resources/skin/modern/contents.rdf
http://landfill.mozilla.org/mxr-test/mozilla/search?string=contents.rdf&find=%2Fcalendar%2F&filter=

R calendar/resources/content/menuOverlay.xul
http://landfill.mozilla.org/mxr-test/mozilla/search?string=menuOverlay.xul&find=%2Fcalendar%2F&filter=

R calendar/resources/content/sunbird_small.png
http://landfill.mozilla.org/mxr-test/mozilla/search?string=sunbird_small.png&find=%2Fcalendar%2F&filter=


The previous list of removed files remains the same.
Attachment #228568 - Attachment is obsolete: true
Attachment #228682 - Flags: first-review?
Attachment #228568 - Flags: first-review?(jminta)
Attachment #228682 - Flags: first-review? → first-review?(mvl)
Comment on attachment 228682 [details] [diff] [review]
rev1 - does a little more cleanup after mxr searches (CHECKED IN)

r=mvl
Let's hope we don't break the part of the world we still care about.
Attachment #228682 - Flags: first-review?(mvl) → first-review+
rev 1 patch and list of removed files landed on MOZILLA_1_8_BRANCH and trunk
Also remove the following:

/mozilla/calendar/resources/skin/modern/alertDialog.css
/mozilla/calendar/resources/skin/modern/all_day_event.png
/mozilla/calendar/resources/skin/modern/all_day_event_down.png
/mozilla/calendar/resources/skin/modern/all_day_event_hover.png
/mozilla/calendar/resources/skin/modern/btn1.png
/mozilla/calendar/resources/skin/modern/calendar-16.gif
/mozilla/calendar/resources/skin/modern/calendar-button.css
/mozilla/calendar/resources/skin/modern/calendar.css
/mozilla/calendar/resources/skin/modern/calendar_topbar.gif
/mozilla/calendar/resources/skin/modern/calendartoolbar.png
/mozilla/calendar/resources/skin/modern/calendartoolbar_small.png
/mozilla/calendar/resources/skin/modern/dialogOverlay.css
/mozilla/calendar/resources/skin/modern/dot.png
/mozilla/calendar/resources/skin/modern/dot_selected.png
/mozilla/calendar/resources/skin/modern/eventDialog.css
/mozilla/calendar/resources/skin/modern/overlay.css
/mozilla/calendar/resources/skin/modern/pageupdown.png
/mozilla/calendar/resources/skin/modern/prefs.css
/mozilla/calendar/resources/skin/modern/prevnextarrow.png
/mozilla/calendar/resources/skin/modern/selectAddresses.css
/mozilla/calendar/resources/skin/modern/synch_animated.gif
/mozilla/calendar/resources/skin/modern/taskbar-cal-act.gif
/mozilla/calendar/resources/skin/modern/taskbar-cal.gif
/mozilla/calendar/resources/skin/modern/taskbar-calalarm-act.gif
/mozilla/calendar/resources/skin/modern/taskbar-calalarm.gif
/mozilla/calendar/resources/skin/modern/week-view-corner.png
/mozilla/calendar/resources/skin/modern/datetimepickers/calendar_down.png
/mozilla/calendar/resources/skin/modern/datetimepickers/calendar_hover.png
/mozilla/calendar/resources/skin/modern/datetimepickers/clock_down.png
/mozilla/calendar/resources/skin/modern/datetimepickers/clock_hover.png
/mozilla/calendar/resources/skin/modern/datetimepickers/datepicker.css
/mozilla/calendar/resources/skin/modern/datetimepickers/datetimepickers.css
/mozilla/calendar/resources/skin/modern/datetimepickers/left-arrow-hover.gif
/mozilla/calendar/resources/skin/modern/datetimepickers/left-arrow.gif
/mozilla/calendar/resources/skin/modern/datetimepickers/minimonth.css
/mozilla/calendar/resources/skin/modern/datetimepickers/right-arrow-hover.gif
/mozilla/calendar/resources/skin/modern/datetimepickers/right-arrow.gif
/mozilla/calendar/resources/skin/modern/datetimepickers/timepicker.css
/mozilla/calendar/resources/skin/modern/unifinder/checkbox_checked.png
/mozilla/calendar/resources/skin/modern/unifinder/checkbox_unchecked.png
/mozilla/calendar/resources/skin/modern/unifinder/priority_header.png
/mozilla/calendar/resources/skin/modern/unifinder/priority_high.png
/mozilla/calendar/resources/skin/modern/unifinder/priority_low.png

Since all of these will show up in lxr as "still in use" since we are retaining most of the "classic" skin, I'm not including those links.
Attachment #228774 - Flags: first-review?
Attachment #228774 - Flags: first-review? → first-review?(mvl)
Attachment #228682 - Attachment description: rev1 - does a little more cleanup after mxr searches → rev1 - does a little more cleanup after mxr searches (CHECKED IN)
Comment on attachment 228774 [details] [diff] [review]
rev0 - stops packaging modern skin

you also need to fix /calendar/installer/windows/browser.jst to not try to install modern.
r=mvl with that fixed
Attachment #228774 - Flags: first-review?(mvl) → first-review+
Found another one to delete:

/mozilla/calendar/resources/content/calendarManagement.js

r?mvl
I very much assume you wanted to say calendarManager.js
r=mvl if that's the case.
Closing this as we got rid of a bunch o' cruft
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: