Bug 1528186 Set scope for built-in addon location r=zombie
authorAndrew Swan <aswan@mozilla.com>
Fri, 22 Feb 2019 10:35:57 -0800
changeset 461112 13457088df2c75726f901023a31c61d63cf3bcb5
parent 461111 c04aa3d7aa8e46bd547d8e0e1a1ac7a88ae3e4c4
child 461113 c4ce50209f1966dfd7b10c6060b5a625395e0dca
push id35618
push usershindli@mozilla.com
push dateTue, 26 Feb 2019 16:54:44 +0000
treeherdermozilla-central@d326a9d5f77b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerszombie
bugs1528186
milestone67.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1528186 Set scope for built-in addon location r=zombie Differential Revision: https://phabricator.services.mozilla.com/D20848
toolkit/mozapps/extensions/internal/XPIProvider.jsm
--- a/toolkit/mozapps/extensions/internal/XPIProvider.jsm
+++ b/toolkit/mozapps/extensions/internal/XPIProvider.jsm
@@ -850,17 +850,17 @@ class XPIStateLocation extends Map {
 }
 
 class TemporaryLocation extends XPIStateLocation {
   /**
    * @param {string} name
    *        The string identifier for the install location.
    */
   constructor(name) {
-    super(name, null, null);
+    super(name, null, AddonManager.SCOPE_TEMPORARY);
     this.locked = false;
   }
 
   makeInstaller() {
     // Installs are a no-op. We only register that add-ons exist, and
     // run them from their current location.
     return {
       installAddon() {},
@@ -883,17 +883,17 @@ class TemporaryLocation extends XPIState
 
 var TemporaryInstallLocation = new TemporaryLocation(KEY_APP_TEMPORARY);
 
 /**
  * A "location" for addons installed from assets packged into the app.
  */
 var BuiltInLocation = new class _BuiltInLocation extends XPIStateLocation {
   constructor() {
-    super(KEY_APP_BUILTINS, null, null);
+    super(KEY_APP_BUILTINS, null, AddonManager.SCOPE_APPLICATION);
     this.locked = false;
   }
 
   // The installer object is responsible for moving files around on disk
   // when (un)installing an addon.  Since this location handles only addons
   // that are embedded within the browser, these are no-ops.
   makeInstaller() {
     return {