w3c / performance-timeline

Performance Timeline
https://w3c.github.io/performance-timeline/
Other
111 stars 27 forks source link

API shape for monitoring new navigation-like entries such as App History, BFCache, etc #182

Open nicjansma opened 3 years ago

nicjansma commented 3 years ago

As a followup to recent changes for the WICG App History proposal to integrate it with the PerformanceTimeline, we discussed on a recent W3C WebPerf call what kind of API shape would be ideal for monitoring all of the new possible navigation entry types, e.g.:

One question is: How we want these new navigation types to integrate with the existing PerformanceTimeline and PerformanceObservers?

Today, you simply get a single "the navigation" NavigationTiming entry if you do this:

const observer = new PerformanceObserver(...);
observer.observe({ type: "navigation" });

With these new possible "navigation-like" types, would you get additional entries from the above observer?

Or should we expose them elsewhere?

For example, we may want to require developers to specifically observe new types if they want them (for back-compat):

const observer = new PerformanceObserver(...);
observer.observe({ types: ["navigation", "same-document-navigation"] });

We could also offer a new "super" type to listen for, that would emit entries for the above navigations, plus any new ones that are created in the future:

const observer = new PerformanceObserver(...);
observer.observe({ type: "all-navigations" });

The alternative is to simply emit these new ones under the navigation observer, as IIRC we crawled the web and found that nearly everyone looking at NavigationTiming from a PerformanceObserver was just looking at the [0]th entry.

We had some discussion about this on a recent call, and you can find our minutes here;

This issue is intended as a continuation of that discussion to gather more feedback on the above ideas.

yoavweiss commented 3 years ago

^^ @domenic

domenic commented 3 years ago

Thanks for starting this thread!

From the web developers we've talked to who work on SPAs, they expect to not start getting SPA navs from their cross-document nav telemetry endpoints, and they expect to get a different set of data. In particular they found the idea of getting all the network timing stuff for SPA navs confusing.

Also, a couple points of clarification:

clelland commented 3 years ago

In meeting with a number of RUM analytics providers, Google has heard solid support for a new event type for bfcache navs, rather than exposing them as PerformanceNavigationTiming events. It wouldn't need all of the network-level timing members.

There's also a lot of interest, in that case, to have a future-proof "all-navigations" observer (we didn't ask for opinions on the name there; I'm sure there's room to bikeshed)

clelland commented 3 years ago

192 is the start of an attempt to add support for this.

It seems that there are a couple of ways we could go to have the spec recognize certain performance entries as 'navigations':

  1. The simplest and least extensible is to hard code the list of types which should be considered navigations into this spec, naming them as specific strings.
  2. Just a little bit better is to add an internal boolean isNavigation to PerformanceEntry. This would be set by navigation timing events, app history navs, bfcache navs, etc. In algorithms, we'd use that boolean to match events of pseudo-"type" all-navigations. This makes navigations special, and requires specific handling for them in this spec.
  3. We could instead add a category enum to PerformanceEntry, with values like "Navigation", "Resource", "User", etc, and then specs which create PerformanceEntry objects will need to choose a category for their events. This is similar to the first option, but navigations aren't special: Every category could have similar treatment. It still requires central control over the category list in the PerformanceTimeline spec.
  4. (Most flexible and decentralized) We could make the category a free-form string, set by each spec independently. NavigationTiming, AppHistory, and bfcache navs would all set this to "navigations" or something similar, and PerformanceTimeline wouldn't know about the values that any particular entry uses. Getting entries of "type" all-X would just return all those with category X.

These should all be invisible to the user; this is just about spec ergonomics, and how easy it is to expand on this idea in the future. My inclination is to avoid over-engineering by starting with #2 (an isNavigation boolean) and moving to the more complex solutions when and only when there is a real use case for them.

clelland commented 1 year ago

There's also been significant discussion of this on https://github.com/WICG/soft-navigations/issues/12, but I'll try to steer that discussion back here, since this is where it would eventually need to be implemented.

It sounds like what we need is

And maybe, for ergonomics: