Turn on chrome.storage.sync by default

RESOLVED FIXED in Firefox 53

Status

()

Toolkit
WebExtensions: General
RESOLVED FIXED
5 months ago
5 months ago

People

(Reporter: andym, Assigned: glasserc)

Tracking

Trunk
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

5 months ago
The testing has been completed and things are looking good. We'd like to turn on chrome.storage.sync by default. That means flipping the pref and pointing to the production server.

Then we can let it ride the trains out to production in Firefox 53.

Still waiting on bug 1311510 to be completed before it hits beta.

Need info'ing Ethan and Krupa just to check that's ok with them.
(Reporter)

Updated

5 months ago
Flags: needinfo?(krupa.mozbugs)
Flags: needinfo?(eglassercamp)
Comment hidden (mozreview-request)
(Assignee)

Comment 2

5 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ad4fa9eeb1e9
(Assignee)

Comment 3

5 months ago
Attached please find a patch that should turn on chrome.storage.sync by default.
Flags: needinfo?(eglassercamp)
Comment hidden (mozreview-request)
(Assignee)

Comment 5

5 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ad819c7358b6

Comment 6

5 months ago
With the caveat that https://bugzilla.mozilla.org/show_bug.cgi?id=1311510 needs to be complete before the feature hits beta, QA = a+ on this feature. All the test results can be tracked at https://goo.gl/QcGgvH
Flags: needinfo?(krupa.mozbugs)

Comment 7

5 months ago
mozreview-review
Comment on attachment 8827992 [details]
Bug 1331467: Enable chrome.storage.sync by default,

https://reviewboard.mozilla.org/r/105544/#review107116
Attachment #8827992 - Flags: review?(kmaglione+bmo) → review+
(Assignee)

Updated

5 months ago
Keywords: checkin-needed

Comment 8

5 months ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/442d75009799
Enable chrome.storage.sync by default, r=kmag
Keywords: checkin-needed
Backed out for test_ext_storage_content.html failures.
https://treeherder.mozilla.org/logviewer.html#?job_id=70846690&repo=autoland

https://hg.mozilla.org/integration/autoland/rev/68abeaaf3a2e1c4f32e618de595006b4c65a15ca
Comment hidden (mozreview-request)
(Assignee)

Comment 11

5 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=09223e98b279
(Assignee)

Updated

5 months ago
Keywords: checkin-needed
Assignee: nobody → eglassercamp

Comment 12

5 months ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/44f4eef0b271
Enable chrome.storage.sync by default, r=kmag
Keywords: checkin-needed

Comment 13

5 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/44f4eef0b271
Status: NEW → RESOLVED
Last Resolved: 5 months ago
status-firefox53: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.