Closed Bug 1505858 Opened 2 years ago Closed 2 years ago
Record full paint time before waiting for composite in telemetry (CONTENT
_FULL _PAINT _TIME)
4.48 KB, patch
|Details | Diff | Splinter Review|
3.85 KB, patch
|Details | Diff | Splinter Review|
47 bytes, text/x-phabricator-request
|Details | Review|
2.19 KB, text/plain
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
Assignee: nobody → jmuizelaar
Priority: -- → P2
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).
This seems to work but I wasn't able to see much time being spent in OMTP so I'm not positive.
I took a different approach to exposing RenderStats in the patch on bug 1499251. I'd say it's cleaner but more invasive. Thoughts?
(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.
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+
Attachment #9025247 - Flags: review?(chutten) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/17d1440b76b1 Record full paint time before waiting for composite in telemetry. r=mattwoodrow
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
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
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/236208271581 Record full paint time before waiting for composite in telemetry. r=mattwoodrow
Summary: Record full paint time before waiting for composite in telemetry → Record full paint time before waiting for composite in telemetry (CONTENT_FULL_PAINT_TIME)
You need to log in before you can comment on or make changes to this bug.