Bug 1254312 - [e10s] Fix browser_plugin_enabled_state_locked.js to run in e10s. r?mconley draft
authorJared Wein <jwein@mozilla.com>
Mon, 07 Mar 2016 17:31:14 -0500
changeset 337525 24b667fb6db683963244bda1e225ff810359d43d
parent 337312 fd095cbb65ad0d4e20c10f7a1c1c3e7dcf3abdab
child 515686 6eaa768a43fcb82ee70b00ea567c9ce320503739
push id12382
push userjwein@mozilla.com
push dateMon, 07 Mar 2016 22:32:25 +0000
reviewersmconley
bugs1254312
milestone47.0a1
Bug 1254312 - [e10s] Fix browser_plugin_enabled_state_locked.js to run in e10s. r?mconley MozReview-Commit-ID: 4FDbMuNCbfV
toolkit/mozapps/extensions/test/browser/browser-common.ini
toolkit/mozapps/extensions/test/browser/browser_plugin_enabled_state_locked.js
--- a/toolkit/mozapps/extensions/test/browser/browser-common.ini
+++ b/toolkit/mozapps/extensions/test/browser/browser-common.ini
@@ -42,17 +42,16 @@ skip-if = buildapp == 'mulet'
 [browser_experiments.js]
 skip-if = e10s
 [browser_list.js]
 [browser_metadataTimeout.js]
 [browser_searching.js]
 [browser_sorting.js]
 [browser_sorting_plugins.js]
 [browser_plugin_enabled_state_locked.js]
-skip-if = e10s # TypeError: Promise.defer is not a function
 [browser_uninstalling.js]
 [browser_install.js]
 [browser_recentupdates.js]
 [browser_manualupdates.js]
 [browser_globalwarnings.js]
 [browser_globalinformations.js]
 [browser_eula.js]
 skip-if = buildapp == 'mulet'
--- a/toolkit/mozapps/extensions/test/browser/browser_plugin_enabled_state_locked.js
+++ b/toolkit/mozapps/extensions/test/browser/browser_plugin_enabled_state_locked.js
@@ -25,19 +25,19 @@ function getTestPluginPref() {
 }
 
 registerCleanupFunction(() => {
   Services.prefs.unlockPref(getTestPluginPref());
   Services.prefs.clearUserPref(getTestPluginPref());
 });
 
 function getPlugins() {
-  let deferred = Promise.defer();
-  AddonManager.getAddonsByTypes(["plugin"], plugins => deferred.resolve(plugins));
-  return deferred.promise;
+  return new Promise(resolve => {
+    AddonManager.getAddonsByTypes(["plugin"], plugins => resolve(plugins));
+  });
 }
 
 function getTestPlugin(aPlugins) {
   let testPluginId;
 
   for (let plugin of aPlugins) {
     if (plugin.name == "Test Plug-in") {
       testPluginId = plugin.id;
@@ -71,25 +71,25 @@ function checkStateMenuDetail(locked) {
   Assert.equal(Services.prefs.prefIsLocked(getTestPluginPref()), locked,
     "Preference should be " + (locked === true ? "" : "un") + "locked.");
 
   // open details menu
   let details = gManagerWindow.document.getAnonymousElementByAttribute(gPluginElement, "anonid", "details-btn");
   is_element_visible(details, "Details link should be visible.");
   EventUtils.synthesizeMouseAtCenter(details, {}, gManagerWindow);
 
-  let deferred = Promise.defer();
-  wait_for_view_load(gManagerWindow, function() {
-    let menuList = gManagerWindow.document.getElementById("detail-state-menulist");
-    is_element_visible(menuList, "Details state menu should be visible.");
-    Assert.equal(menuList.disabled, locked,
-      "Details state menu enabled state should be correct.");
-    deferred.resolve();
+  return new Promise(resolve => {
+    wait_for_view_load(gManagerWindow, function() {
+      let menuList = gManagerWindow.document.getElementById("detail-state-menulist");
+      is_element_visible(menuList, "Details state menu should be visible.");
+      Assert.equal(menuList.disabled, locked,
+        "Details state menu enabled state should be correct.");
+      resolve();
+    });
   });
-  return deferred.promise;
 }
 
 add_task(function* initializeState() {
   Services.prefs.setIntPref(getTestPluginPref(), Ci.nsIPluginTag.STATE_ENABLED);
   Services.prefs.unlockPref(getTestPluginPref());
   gManagerWindow = yield open_manager();
   gCategoryUtilities = new CategoryUtilities(gManagerWindow);
   yield gCategoryUtilities.openType("plugin");