firefox-devtools/profiler

Clarify that CPU is between samples and category&stack are at sample time

Aperta

#3240 aperta il 25 mar 2021

 (5 commenti) (1 reazione) (0 assegnatari)TypeScript (486 fork)auto 404
help wantedpolishtimeline

Metriche repository

Star
 (1515 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

When CPU numbers are shown, the tooltip looks like this: profiler-cpu-tooltip So it's something like this:

Category: [] Layout... CPU Usage: 97% (...) Stack: parse_into()... ...

I'm concerned that we're mixing two "domains" of information, and it may give the wrong impression, e.g.: "Wow, this parse_into in layout is taking 97% of a CPU!". But in fact, all we can say from this data is: Since the previous sample and now, we have used 97% of a CPU; And at this exact sample time we were calling parse_into in layout.

So I think we should clarify the difference. Suggestion:

CPU Usage since previous sample: 97% (...) ..................................................................................... Sample at time: 0.8465s Category: [] Layout... Stack: parse_into()... ...

This clearly separates the more continuous CPU numbers from the point-like sample data.

As a bonus it shows the precise timestamp. I don't know how valuable that is, but for example it could help find markers at that time in the Marker Table.

┆Issue is synchronized with this Jira Task

Guida contributor