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

Connection between Navigation Timing 2 and Performance Timeline is underdefined #1

Closed
plehegar opened this issue Sep 26, 2014 · 6 comments
Assignees

Comments

@plehegar
Copy link
Member

[[
I see nothing anywhere in
https://dvcs.w3.org/hg/webperf/raw-file/tip/specs/NavigationTiming2/Overview.html
that creates instances of PerformanceNavigationTiming. Should there be
something? There are examples that claim that such instances should
exist and that performance.getEntriesByType should be able to find them,
but I see no normative basis for that claim.
]]
http://lists.w3.org/Archives/Public/public-web-perf/2014Jun/0015.html
(and follow up)

@plehegar plehegar self-assigned this Sep 26, 2014
@igrigorik
Copy link
Member

  • AI: need to clarify with Boris

@toddreifsteck
Copy link
Member

Internet Explorer implements the following but they do not appear in normative text--
performance.getEntriesByName('document') returns navigation timings.
performance.getEntriesByType('navigation') returns navigation timings.

It appears that the intent of the spec was to support accessing the updated PerformanceNavigationTiming via these 2 methods and to keep the original performance.timing returning PerformanceTiming.

Given that, we should update the normative text.

@igrigorik @bzbarsky What do you think?

@igrigorik
Copy link
Member

In the context of the new redirect logic we've been discussing.. I think this becomes a requirement, and I'll have to fix this as part of that pull.

@bzbarsky
Copy link

@toddreifsteck Actually normatively defining when PerformanceNavigationTiming instances are created and added to the performance timeline sounds reasonable, yes. That's what I asked for in that thread 9 months ago.

Once you do that, I think the getEntriesByName/getEntriesByType behavior falls out automatically.

@plehegar
Copy link
Member Author

First attempt at #36

@igrigorik
Copy link
Member

Resolved via #36, closing.

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

4 participants