Bug 1557979 - Follow-up, take 2: Fix assignment of 'hasPageMenu'. r=darktrojan DONTBUILD
authorJorg K <jorgk@jorgk.com>
Mon, 10 Jun 2019 08:53:31 +0200
changeset 35819 222be3dc3eb1bd86a666ac6b0b0cff78cb078c82
parent 35818 b62e68799a992bd8baccd09a6f4e5937878b1c7a
child 35820 fe5ced6a8398bb4438540773bcd19ecea67270d7
push id392
push userclokep@gmail.com
push dateMon, 02 Sep 2019 20:17:19 +0000
reviewersdarktrojan
bugs1557979
Bug 1557979 - Follow-up, take 2: Fix assignment of 'hasPageMenu'. r=darktrojan DONTBUILD
mail/base/content/nsContextMenu.js
--- a/mail/base/content/nsContextMenu.js
+++ b/mail/base/content/nsContextMenu.js
@@ -77,18 +77,17 @@ nsContextMenu.prototype = {
       return;
     }
 
     this.isContentSelected = this.isContentSelection();
 
     this.hasPageMenu = false;
     if (!aIsShift) {
       let menuObject = PageMenuParent.maybeBuild(this.target);
-      PageMenuParent.addToPopup(menuObject, null, aPopup);
-      this.hasPageMenu = true;
+      this.hasPageMenu = PageMenuParent.addToPopup(menuObject, null, aPopup);
 
       let subject = {
         menu: aPopup,
         tab: document.getElementById("tabmail") ?
              document.getElementById("tabmail").currentTabInfo : undefined,
         isContentSelected: this.isContentSelected,
         inFrame: this.inFrame,
         isTextSelected: this.isTextSelected,