Closed
Bug 1459334
Opened 6 years ago
Closed 6 years ago
No telemetry is logged for the animation inspector
Categories
(DevTools :: General, enhancement, P2)
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.
Reporter | ||
Comment 1•6 years ago
|
||
@pbro What would you like us to do about this?
Flags: needinfo?(pbrosset)
Comment 2•6 years ago
|
||
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)
Comment 3•6 years ago
|
||
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)
Comment 4•6 years ago
|
||
Makes sense.
Forwarding to Mike who was the one who noticed the issue first.
Flags: needinfo?(pbrosset) → needinfo?(mratcliffe)
Reporter | ||
Comment 5•6 years ago
|
||
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
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•