search

WICG / frame-timing

Frame Timing API
https://wicg.github.io/frame-timing/
Other
170 stars 18 forks source link

The wording around the cpuTime attribute could use some clarification #6

Closed mithro closed 10 years ago

mithro commented 10 years ago

The spec currently says;

cpuTime attribute
The cpuTime attribute must return a [High Resolution Time] duration giving the total CPU time used to process this frame.

It is my understanding that the intention is for the cpuTime on the PerformanceMainFrameTiming to include all cpu time spent in the system (including in the compositor and other parts of the system) to get this main frame onto the screen. Is that correct?

It is probably worth adding something like "(best?) estimate of" to the description too?

natduca commented 10 years ago

Is this dupe of issue #1?

igrigorik commented 10 years ago

No, I believe the intent is to capture the CPU time of the main + compositor threads only. I.e. same as Chrome Devtools provides on paint + layout events.

natduca commented 10 years ago

We'll want to have explicit language that gpu-side costs aren't included. But yeah this is a good bug to talk through the details. Sorry bout the incorrect dupage. :)

mithro commented 10 years ago

Could we also have clear language about this being an estimate? Maybe it's implied with any performance metric?

igrigorik commented 10 years ago

dupe of #1, closing this.