Bug 1401876 - (Part 1) Close datetime picker on popuphidden. r=mconley

MozReview-Commit-ID: FUqbtAftQxh

--HG--
extra : rebase_source : 9570536f7d46d5e8a481d36ed90d81c6f4e8183d

toolkit/content/tests/browser/head.js

@@ -190,6 +190,7 @@ class DateTimeTestHelper {
       let pickerClosePromise = new Promise(resolve => {
         this.panel.addEventListener("popuphidden", resolve, {once: true});
       });
+      this.panel.hidePopup();
       this.panel.closePicker();
       await pickerClosePromise;
     }

toolkit/content/widgets/datetimepopup.xml

@@ -75,7 +75,6 @@
           this.dateTimePopupFrame.removeEventListener("load", this, true);
           this.dateTimePopupFrame.contentDocument.removeEventListener("message", this);
           this.dateTimePopupFrame.setAttribute("src", "");
-          this.hidePopup();
           this.hidden = true;
         ]]></body>
       </method>
@@ -320,6 +319,7 @@
               break;
             }
             case "ClosePopup": {
+              this.hidePopup();
               this.closePicker();
               break;
             }

toolkit/modules/DateTimePickerHelper.jsm

@@ -90,6 +90,7 @@ this.DateTimePickerHelper = {
         if (browser) {
           browser.messageManager.sendAsyncMessage("FormDateTime:PickerClosed");
         }
+        this.picker.closePicker();
         this.close();
         break;
       }