Jeff Muizelaar [:jrmuizel] Assignee
	Description • 6 years ago

We currently have CONTENT_PAINT_TIME which just includes main thread time. For teasing apart CONTENT_FRAME_TIME it would be pretty useful to have a metric that includes all of the time before wainting for the composite/render.

With non-WR this would include CONTENT_PAINT_TIME + OMTP_TIME + (maybe time to arrive at compositor)
With WR this would be CONTENT_PAINT_TIME + SCENE_BUILDING_TIME

	Matt Woodrow (:mattwoodrow)
	Comment 1 • 6 years ago

Making both record the marker at the point we reach the compositor makes sense, since that's where the vsync message gets sent, and we drop a frame if the vsync gets there first.

Note that non-WR can do sometimes substantial amounts of work here (deserializing the layers update, maybe uploads), but we want to record the time before that work (since that work blocks the vsync message from arriving).

	Jeff Muizelaar [:jrmuizel] Assignee
	Comment 3 • 6 years ago

This seems to work but I wasn't able to see much time being spent in OMTP so I'm not positive.

	Jeff Muizelaar [:jrmuizel] Assignee
	Comment 5 • 6 years ago

I took a different approach to exposing RenderStats in the patch on bug 1499251. I'd say it's cleaner but more invasive. Thoughts?

	Jeff Muizelaar [:jrmuizel] Assignee
	Comment 6 • 6 years ago

(In reply to Jeff Muizelaar [:jrmuizel] from comment #5)
> I took a different approach to exposing RenderStats in the patch on bug
> 1499251. I'd say it's cleaner but more invasive. Thoughts?

Wrong bug.

	Chris H-C :chutten
	Comment 8 • 6 years ago

Comment on attachment 9025247 [details]
request.md

Preliminary note:

  For permanent collections it is often a good idea to have an automated test ensuring that data continues to be collected correctly. This reduces the chance of future code changes ruining a collection that might not be looked at as often as one that expires and is renewed.

DATA COLLECTION REVIEW RESPONSE:

    Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate? 

Yes. Standard Telemetry mechanisms apply.

    Is there a control mechanism that allows the user to turn the data collection on and off?

Yes. Standard Telemetry mechanisms apply.


    If the request is for permanent data collection, is there someone who will monitor the data over time?

Yes, Jeff Muizelaar.

    Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under? **

Category 1, Technical.

    Is the data collection request for default-on or default-off?

Default on, all channels.

    Does the instrumentation include the addition of any new identifiers (whether anonymous or otherwise; e.g., username, random IDs, etc. See the appendix for more details)?

No.

    Is the data collection covered by the existing Firefox privacy notice? 

Yes.

    Does there need to be a check-in in the future to determine whether to renew the data?

No. Permanent collection.

---
Result: datareview+

	Pulsebot
	Comment 9 • 6 years ago

Pushed by jmuizelaar@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/17d1440b76b1
Record full paint time before waiting for composite in telemetry. r=mattwoodrow

	Narcis Beleuzu [:NarcisB]
	Comment 10 • 6 years ago

Backed out for bustages on WebRenderBridgeParent.cpp

Backout link: https://hg.mozilla.org/integration/autoland/rev/3862dac2171ec28710461663df8e86f5f5d748f9
Push link: https://hg.mozilla.org/integration/autoland/rev/17d1440b76b1c3cc4bfecb11b75c457da95dcf23
Log link: https://treeherder.mozilla.org/logviewer.html#?job_id=212092636&repo=autoland&lineNumber=24228

	Narcis Beleuzu [:NarcisB]
	Comment 11 • 6 years ago

There were also bustages "Unified_cpp_gfx_layers11.i_o] Error 1"
Log link: https://treeherder.mozilla.org/logviewer.html#?job_id=212092664&repo=autoland&lineNumber=20168

	Pulsebot
	Comment 12 • 6 years ago

Pushed by jmuizelaar@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/236208271581
Record full paint time before waiting for composite in telemetry. r=mattwoodrow

	Raul Gurzau (:RaulG)
	Comment 13 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/236208271581