Bug 84893 - [UserTiming] UserTiming metabug
Summary: [UserTiming] UserTiming metabug
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: http://dvcs.w3.org/hg/webperf/raw-fil...
Keywords:
Depends on: 90963 116551
Blocks: 99464
  Show dependency treegraph
 
Reported: 2012-04-25 14:51 PDT by James Simonsen
Modified: 2017-01-27 22:39 PST (History)
16 users (show)

See Also:


Attachments
Patch (52.88 KB, patch)
2012-07-04 04:27 PDT, pdeng6
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from gce-cr-linux-03 (293.51 KB, application/zip)
2012-07-04 05:48 PDT, WebKit Review Bot
no flags Details
Archive of layout-test-results from gce-cr-linux-07 (293.29 KB, application/zip)
2012-07-04 06:55 PDT, WebKit Review Bot
no flags Details
Patch (97.29 KB, patch)
2012-07-08 23:58 PDT, pdeng6
no flags Details | Formatted Diff | Diff
Patch (97.66 KB, patch)
2012-07-09 06:27 PDT, pdeng6
no flags Details | Formatted Diff | Diff
Patch (96.17 KB, patch)
2012-07-09 08:19 PDT, pdeng6
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description James Simonsen 2012-04-25 14:51:58 PDT
The User Timing API is ready for implementation. This metabug is to track work for implementing it in WebKit. This will need to be announced on webkit-dev before anyone begins. As needed, please add specific implementation bugs that block this.
Comment 1 pdeng6 2012-07-04 04:27:38 PDT
Created attachment 150769 [details]
Patch
Comment 2 WebKit Review Bot 2012-07-04 05:48:22 PDT
Comment on attachment 150769 [details]
Patch

Attachment 150769 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/13139504

New failing tests:
fast/dom/Window/window-properties-performance.html
Comment 3 WebKit Review Bot 2012-07-04 05:48:25 PDT
Created attachment 150778 [details]
Archive of layout-test-results from gce-cr-linux-03

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: gce-cr-linux-03  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.39-gcg-201203291735-x86_64-with-Ubuntu-10.04-lucid
Comment 4 WebKit Review Bot 2012-07-04 06:55:23 PDT
Comment on attachment 150769 [details]
Patch

Attachment 150769 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/13137708

New failing tests:
fast/dom/Window/window-properties-performance.html
Comment 5 WebKit Review Bot 2012-07-04 06:55:29 PDT
Created attachment 150785 [details]
Archive of layout-test-results from gce-cr-linux-07

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: gce-cr-linux-07  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.39-gcg-201203291735-x86_64-with-Ubuntu-10.04-lucid
Comment 6 pdeng6 2012-07-08 23:58:13 PDT
Created attachment 151194 [details]
Patch
Comment 7 pdeng6 2012-07-09 06:27:38 PDT
Created attachment 151244 [details]
Patch
Comment 8 WebKit Review Bot 2012-07-09 07:22:01 PDT
Comment on attachment 151244 [details]
Patch

Attachment 151244 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/13162282
Comment 9 pdeng6 2012-07-09 08:19:05 PDT
Created attachment 151249 [details]
Patch
Comment 10 James Simonsen 2012-07-09 17:12:48 PDT
Thanks for the patch. Just a couple of high level comments at this point:

- I probably shouldn't have said "ready for implementation" without taking another pass through the spec. Sorry. The optional return types are kinda icky. I'd rather we just used the Performance Timeline getEntries*() functions. I've started a thread on public-webperf. You might want to hold off until that's settled.

- Everything needs to be vendor-prefixed until the spec moves forward. So all of the functions need to start with "webkit".

- I'd prefer to use the W3C style test suites instead of adding more WebKit specific ones. It'd be better to just write one set of tests and have them work for both WebKit and the W3C. I have bug 84887 that imports the W3C tests for Navigation Timing. Maybe we can build off that.

- User Timing should integrate with Performance Timeline. Maybe the spec isn't very clear on that, but the returned objects (PerformanceMeasure and PerformanceMark) should derive from PerformanceEntry. They should also be returned when you call the Performance Timeline getEntries*() functions. If the spec isn't clear, please post comments on public-webperf.

- This is a metabug. It shouldn't have any patches on it. We should create additional bugs to implement aspects of the spec and upload patches to those. Those bugs would block this metabug. For instance, you could have one bug for the main implementation and another to integrate it with Performance Timeline and one to remove the vendor prefixes when ready. Then you wouldn't have to implement everything all at once either. I suspect there'll be some other tweaks here and there as we go too. That's how the other timing specs have been.
Comment 11 pdeng6 2012-07-10 05:52:19 PDT
Thanks for your kind comments, they are very helpful. :)

Since getMarks() and getMeasures() interface maybe change as W3C spec move forward, I would like to implement other main interfaces firstly. patch will be uploaded in other bug entry.
Comment 12 Anders Carlsson 2014-02-05 10:57:35 PST
Comment on attachment 151249 [details]
Patch

Clearing review flag on patches from before 2014. If this patch is still relevant, please reset the r? flag.
Comment 13 Joseph Pecoraro 2017-01-27 22:39:35 PST
User Timing eventually landed. I'll be turning it on experimentally soon. This bug, and its patches, are quite old, so I'm going to close this out.