-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Remove never-supported Budget APIs #6924
Conversation
dee2ac9
to
163359e
Compare
163359e
to
339db73
Compare
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.
I can confirm this with the commit message I wrote for #6535:
Only the reserve() method was shipped, and later removed:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/yBtmc-4xl_o/GE0vneAVDQAJ
https://groups.google.com/a/chromium.org/d/msg/blink-dev/18r3whCBv0I/b8qrtFTsDAAJhttps://software.hixie.ch/utilities/js/live-dom-viewer/?saved=8362 was
tested on Chrome and Opera on Windows to verify the ranges. Android
WebView was never supported per the blink-dev intent, and the other data
was mirrored.
@chrisdavidmills We did this removal before we discussed how to handle the disposition of pages. Would you mind deleting the https://wiki.developer.mozilla.org/docs/Web/API/BudgetState tree? |
@chrisdavidmills actually, on second thought, there a couple of others to consider. I'll open an issue on mdn/sprints with a list of all of them instead. Sorry for the noise. |
It's all good. I just deleted the BudgetState tree. |
@chrisdavidmills Oh, good news: that was all of them after all. Thank you! |
A checklist to help your pull request get merged faster: