Skip to content
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

PerformanceEntry.startTime/duration should be DOMHighResTimeStamp #31

Closed
foolip opened this issue Jul 7, 2015 · 4 comments
Closed

PerformanceEntry.startTime/duration should be DOMHighResTimeStamp #31

foolip opened this issue Jul 7, 2015 · 4 comments

Comments

@foolip
Copy link
Member

foolip commented Jul 7, 2015

http://w3c.github.io/performance-timeline/#the-performanceentry-interface

DOMString looks like a typo in the IDL, since the definitions talk about DOMHighResTimeStamp.

@igrigorik
Copy link
Member

@plehegar sanity check: typo, or was this intentional?

@toddreifsteck
Copy link
Member

This appears to be a typo from my review. Curious what @plehegar says

@eliperelman
Copy link

I think typo, [1] and others specifically say DOMHighResTimeStamp:

[1] http://www.w3.org/TR/user-timing/#performancemeasure

@plehegar
Copy link
Member

Here is when the mistake was introduced. 3c13485

Will produce a PR shortly.

igrigorik added a commit that referenced this issue Jul 15, 2015
mpb pushed a commit to mpb/performance-timeline that referenced this issue Jul 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants