Closed Bug 1386021 Opened 3 years ago Closed 3 years ago
Observer ride the trains
59 bytes, text/x-review-board-request
+++ This bug was initially created as a clone of Bug #1271487 +++ As per bug 1271487, this has been baking in Nightly since 49. Chrome appears to have had it generally available since Chrome 52. It'd be wonderful to get this to ride the trains (maybe even 56?), since it's in use in the wild now. I'm hoping to use the Google TTI polyfill as soon as I reasonably can, which requires this API, as well as the Long Tasks API https://github.com/GoogleChrome/tti-polyfill.
Note that there are still 4 failures in testing/web-platform/tests/performance-timeline/ but I think they don't stop shipping performance observer. The four failures are; 1. po-observe.html (lack of navigation timing support: bug 1043083) 2. po-navigation.html (lack of navigation timing support: bug 1043083) 3. idlharness.html (toJSON support: bug 1375829) 4. case-sensitivity.any.js (no bug) As for 4, the test case is supposed to be received resource entries in worker that the resource entries actually were loaded in window context. It's not clear to me that we should include such entries in worker. But anyway, it's not related to performance observer.
Thank you so much for doing this. Would you mind sending an intent to ship?
Sure, I am preparing it. https://treeherder.mozilla.org/#/jobs?repo=try&revision=63a407e4a688f275e1aea1cd7b88664a2502a72f Now I am looking for the status for WebKit and Edge..
Great, thanks. For Edge I think you have: https://developer.microsoft.com/en-us/microsoft-edge/platform/status/performanceobserver/
Thanks! I found the WebKit status.
Intent to ship: https://groups.google.com/forum/#!topic/mozilla.dev.platform/ELX9G6OOAjw The try in comment 3 has lots of oranges, but all of them are unrelated to PerformanceObserver.
Comment on attachment 8904863 [details] Bug 1386021 - Enable PerformanceObserver on all channels. https://reviewboard.mozilla.org/r/176638/#review181640 ::: dom/performance/tests/test_performance_observer.html:6 (Diff revision 1) > <!-- > Any copyright is dedicated to the Public Domain. > http://creativecommons.org/publicdomain/zero/1.0/ > --> > <!DOCTYPE html> > <meta charset=utf-8> move this into <head> ::: dom/performance/tests/test_worker_observer.html:6 (Diff revision 1) > <!-- > Any copyright is dedicated to the Public Domain. > http://creativecommons.org/publicdomain/zero/1.0/ > --> > <!DOCTYPE html> > <meta charset=utf-8> same here
Attachment #8904863 - Flags: review?(amarchesini) → review+
Thanks for the review!
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/9d86c033299e Enable PerformanceObserver on all channels. r=baku
:hiro, thanks so much for making this happen, it's like we're getting an expected gift for 57!
Adding dev-doc-needed back, because the browser compatibility chart at the bottom of https://developer.mozilla.org/en-US/docs/Web/API/PerformanceObserver needs to be updated, and I can't quite figure out what the correct syntax is expected to be.
I've removed all the experimental banners (since it is now enabled by default in two major browsers), and updated the support tables on all relevant pages: https://developer.mozilla.org/en-US/docs/Web/API/PerformanceObserver (and child pages) https://developer.mozilla.org/en-US/docs/Web/API/PerformanceObserverEntryList (and child pages) I've also removed it from the experimental features page: https://developer.mozilla.org/en-US/Firefox/Experimental_features And I've added a note to the Fx57 rel notes: https://developer.mozilla.org/en-US/Firefox/Releases/57#New_APIs Let me know if that looks OK. Cheers!
You need to log in before you can comment on or make changes to this bug.