Closed
Bug 1331467
Opened 8 years ago
Closed 8 years ago
Turn on chrome.storage.sync by default
Categories
(WebExtensions :: General, defect)
WebExtensions
General
Tracking
(firefox53 fixed)
RESOLVED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: andy+bugzilla, Assigned: glasserc)
References
Details
Attachments
(1 file)
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•8 years ago
|
Flags: needinfo?(krupa.mozbugs)
Flags: needinfo?(eglassercamp)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ad4fa9eeb1e9
Assignee | ||
Comment 3•8 years 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•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ad819c7358b6
Comment 6•8 years 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•8 years 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•8 years ago
|
Keywords: checkin-needed
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/442d75009799 Enable chrome.storage.sync by default, r=kmag
Keywords: checkin-needed
Comment 9•8 years ago
|
||
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•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=09223e98b279
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Updated•8 years ago
|
Assignee: nobody → eglassercamp
Comment 12•8 years 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•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/44f4eef0b271
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Updated•6 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•