Backed out changeset da68367e23e5 (bug 1488176)
authorGeoff Lankow <geoff@darktrojan.net>
Mon, 14 Jan 2019 14:22:59 +1300
changeset 34187 02def84cbd9c578b7f345ae4301b34f1fe5f1374
parent 34186 7acd73ce23af98db7f9d338741d252e8fcd964fc
child 34188 35c4cf91abd02ae75704627f6c8ad2a0735103aa
push id389
push userclokep@gmail.com
push dateMon, 18 Mar 2019 19:01:53 +0000
bugs1488176
backs outda68367e23e51f72ccd8e1aec01b76eee6732fbd
Backed out changeset da68367e23e5 (bug 1488176)
mail/components/extensions/parent/ext-menus.js
mail/components/extensions/test/browser/browser.ini
--- a/mail/components/extensions/parent/ext-menus.js
+++ b/mail/components/extensions/parent/ext-menus.js
@@ -599,20 +599,19 @@ function addMenuEventInfo(info, contextD
   }
   // If the context was overridden, then frameUrl should be the URL of the
   // document in which the menu was opened (instead of undefined, even if that
   // document is not in a frame).
   if (contextData.originalViewUrl) {
     info.frameUrl = contextData.originalViewUrl;
   }
 
-  // TODO re-enable this with bug 1488176.
-  // if (contextData.selectedMessages && extension.hasPermission("messagesRead")) {
-  //   info.selectedMessages = messageListTracker.startList(contextData.selectedMessages, {extension});
-  // }
+  if (contextData.selectedMessages && extension.hasPermission("messagesRead")) {
+    info.selectedMessages = messageListTracker.startList(contextData.selectedMessages, {extension});
+  }
   for (let folderType of ["displayedFolder", "selectedFolder"]) {
     if (contextData[folderType] && extension.hasPermission("accountsRead")) {
       info[folderType] = convertFolder(contextData[folderType]);
     }
   }
 }
 
 function MenuItem(extension, createProperties, isRoot = false) {
--- a/mail/components/extensions/test/browser/browser.ini
+++ b/mail/components/extensions/test/browser/browser.ini
@@ -2,12 +2,10 @@
 head = head.js
 subsuite = thunderbird
 tags = webextensions
 
 [browser_ext_addressBooksUI.js]
 [browser_ext_browserAction.js]
 [browser_ext_composeAction.js]
 [browser_ext_menus.js]
-skip-if=true # waiting for bug 1488176
 [browser_ext_mailTabs.js]
-skip-if=true # waiting for bug 1488176
 [browser_ext_quickFilter.js]