Bug 1445318 - Remove storage: "persistent" from AddonStudies.jsm. r=mythmon draft
authorJohann Hofmann <jhofmann@mozilla.com>
Fri, 23 Mar 2018 23:13:01 +0100
changeset 771786 1a99a824f00a810d45ad11b261f20563ae473a53
parent 771093 7771df14ea181add1dc4133f0f5559bf620bf976
push id103791
push userjhofmann@mozilla.com
push dateFri, 23 Mar 2018 22:14:08 +0000
reviewersmythmon
bugs1445318
milestone61.0a1
Bug 1445318 - Remove storage: "persistent" from AddonStudies.jsm. r=mythmon This option is deprecated and we would like to remove it soon. Since AddonStudies.jsm runs with system privileges, it is getting the same storage type ("persistent"), no matter the storage option. I verified this through manual testing. MozReview-Commit-ID: 79MuZ7OU1Ti
toolkit/components/normandy/lib/AddonStudies.jsm
--- a/toolkit/components/normandy/lib/AddonStudies.jsm
+++ b/toolkit/components/normandy/lib/AddonStudies.jsm
@@ -43,17 +43,16 @@ ChromeUtils.defineModuleGetter(this, "Te
 Cu.importGlobalProperties(["fetch"]); /* globals fetch */
 
 var EXPORTED_SYMBOLS = ["AddonStudies"];
 
 const DB_NAME = "shield";
 const STORE_NAME = "addon-studies";
 const DB_OPTIONS = {
   version: 1,
-  storage: "persistent",
 };
 const STUDY_ENDED_TOPIC = "shield-study-ended";
 const log = LogManager.getLogger("addon-studies");
 
 /**
  * Create a new connection to the database.
  */
 function openDatabase() {