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

Consider adding deliveryType support #397

Open
tunetheweb opened this issue Oct 18, 2023 · 1 comment
Open

Consider adding deliveryType support #397

tunetheweb opened this issue Oct 18, 2023 · 1 comment

Comments

@tunetheweb
Copy link
Member

Chrome recently added deliveryType which can be set to cached, navigational-prefetch or not empty string ''.

This is available at performance.getEntriesByType('navigation')[0].deliveryType for the document which could be useful to explain the TTFB (and hence FCP and LCP).

Should this supplement/expand the navigationType in the Metric type? Or should this be a separate field? They are not completely distinct (and may be even less so in the future if the usage of deliveryType expands), but maybe just splitting the navigate type into cached, navigational-prefetch, or navigate could be sufficient for now?

@tunetheweb
Copy link
Member Author

tunetheweb commented Oct 19, 2023

We decided this should be a separate field rather than merged into navigationType as there are instances where you'd need both. And given it's then just a straight look up from the navigation entry (which is already available in the attribution data) there is little need to add it.

However, leaving this issue open as might add a comment similar to serverTiming comment so people are aware of this:

/**
* The `navigation` entry of the current page, which is useful for diagnosing
* general page load issues. This can be used to access `serverTiming` for example:
* navigationEntry?.serverTiming
*/
navigationEntry?: PerformanceNavigationTiming | NavigationTimingPolyfillEntry;

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

1 participant