Closed Bug 1208799 Opened 5 years ago Closed 5 years ago

Last video track is being shown in multi video track webms

Categories

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

Product:
Core
Component:
Audio/Video: Playback
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla44
Tracking Flags:
Tracking Status
firefox41 --- wontfix
firefox42 --- verified
firefox43 --- verified
firefox44 --- verified

People

(Reporter: kagami, Assigned: jya)

References

Details

(Keywords: regression, testcase)

Attachments

(2 files)

multi.webm
23.99 KB, video/webm
Details
[webm] Use first track found.
1.61 KB, patch
kinetik
: review+
Sylvestre
: approval-mozilla-aurora+
Sylvestre
: approval-mozilla-beta+
Details | Diff | Splinter Review
Attached video multi.webm — 
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:40.0) Gecko/20100101 Firefox/40.0
Build ID: 20150826185918

Steps to reproduce:

1. Create WebM with two different VP8 video tracks.


Actual results:

Firefox 40.0.3 displays first video track.
Firefox 44.0a1 displays second (last) video track.


Expected results:

Firefox 44.0a1 should display first video track too.
Depends on: 1148102
Assignee: nobody → jyavenard
Keywords: regression
Status: UNCONFIRMED → ASSIGNED
Component: Untriaged → Audio/Video: Playback
Ever confirmed: true
Keywords: testcase
OS: Unspecified → All
Product: Firefox → Core
Hardware: Unspecified → All
Comment on attachment 8666430 [details] [diff] [review]
[webm] Use first track found.

forgot the r? again :(
Attachment #8666430 - Flags: review?(kinetik)
Attachment #8666430 - Flags: review?(kinetik) → review+
ni? myself so uplifting doesn't go below the radar
Flags: needinfo?(jyavenard)
https://hg.mozilla.org/mozilla-central/rev/14c82d6b268c
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox44: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Comment on attachment 8666430 [details] [diff] [review]
[webm] Use first track found.

Approval Request Comment
[Feature/regressing bug #]: 1148102
[User impact if declined]: Regression. Wrong video or audio tracks may be displayed
[Describe test coverage new/current, TreeHerder]: Local test, in central
[Risks and why]: None, apply same logic as old WebMReader
[String/UUID change made/needed]: None
Flags: needinfo?(jyavenard)
Attachment #8666430 - Flags: approval-mozilla-beta?
Attachment #8666430 - Flags: approval-mozilla-aurora?
Comment on attachment 8666430 [details] [diff] [review]
[webm] Use first track found.

Regression, taking it.
Attachment #8666430 - Flags: approval-mozilla-beta?
Attachment #8666430 - Flags: approval-mozilla-beta+
Attachment #8666430 - Flags: approval-mozilla-aurora?
Attachment #8666430 - Flags: approval-mozilla-aurora+
Flags: qe-verify+
Reproduced with 44.0a1 from 2015-09-22 under Windows 7 64-bit → 2nd track is displayed with the attached test case.
Verified fixed with 43.0a2 and 44.0a1 (both from 2015-09-30).

[1] Mac OS X 10.10.4, Ubuntu 14.04 32-bit and Windows 7 64-bit
Status: RESOLVED → VERIFIED
status-firefox43: fixed → verified
status-firefox44: fixed → verified
Confirming this fix with latest 42.0b3 (20151001142456) too, under Windows 7 64-bit, Mac OS X 10.10.4 and Ubuntu 12.04 64-bit.
status-firefox42: fixed → verified
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.