Closed Bug 1373285 Opened 7 years ago Closed 7 years ago

Remove telemetry for Keyboard APZ

Categories

(Core :: Panning and Zooming, defect, P2)

55 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox54 --- unaffected
firefox55 --- fixed
firefox56 --- fixed

People

(Reporter: rhunt, Assigned: rhunt)

References

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

Per bug 1351783 comment 76, this telemetry probe is no longer needed and should be removed from central and beta.
OS: Unspecified → All
Priority: -- → P2
Hardware: Unspecified → All
Version: unspecified → 55 Branch
Since keyboard APZ won't go into 55, we could just do a wholesale backout of bug 1357880 in 55. That would be relatively straightforward. In 56 I assume the keyboard-apz patches might require part of the machinery added for the telemetry so we'd want a partial backout rather than a full backout.
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #1)
> Since keyboard APZ won't go into 55, we could just do a wholesale backout of
> bug 1357880 in 55. That would be relatively straightforward. In 56 I assume
> the keyboard-apz patches might require part of the machinery added for the
> telemetry so we'd want a partial backout rather than a full backout.

Hmm. So the one thing that might be used in key-apz is a flag added in nsGlobalWindow for MayHaveMouseMoveListeners. Per bug 1351783 comment 72, the flag isn't effective for iframes so I'm not sure whether it will be used. It won't be in the initial landing.

I think I'd like to get it all backed out then, and get that part added back if necessary.
See Also: → 1357880
Comment on attachment 8878137 [details]
Bug 1373285 - Remove keyboard apz telemetry probes.

https://reviewboard.mozilla.org/r/149532/#review154108
Attachment #8878137 - Flags: review?(bugs) → review+
Pushed by rhunt@eqrion.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f6b101870099
Remove keyboard apz telemetry probes. r=smaug
https://hg.mozilla.org/mozilla-central/rev/f6b101870099
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Comment on attachment 8878137 [details]
Bug 1373285 - Remove keyboard apz telemetry probes.

Approval Request Comment
[Feature/Bug causing the regression]: No regression, unneeded telemetry probe
[User impact if declined]: Potential slight performance regression when adding event listeners to an EventTarget
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: Removal of telemtry probe that isn't needed
[String changes made/needed]: None
Attachment #8878137 - Flags: approval-mozilla-beta?
Requesting beta uplift for removing a telemetry probe that we have all the data for and we didn't plan on letting go to release.
Comment on attachment 8878137 [details]
Bug 1373285 - Remove keyboard apz telemetry probes.

telemetry probe removal, beta55+
Attachment #8878137 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: