Bug 1336763 - Don't ask content process for permission to unload a window if it never set an onbeforeunload event handler. r=dao

MozReview-Commit-ID: JfNz5SdKRTN --HG-- extra : rebase_source : 8bdc4ae2a0fb6e3739c7cdad9b5b09ee94da3429
2017-04-13 19:13:34 -04:00 · 2017-04-13 19:13:34 -04:00 · 3456896ce9
--- a/toolkit/content/widgets/remote-browser.xml
+++ b/toolkit/content/widgets/remote-browser.xml
@ -302,6 +302,13 @@
      <method name="permitUnload">
        <body>
        <![CDATA[
+          let { frameLoader } = this.QueryInterface(Components.interfaces.nsIFrameLoaderOwner);
+          let tabParent = frameLoader.tabParent;
+
+          if (!tabParent.hasBeforeUnload) {
+            return { permitUnload: true, timedOut: false };
+          }
+
          const kTimeout = 5000;

          let finished = false;