HTMLMediaElement::UpdateReadyStateForData can lead to invalid audio format being reported.

RESOLVED FIXED in Firefox 40

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jya, Assigned: jya)

Tracking

Trunk
mozilla40
x86
macOS
Points:
---

Firefox Tracking Flags

(firefox40 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
Under some circumstances, invalid audio track format can be reported.

HTMLMediaElement::UpdateReadyStateForData calls MetadataLoaded with a dummy MediaInfo object that is only properly constructed for the video track.

Problem was made worse by bug 1129263 as prior the invalid data was mostly ignored.

The MediaInfo object passed, contains the information from the default AudioInfo constructor. As such we will only ever report audio with 44.1kHz, Stereo with a track ID of 2.
(Assignee)

Updated

4 years ago
Depends on: 1153049
(Assignee)

Comment 1

4 years ago
Created attachment 8591494 [details] [diff] [review]
Remove redundant mMediaSize member

Remove mMediaSize class member (and use mMediaInfo instead) and various reworks. The actual fix for this bug will come for free with bug 1153049. Various fixes/cleans along the way introduced in bug 1129263. Fix 'resize' event being fired twice
Attachment #8591494 - Flags: review?(roc)
(Assignee)

Updated

4 years ago
Assignee: nobody → jyavenard
Status: NEW → ASSIGNED
(Assignee)

Comment 2

4 years ago
Comment on attachment 8591494 [details] [diff] [review]
Remove redundant mMediaSize member

this cause resize event to be fired out of order. better fix coming
Attachment #8591494 - Attachment is obsolete: true
Attachment #8591494 - Flags: review?(roc)
(Assignee)

Updated

4 years ago
See Also: → bug 951215
Comment on attachment 8591564 [details] [diff] [review]
Remove redundant mMediaSize member

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

nice
Attachment #8591564 - Flags: review?(roc) → review+
https://hg.mozilla.org/mozilla-central/rev/818a69e48d38
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox40: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.