Closed
Bug 1497992
Opened 6 years ago
Closed 6 years ago
Upstream or remove VideoReceiver::Reset
Categories
(Core :: WebRTC: Audio/Video, enhancement, P3)
Tracking
()
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: dminor, Assigned: dminor)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
We added this, we should see if it is still necessary and if so upstream, otherwise remove.
Assignee | ||
Comment 1•6 years ago
|
||
Similar to Bug 1497974, this appears to affect the older, no longer used jitter buffer.
Assignee: nobody → dminor
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•6 years ago
|
||
This ends up calling VCMReceiver::Reset() which resets the state of the VCMJitterBuffer. We no longer use VCMJitterBuffer, which is the old jitter buffer implementation, so this code no longer has any effect and can be removed.
Pushed by dminor@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9e526b3093ee Remove VideoReceiver::Reset; r=pehrsons
![]() |
||
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9e526b3093ee
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in
before you can comment on or make changes to this bug.
Description
•