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

Added startTime to IDL #57

Closed
wants to merge 1 commit into from
Closed

Conversation

yoavweiss
Copy link
Contributor

Closes #56

@igrigorik
Copy link
Member

@plehegar
Copy link
Member

Might be a dependency issue. v1 relies on the old performance timeline spec at the moment.
https://rawgit.com/w3c/resource-timing/V1/index.html#performanceresourcetiming

If we keep it that way, we're probably going to create more confusion than it is worth. We should consider linking to Performance Timeline v2 from RT v1 instead and convince the Director it's better that way instead of creating confusion.

@yoavweiss
Copy link
Contributor Author

Isn't it defined on PerformanceEntry? https://www.w3.org/TR/performance-timeline-2/#dom-performanceentry

It certainly is. My bad :/

Might be a dependency issue. v1 relies on the old performance timeline spec at the moment.
https://rawgit.com/w3c/resource-timing/V1/index.html#performanceresourcetiming

If we keep it that way, we're probably going to create more confusion than it is worth. We should consider linking to Performance Timeline v2 from RT v1 instead and convince the Director it's better that way instead of creating confusion.

Seems like current spec's definition of PerformanceEntry leads to https://www.w3.org/TR/performance-timeline-2/#performanceentry

I think we can close this.

@yoavweiss yoavweiss closed this May 31, 2016
@plehegar
Copy link
Member

indeed. No action needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants