Closed
Bug 1194884
Opened 9 years ago
Closed 9 years ago
WebM playback only decodes half of the frames
Categories
(Core :: Audio/Video: Playback, defect)
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox41 | --- | unaffected |
firefox42 | + | fixed |
firefox43 | + | fixed |
People
(Reporter: antonio_mario_novo, Assigned: jya)
References
Details
(Keywords: regression)
Attachments
(4 files)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
j
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
j
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (masking-agent; rv:42.0) Gecko/20100101 Firefox/42.0
Build ID: 20150813030208
Steps to reproduce:
Open and play a WebM file and its equivalent MP4 (container), examnple:
http://techslides.com/demos/sample-videos/small.webm
and
http://techslides.com/demos/sample-videos/small.mp4
Actual results:
While the MP4 file plays-back as it should, the WebM lacks _exactly_ half of its frames (not decoded nor painted).
Play-back is still done correctly, no sound or temporal problems arise, but the apparent lack of decent frame rate can be seen.
Expected results:
Playback of WebM files should decode all frames needed to play-back the file the best it can (on the target machine).
Contrary to bug 571386, where frames are dropped somewhat randomly (maybe according to load?) exactly half of the frames on the video file are not decoded.
Also, according to layers.acceleration.draw-fps, both tabs/layers lock in at 75FPS while playing both files (75Hz screen refresh rate). CPU load remains the same in both cases.
Comment 1•9 years ago
|
||
[Tracking Requested - why for this release]:
[Tracking Requested - why for this release]:
I can see the dropping frams in "Show Statics".
https://hg.mozilla.org/mozilla-central/rev/4e883591bb5dff021c108d3e30198a99547eed1e
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:43.0) Gecko/20100101 Firefox/43.0 ID:20150814030208
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=bdb808a2dd66&tochange=195776f0148b
Regressed by: Bug 1185792
Blocks: 1185792
Status: UNCONFIRMED → NEW
status-firefox41:
--- → unaffected
status-firefox42:
--- → affected
status-firefox43:
--- → affected
tracking-firefox42:
--- → ?
tracking-firefox43:
--- → ?
Component: General → Audio/Video: Playback
Ever confirmed: true
Keywords: regression
Version: 43 Branch → 42 Branch
Comment 2•9 years ago
|
||
Assignee | ||
Comment 3•9 years ago
|
||
Jan, can you have a look? seems to be what's causing bug 762774
Flags: needinfo?(j)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → jyavenard
Assignee | ||
Comment 4•9 years ago
|
||
Remove duplicated code.
Also some fixes with log types.
Attachment #8648395 -
Flags: review?(j)
Assignee | ||
Comment 5•9 years ago
|
||
Attachment #8648397 -
Flags: review?(j)
Updated•9 years ago
|
Attachment #8648395 -
Flags: review?(j) → review+
Updated•9 years ago
|
Attachment #8648397 -
Flags: review?(j) → review+
Comment 8•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e8ce7726fa5a
https://hg.mozilla.org/mozilla-central/rev/959cc9cb112f
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Tracking since this is a regression.
Jean-Yves, want to nominate this for uplift?
Assignee | ||
Comment 10•9 years ago
|
||
Yes.
I have a long list to uplift for aurora and beta.
Flags: needinfo?(jyavenard)
Assignee | ||
Comment 11•9 years ago
|
||
Comment on attachment 8648397 [details] [diff] [review]
[webm] P2. Retrieve all VPX decoded frames from decoder. r=j^
Approval Request Comment
[Feature/regressing bug #]: 1185792
[User impact if declined]: webm videos will play with 50% dropped frames
[Describe test coverage new/current, TreeHerder]: in central, local test. confirmed issue fixed by users.
[Risks and why]: Very low, this is now using the same logic as the previous WebMReader VPx decoder.
[String/UUID change made/needed]: None
Attachment #8648397 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
Attachment #8648397 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in
before you can comment on or make changes to this bug.
Description
•