Closed Bug 1513021 Opened 5 years ago Closed 5 years ago

Enable the storage access API on desktop

Categories

(Core :: DOM: Core & HTML, enhancement, P1)

65 Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla66
Tracking Status
relnote-firefox --- 65+
firefox65 --- fixed
firefox66 --- fixed

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

(An intent to ship email needs to be sent for this.  We intend to enable this to ride the trains for 65.)
Priority: -- → P1
Version: unspecified → 65 Branch
Depends on: 1512625
Attachment #9032298 - Attachment description: Bug 1513021 - Ship the Storage Access API on all desktop platform; → Bug 1513021 - Ship the Storage Access API on all desktop platforms;
Comment on attachment 9032298 [details]
Bug 1513021 - Ship the Storage Access API on all desktop platforms;

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Not a regression

User impact if declined: We'd like to ship this API in 65, please see the intent to ship thread for rationale: https://groups.google.com/forum/#!topic/mozilla.dev.platform/BZ36ZDV0Fik

Is this code covered by automated tests?: Yes

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: No

If yes, steps to reproduce: 

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): This has been turned on in Nightly for a few cycles.  The API itself will not do anything unless the default cookie restrictions feature (bug 1473978) is shipped, and it isn't yet known to be used by a lot of third-party trackers, so we expect the risk of shipping this API at this point to be minimal.

String changes made/needed: None
Attachment #9032298 - Flags: approval-mozilla-beta?
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2dc724f82cb1
Ship the Storage Access API on all desktop platforms; r=baku
https://hg.mozilla.org/mozilla-central/rev/2dc724f82cb1
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Comment on attachment 9032298 [details]
Bug 1513021 - Ship the Storage Access API on all desktop platforms;

[Triage Comment]
Already enabled on Nightly for a few cycles and doesn't do much by itself unless the default cookie restrictions are also enabled (which is not yet the case). Approved for 65.0b7 so as to not block any future decisions around this feature as we move forward.
Attachment #9032298 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Is there anything QA can/should be doing with this for now to be on the lookout for any regressions or functional issues?
Flags: needinfo?(ehsan)
(In reply to Ryan VanderMeulen [:RyanVM] from comment #6)
> Is there anything QA can/should be doing with this for now to be on the
> lookout for any regressions or functional issues?

No specific QA needed here, this feature has been receiving QA on the Nightly train.  (And yes, we should of course be watching for regressions.)
Flags: needinfo?(ehsan)

I've documented this:

Removed it from experimental features pages now it is enabled in desktop:
https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Experimental_features

Removed experimental banners from pages as it is now available in two browsers by default:
https://developer.mozilla.org/en-US/docs/Web/API/Document
https://developer.mozilla.org/en-US/docs/Web/API/Document/hasStorageAccess
https://developer.mozilla.org/en-US/docs/Web/API/Document/requestStorageAccess

Updated the browser compat data:
https://github.com/mdn/browser-compat-data/pull/3303

Let me know if this looks OK. Thanks!

Thanks, looks great!

Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: