Closed
Bug 895754
Opened 12 years ago
Closed 12 years ago
Update VideoPlaybackQuality to current MSE spec (remove playbackJitter, add totalFrameDelay)
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla25
People
(Reporter: kinetik, Assigned: kinetik)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
11.82 KB,
patch
|
cajbir
:
review+
|
Details | Diff | Splinter Review |
See https://www.w3.org/Bugs/Public/show_bug.cgi?id=22148 for details.
|
Assignee | |
Comment 1•12 years ago
|
||
|
|
Assignee | |
Updated•12 years ago
|
Attachment #778283 -
Attachment description: Update VideoPlaybackQuality to current MSE spec (remove playbackJitter, add totalFrameDelay). r=doublec → patch v0
Attachment #778283 -
Flags: review?(chris.double)
|
||
Comment 2•12 years ago
|
||
Comment on attachment 778283 [details] [diff] [review]
patch v0
Review of attachment 778283 [details] [diff] [review]:
-----------------------------------------------------------------
::: content/media/MediaDecoder.h
@@ +883,1 @@
> // Access protected by mStatsReentrantMonitor.
Should mStatsReentrantMonitor be mReentrantMonitor in this and other comments in this area? It's not part of this patch but it'd be good to correct if it's wrong.
Attachment #778283 -
Flags: review?(chris.double) → review+
|
|
Assignee | |
Comment 3•12 years ago
|
||
(In reply to Chris Double (:doublec) from comment #2)
> Should mStatsReentrantMonitor be mReentrantMonitor in this and other
> comments in this area? It's not part of this patch but it'd be good to
> correct if it's wrong.
Thanks, fixed this on checkin.
https://hg.mozilla.org/integration/mozilla-inbound/rev/63020f14491b
|
||
Comment 4•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in
before you can comment on or make changes to this bug.
Description
•