Bug 1507304 - Remove unused event handler causing errors after bug 1500620. r=philipp a=jorgk
authorGeoff Lankow <geoff@darktrojan.net>
Thu, 15 Nov 2018 11:17:40 +1300
changeset 33491 76b887c99b713bd0dd4f358e6f4f65d3aad15d6b
parent 33490 7b73575205fc877a14e236a77720fbfbf8e9fa82
child 33492 40efa95a8ab09a06f4e8778bfdb1dfad8a940c3c
push id387
push userclokep@gmail.com
push dateMon, 10 Dec 2018 21:30:47 +0000
reviewersphilipp, jorgk
bugs1507304, 1500620
Bug 1507304 - Remove unused event handler causing errors after bug 1500620. r=philipp a=jorgk
calendar/resources/content/datetimepickers/datetimepickers.xml
--- a/calendar/resources/content/datetimepickers/datetimepickers.xml
+++ b/calendar/resources/content/datetimepickers/datetimepickers.xml
@@ -370,27 +370,23 @@
                 // aEvent.target is the minimonth
                 datepicker.update(new Date(aEvent.target.value), true);
                 // select changed value so no cursor appears (can't type to it).
                 datepicker.select();
                 aEvent.target.parentNode.hidePopup();
             }
         ]]></body>
       </method>
-
-      </implementation>
-
-      <handlers>
-          <handler event="bindingattached" action="this.initialize();"/>
+    </implementation>
 
-          <handler event="blur" phase="capturing"><![CDATA[
-              this.parseTextBoxDate(true);
-          ]]></handler>
-      </handlers>
-
+    <handlers>
+      <handler event="blur" phase="capturing"><![CDATA[
+          this.parseTextBoxDate(true);
+      ]]></handler>
+    </handlers>
   </binding>
 
   <binding id="datepicker-forever" extends="chrome://calendar/content/datetimepickers/datetimepickers.xml#datepicker">
     <content>
       <xul:hbox flex="1" id="hbox" class="datepicker-box-class">
         <xul:menulist anonid="foreverMenulist" editable="true" type="panel" sizetopopup="false"
                       class="datepicker-text-class"
                       onchange="this.kDatePicker.parseTextBoxDate(true, event);"
@@ -627,21 +623,19 @@
       </method>
 
       <method name="select">
         <body><![CDATA[
             // select all in text box
             this.kTextBox.select();
         ]]></body>
       </method>
-
     </implementation>
 
     <handlers>
-      <handler event="bindingattached" action="this.initialize();"/>
       <handler event="blur" phase="capturing"><![CDATA[
           this.parseTextBoxTime(true);
       ]]></handler>
     </handlers>
   </binding>
 
   <binding id="timepicker-hour">
     <content>
@@ -1334,20 +1328,16 @@
                 document.getAnonymousElementByAttribute(this, "anonid", "time-picker-hour-box-" + i)
                         .setAttribute("label", i - 12);
             }
             document.getAnonymousElementByAttribute(this, "anonid", "time-picker-hour-grid")
                     .setAttribute("format12hours", "true");
         ]]></body>
       </method>
     </implementation>
-
-    <handlers>
-      <handler event="bindingattached" action="this.initialize();"/>
-    </handlers>
   </binding>
 
   <binding id="datetimepicker" extends="chrome://calendar/content/datetimepickers/datetimepickers.xml#datetimepicker-base"
            inherits="value,onchange,disabled,datepickerdisabled,timepickerdisabled">
     <!-- onchange was simply "onDatePick()" in Moz1.6, but stopped working in Moz1.7
          so had to add navigation by parents. -->
     <content>
       <xul:hbox flex="1" anonid="hbox">
@@ -1431,21 +1421,16 @@
             dateTime.setHours(newTime.getHours());
             dateTime.setMinutes(newTime.getMinutes());
             dateTime.setSeconds(newTime.getSeconds());
             this.mValue = dateTime;
             this.kDatePicker.update(dateTime, false);
         ]]></body>
       </method>
     </implementation>
-
-    <handlers>
-      <handler event="bindingattached" action="this.initialize();"/>
-    </handlers>
-
   </binding>
 
   <binding id="datetimepicker-base" extends="chrome://global/content/bindings/general.xml#basecontrol"
            inherits="value,onchange">
     <implementation>
       <constructor><![CDATA[
           ChromeUtils.import("resource://gre/modules/Services.jsm");