Closed
Bug 1231169
Opened 9 years ago
Closed 9 years ago
report rust mp4parse track status in telemetry
Categories
(Core :: Audio/Video: Playback, defect)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla46
People
(Reporter: rillian, Assigned: rillian)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
vladan
:
review+
|
Details | Diff | Splinter Review |
Bug 1229615 added a check reporting rust vs stagefright mp4 track counts for audio and video. This bug is about reporting that info through telemetry so we get some feedback on wild file handling.
Assignee | ||
Comment 1•9 years ago
|
||
WIP patch
Assignee | ||
Comment 2•9 years ago
|
||
Comment on attachment 8696723 [details] [diff] [review]
Report demuxer track comparison in telemetry
It builds! https://treeherder.mozilla.org/#/jobs?repo=try&revision=66dea9c21010&selectedJob=14468911
Adding ally for data peer review. Are you still doing these?
Attachment #8696723 -
Flags: review?(kinetik)
Attachment #8696723 -
Flags: feedback?(a.m.naaktgeboren)
Updated•9 years ago
|
Attachment #8696723 -
Flags: review?(kinetik) → review+
Assignee | ||
Updated•9 years ago
|
Attachment #8696723 -
Flags: feedback?(vladan.bugzilla)
Comment 3•9 years ago
|
||
Comment on attachment 8696723 [details] [diff] [review]
Report demuxer track comparison in telemetry
Review of attachment 8696723 [details] [diff] [review]:
-----------------------------------------------------------------
::: toolkit/components/telemetry/Histograms.json
@@ +6276,5 @@
> },
> + "MEDIA_RUST_MP4PARSE_TRACK_MATCH_AUDIO": {
> + "expires_in_version": "50",
> + "kind": "boolean",
> + "description": "(Bug 1231169) Whether rust and stagefight mp4 parser audio track results match.",
put the bug #in a separate bug_numbers field
add alert_emails fields
Attachment #8696723 -
Flags: feedback?(vladan.bugzilla)
Assignee | ||
Comment 4•9 years ago
|
||
Thanks. More like this?
Carrying r=kinetik for media review.
Attachment #8696723 -
Attachment is obsolete: true
Attachment #8696723 -
Flags: feedback?(a.m.naaktgeboren)
Attachment #8697690 -
Flags: review?(vladan.bugzilla)
Comment 5•9 years ago
|
||
Comment on attachment 8697690 [details] [diff] [review]
Report demuxer track comparison in telemetry v2
Review of attachment 8697690 [details] [diff] [review]:
-----------------------------------------------------------------
This looks good
Attachment #8697690 -
Flags: review?(vladan.bugzilla) → review+
Comment 7•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in
before you can comment on or make changes to this bug.
Description
•