Closed Bug 1459334 Opened 6 years ago Closed 6 years ago

No telemetry is logged for the animation inspector

Categories

(DevTools :: General, enhancement, P2)

57 Branch
enhancement

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: miker, Unassigned)

References

(Blocks 1 open bug)

Details

The id of the animation inspector is "newanimationinspector" but we are expecting "animationinspector" so we don't log any telemetry for it.
@pbro What would you like us to do about this?
Flags: needinfo?(pbrosset)
Probably related to the ongoing migration from the old to the new animation inspector. The panel is being replaced with a new implementation (in React) Daisuke is doing the work on this one, and the old animationinspector will get deleted very soon. Let me needinfo Daisuke here as he probably knows about this.
Flags: needinfo?(pbrosset) → needinfo?(dakatsuka)
Thanks, Mike, Patrick. We will delete the old one after QA passed. I think, it would able to be by the end of May. May we wait for that time?? (Even so we should be able to log for channels except Nightly.)
Flags: needinfo?(dakatsuka) → needinfo?(pbrosset)
Makes sense. Forwarding to Mike who was the one who noticed the issue first.
Flags: needinfo?(pbrosset) → needinfo?(mratcliffe)
Bug 1458206 includes changes that send a telemetry event for "animationinspector" if the toolid === "newanimationinspector" || "animationinspector" so we don't really need this bug.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(mratcliffe)
Resolution: --- → WONTFIX
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.