Closed Bug 1460753 Opened 4 years ago Closed 4 years ago

TOTAL_SCROLL_Y and PAGE_MAX_SCROLL_Y have expired

Categories

(Core :: Graphics, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: rhunt, Assigned: rhunt)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

TOTAL_SCROLL_Y and PAGE_MAX_SCROLL_Y both expired in 58 so we're not getting any more data from these. [1]

[1] https://searchfox.org/mozilla-central/rev/3f17a234769d25fca5144ebb8abc8e1cb3c56c16/toolkit/components/telemetry/Histograms.json#12206
Harald do we want to renew these or remove the code here?
Flags: needinfo?(hkirschner)
Our recent discussions around performance have started to look more into user interaction with pages as a measure of quality. I suggest to renew these.
(In reply to :Dominik Strohmeier [:dstrohmeier] from comment #2)
> Our recent discussions around performance have started to look more into
> user interaction with pages as a measure of quality. I suggest to renew
> these.

I'm fine with it being renewed, but someone needs to file the bug and take ownership of the probe.

If not, we really should remove this probe as it instruments a code path that is hot, and currently the results aren't even being collected.
Vicky, what do you think about the perf team owning that probe? The hypothesis was that these metrics would be impacted by scroll performance improvements (like webrender), but they need more testing & investment.
Flags: needinfo?(hkirschner) → needinfo?(vchin)
Stuart started an effort to review all the telemetry probes, is this bug a result of that effort? 
Reviewing the probes and understanding what data we want to gather to measure performance is on the list of things I'd like to discuss when Stuart is back.
Flags: needinfo?(vchin)
(In reply to Vicky Chin [:vchin] from comment #5)
> Stuart started an effort to review all the telemetry probes, is this bug a
> result of that effort? 
> Reviewing the probes and understanding what data we want to gather to
> measure performance is on the list of things I'd like to discuss when Stuart
> is back.

Not exactly, I implemented the original probe here for Harald and just recently
noticed the probe was expired so I filed this bug.
As the probe is currently expired and there's no clear owner for renewing it, I'd like to remove it. It can always be added back in if desired in the future.

Let me know if there are any objections.
Attachment #8987997 - Flags: review?(bugs)
Attachment #8987997 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/999300adcd1e
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.