1231169 - report rust mp4parse track status in telemetry

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect)

Description

[Ralph Giles (:rillian)]

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.

Comment 1

[Ralph Giles (:rillian)]

Attachment: Report demuxer track comparison in telemetry

WIP patch

Comment 2

[Ralph Giles (:rillian)]

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?

Comment 3

[Vladan Djeric (:vladan)]

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

Comment 4

[Ralph Giles (:rillian)]

Attachment: Report demuxer track comparison in telemetry v2

Thanks. More like this?

Carrying r=kinetik for media review.

Comment 5

[Vladan Djeric (:vladan)]

Comment on attachment 8697690 [details] [diff] [review]
Report demuxer track comparison in telemetry v2

Review of attachment 8697690 [details] [diff] [review]:
-----------------------------------------------------------------

This looks good

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1526190a14f1

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/1526190a14f1