Closed Bug 1445318 Opened 2 years ago Closed 2 years ago

Remove storage: "persistent" from AddonStudies.jsm

Categories

(Firefox :: Normandy Client, enhancement, P1)

60 Branch
enhancement

Tracking

()

RESOLVED FIXED
Firefox 61
Tracking Status
firefox61 --- fixed

People

(Reporter: johannh, Assigned: johannh)

References

(Blocks 1 open bug)

Details

(Whiteboard: [storage-v2])

Attachments

(1 file)

As part of deprecating the storage option for indexedDB.open(), we would like to remove the usage of storage: "persistent" in AddonStudies.jsm.

Since that code seems to run on chrome principal there should not be any bad consequences like data loss, but before I make a patch I'd like to be able to end-to-end test this to be sure.

Mike, what's the easiest way I can test the functionality of that database?

Thanks!
Flags: needinfo?(mcooper)
Status: NEW → ASSIGNED
Hi Johann, thanks for looking into this. Probably the easiest way to test this is to interface with a Normandy server, creating a recipe there that would utilize this storage. Then after the patch, you could make sure that the code using the storage acts correctly before and after the patch.

To get access to a server, you have 3 choices

1) We can get you access to the Normandy staging server, where you can create a recipe that will target your builds of Firefox.
2) You could set up a local instance of Normandy server, so you can test it entirely locally.
3) I can work with you directly to set up a recipe on the staging server to test this.

The code in question uses that database to maintain state about studies between runs of Firefox. The biggest thing you could test is to get enrolled in a study, then use about:studies to unenroll. After that, you want to make sure that about:studies continues to show the study, and that you don't get re-enrolled. Both of these behaviors rely on this storage.

Alternatively, you could probably trigger the database to be used via a browser console. That seems like a weaker test to me though. If you'd rather explore this instead, I can help with that too.
Flags: needinfo?(mcooper)
Whiteboard: [storage-v2][triage] → [storage-v2]
Thanks for the help getting a test study up and running! I manually verified that this does not change the database type, but you might want to do another quick check as well.
Comment on attachment 8961961 [details]
Bug 1445318 - Remove storage: "persistent" from AddonStudies.jsm.

https://reviewboard.mozilla.org/r/230804/#review237156

I tested this locally, and it seems to work for me too. Thanks for the patch!
Attachment #8961961 - Flags: review?(mcooper) → review+
Pushed by jhofmann@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3ac144929b5b
Remove storage: "persistent" from AddonStudies.jsm. r=mythmon
https://hg.mozilla.org/mozilla-central/rev/3ac144929b5b
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
You need to log in before you can comment on or make changes to this bug.