-
Notifications
You must be signed in to change notification settings - Fork 167
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 DOMTimeStamp #1021
Remove DOMTimeStamp #1021
Conversation
EpochTimeStamp is now in HR-Time. |
Filed meta Gecko bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1735096 ... along with various dependent changes. |
WebKit has implemented EpochTimeStamp (and removed DOMTimeStamp): https://bugs.webkit.org/show_bug.cgi?id=231496 |
Chromium is also in the process of being updated: https://bugs.chromium.org/p/chromium/issues/detail?id=1258466 |
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.
Looks like all uses have been eliminated! https://dontcallmedom.github.io/webidlpedia/names/DOMTimeStamp.html
given dontcallmedom/webidlpedia#24, here is the list of remaining DOMTimeStamp mentions:
|
Thank you, @dontcallmedom! Do people think we should revert this for now, since some of those discussions are still ongoing? Or just try to forge ahead? I'll file a Notifications API issue... |
Personally, I think we should forge ahead. Changes have landed in WebKit, there are patches really close to landing in Gecko, and bugs have been filed on Chromium (not sure about the status of the implementation tho). |
Sent PR to WebRTC. |
Add the EpochTimeStamp, which can be used to incrementally rename/remove DOMTimeStamp. See also w3c/hr-time#124 and whatwg/webidl#1021 Differential Revision: https://phabricator.services.mozilla.com/D128030
Add the EpochTimeStamp, which can be used to incrementally rename/remove DOMTimeStamp. See also w3c/hr-time#124 and whatwg/webidl#1021 Differential Revision: https://phabricator.services.mozilla.com/D128030
Add the EpochTimeStamp, which can be used to incrementally rename/remove DOMTimeStamp. See also w3c/hr-time#124 and whatwg/webidl#1021 Differential Revision: https://phabricator.services.mozilla.com/D128030
Add the EpochTimeStamp, which can be used to incrementally rename/remove DOMTimeStamp. See also w3c/hr-time#124 and whatwg/webidl#1021 Differential Revision: https://phabricator.services.mozilla.com/D128030
Closes #2
DOMTimeStamp becomes EpochTimeStamp in HR-TIME.
w3c/hr-time#124
Gecko bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1735096
Preview | Diff