-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Chromium versions for StorageManager API #11698
Update Chromium versions for StorageManager API #11698
Conversation
This PR updates and corrects the real values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the `StorageManager` API, based upon results from the [mdn-bcd-collector](https://mdn-bcd-collector.appspot.com) project (v3.2.2). Tests Used: https://mdn-bcd-collector.appspot.com/tests/api/StorageManager
Let's see... this data came from #2272 with reference to https://chromium.googlesource.com/chromium/src/+/21ba553558ed7cd65f59199b28dd8732f5dc93a0%5E%21/third_party/WebKit/Source/modules/quota/StorageManager.idl but without mentioning |
@@ -6,10 +6,10 @@ | |||
"spec_url": "https://storage.spec.whatwg.org/#storagemanager", | |||
"support": { | |||
"chrome": { | |||
"version_added": "48" | |||
"version_added": "55" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | ||
], | ||
"chrome": { | ||
"version_added": "55" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also confirmed by https://bugs.chromium.org/p/chromium/issues/detail?id=502373#c23
} | ||
], | ||
"chrome": { | ||
"version_added": "55" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also confirmed by https://bugs.chromium.org/p/chromium/issues/detail?id=502373#c23
@@ -54,10 +54,10 @@ | |||
"spec_url": "https://storage.spec.whatwg.org/#ref-for-dom-storagemanager-estimate", | |||
"support": { | |||
"chrome": { | |||
"version_added": "52" | |||
"version_added": "61" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"version_added": "52" | ||
}, | ||
{ | ||
"alternative_name": "requestPersistent", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deleting this stuff that was never in stable Chrome is 👍
This PR updates and corrects the real values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the
StorageManager
API, based upon results from the mdn-bcd-collector project (v3.2.2).Tests Used: https://mdn-bcd-collector.appspot.com/tests/api/StorageManager