Bug 507700 - WARNING: Illegal character in window name for Calendar. r=philipp
authorMartin Schroeder <mschroeder@mozilla.x-home.org>
Wed, 09 Sep 2009 13:04:33 +0200
changeset 3600 60c2cd3ddeaf40f152b559cb71fccde06c8a188f
parent 3599 a98db41b5ca737b647d2a7964cb7d6bffee04fa6
child 3601 2d36df75680d0c4898e22b55aff8b91368dc9b55
push id2916
push usermschroeder@mozilla.x-home.org
push dateWed, 09 Sep 2009 11:04:50 +0000
treeherdercomm-central@60c2cd3ddeaf [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersphilipp
bugs507700
Bug 507700 - WARNING: Illegal character in window name for Calendar. r=philipp
calendar/base/content/dialogs/calendar-alarm-dialog.js
calendar/base/src/calCalendarManager.js
calendar/base/src/calUtils.js
calendar/providers/gdata/content/gdata-migration.js
calendar/resources/content/calendar.js
--- a/calendar/base/content/dialogs/calendar-alarm-dialog.js
+++ b/calendar/base/content/dialogs/calendar-alarm-dialog.js
@@ -182,17 +182,17 @@ function updateRelativeDates() {
  * @param event           The event used to determine the position of the popup
  * @param aContainerItem  The container item as described above
  */
 function openSnoozeWindow(event, aContainerItem) {
     const uri = "chrome://calendar/content/calendar-alarm-snooze-popup.xul";
     var pos = ",left=" + (event.target.boxObject.screenX - 3) +
              ",top=" + (event.target.boxObject.screenY + event.target.boxObject.height - 3);
     window.openDialog(uri,
-                      "Calendar:AlarmSnoozePopup",
+                      "CalendarAlarmSnoozePopup",
                       "chrome,dependent=yes,titlebar=no" + pos,
                       aContainerItem);
 }
 
 /**
  * Function to snooze all alarms the given number of minutes.
  *
  * @param aDurationMinutes    The duration in minutes
--- a/calendar/base/src/calCalendarManager.js
+++ b/calendar/base/src/calCalendarManager.js
@@ -297,22 +297,22 @@ calCalendarManager.prototype = {
         // Now find a window. The best choice would be the most recent
         // addons window, otherwise the most recent calendar window, or we
         // create a new toplevel window.
         let wm = Components.classes["@mozilla.org/appshell/window-mediator;1"]
                            .getService(Components.interfaces.nsIWindowMediator);
         let win = wm.getMostRecentWindow("Extension:Manager") ||
                   cal.getCalendarWindow();
         if (win) {
-            win.openDialog(uri, "Calendar:ProviderUninstallDialog", features, args);
+            win.openDialog(uri, "CalendarProviderUninstallDialog", features, args);
         } else {
             // Use the window watcher to open a parentless window.
             let ww = Components.classes["@mozilla.org/embedcomp/window-watcher;1"]
                                .getService(Components.interfaces.nsIWindowWatcher);
-            ww.openWindow(null, uri, "Calendar:ProviderUninstallWindow", features, args);
+            ww.openWindow(null, uri, "CalendarProviderUninstallWindow", features, args);
         }
 
 
         // Now that we are done, check if the dialog was accepted or canceled.
         return args.shouldUninstall;
     },
 
     //
--- a/calendar/base/src/calUtils.js
+++ b/calendar/base/src/calUtils.js
@@ -300,17 +300,17 @@ function openCalendarWizard(aCallback) {
 
 /**
  * Opens the calendar properties window for aCalendar
  *
  * @param aCalendar  the calendar whose properties should be displayed
  */
 function openCalendarProperties(aCalendar) {
     openDialog("chrome://calendar/content/calendar-properties-dialog.xul",
-               "Calendar:PropertiesDialog",
+               "CalendarPropertiesDialog",
                "chrome,titlebar,modal,resizable",
                {calendar: aCalendar});
 }
 
 /**
  * Opens the print dialog
  */
 function calPrint() {
--- a/calendar/providers/gdata/content/gdata-migration.js
+++ b/calendar/providers/gdata/content/gdata-migration.js
@@ -119,17 +119,17 @@ function gdata_migration_loader() {
                 // Do this after load, so the calendar window appears before the
                 // wizard is opened.
 
                 // XXX Waiting a second gives the views enough time to display
                 // right, at least on my system. The viewloaded event is quite
                 // view specific, so there is no good non-hacked way to do this.
                 setTimeout(function() {
                     window.openDialog("chrome://gdata-provider/content/gdata-migration-wizard.xul",
-                                      "Gdata:MigrationWizard",
+                                      "GdataMigrationWizard",
                                       "chrome,titlebar,modal,alwaysRaised");
                 }, 1000);
             }
         }
     }
 }
 
 // Add a Load handler to check for migratable calendars in the main window, or
--- a/calendar/resources/content/calendar.js
+++ b/calendar/resources/content/calendar.js
@@ -225,17 +225,17 @@ function changeNumberOfWeeks(menuitem) {
 
 function pickAndGoToDate() {
   var initialDate = currentView().selectedDay.getInTimezone(floating()).jsDate;
   var callback = function receiveAndGoToDate(pickedDate) {
     currentView().goToDay(jsDateToDateTime(pickedDate));
     getMinimonth().value = pickedDate;
   };
   openDialog("chrome://sunbird/content/calendar-gotodate-dialog.xul",
-             "Calendar:GoToDateDialog",
+             "CalendarGoToDateDialog",
              "chrome,modal",
              {callback: callback, date: initialDate});
 }
 
 function sbSwitchToView(newView) {
   var mwWeeksCommand = document.getElementById("menu-numberofweeks-inview");
   if (newView == "multiweek") {
       mwWeeksCommand.removeAttribute("disabled");