Closed
Bug 1208799
Opened 6 years ago
Closed 6 years ago
Last video track is being shown in multi video track webms
Categories
(Core :: Audio/Video: Playback, defect)
Core
Audio/Video: Playback
Tracking
()
VERIFIED
FIXED
mozilla44
People
(Reporter: kagami, Assigned: jya)
References
Details
(Keywords: regression, testcase)
Attachments
(2 files)
23.99 KB,
video/webm
|
Details | |
1.61 KB,
patch
|
kinetik
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → jyavenard
Assignee | ||
Updated•6 years ago
|
Keywords: regression
Assignee | ||
Comment 1•6 years ago
|
||
Updated•6 years ago
|
Status: UNCONFIRMED → ASSIGNED
Component: Untriaged → Audio/Video: Playback
Ever confirmed: true
Keywords: testcase
OS: Unspecified → All
Product: Firefox → Core
Hardware: Unspecified → All
Assignee | ||
Comment 2•6 years ago
|
||
Comment on attachment 8666430 [details] [diff] [review] [webm] Use first track found. forgot the r? again :(
Attachment #8666430 -
Flags: review?(kinetik)
Updated•6 years ago
|
Attachment #8666430 -
Flags: review?(kinetik) → review+
Assignee | ||
Comment 3•6 years ago
|
||
ni? myself so uplifting doesn't go below the radar
Flags: needinfo?(jyavenard)
Assignee | ||
Updated•6 years ago
|
status-firefox41:
--- → wontfix
status-firefox42:
--- → affected
status-firefox43:
--- → affected
status-firefox44:
--- → affected
Comment 5•6 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/14c82d6b268c
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Assignee | ||
Comment 6•6 years ago
|
||
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 7•6 years ago
|
||
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+
Updated•6 years ago
|
Flags: qe-verify+
Comment 10•6 years ago
|
||
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
Comment 11•6 years ago
|
||
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.
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•