Logging of encoder stats in WebRTC broken by multistream landing

RESOLVED FIXED in Firefox 39

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jesup, Assigned: jesup)

Tracking

Trunk
mozilla40
Points:
---

Firefox Tracking Flags

(firefox38 wontfix, firefox38.0.5 affected, firefox39 fixed, firefox40 fixed, firefox-esr38 fixed)

Details

Attachments

(1 attachment)

+++ This bug was initially created as a clone of Bug #1159489 +++

Spun off the issue with encoder stats being broken since multistream landedi n Fx38 found in bug 1159489

Patch is already r=jib
Assignee: nobody → rjesup
Status: NEW → ASSIGNED
Comment on attachment 8600948 [details] [diff] [review]
Fix VideoCodecStats to allow for collecting encoder and decoder stats

Carry forward r=jib
Attachment #8600948 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/10c082aa3a63
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Comment on attachment 8600948 [details] [diff] [review]
Fix VideoCodecStats to allow for collecting encoder and decoder stats

[Approval Request Comment]
If this is not a sec:{high,crit} bug, please state case for ESR consideration:

I want this for ESR since this enhances the ability of users to report issues with WebRTC.

[Feature/regressing bug #]: Bug 1130534 (merging video conduits)

[User impact if declined]: No encoder stats are available for about:webrtc

[Describe test coverage new/current, TreeHerder]: Manual testing of about:webrtc

[Risks and why]: quite low risk; just ensures it registers for all the appropriate callbacks instead of only the first (of decoder or encoder)

[String/UUID change made/needed]: none
Attachment #8600948 - Flags: approval-mozilla-esr38?
Attachment #8600948 - Flags: approval-mozilla-aurora?
Duplicate of this bug: 1154807
Comment on attachment 8600948 [details] [diff] [review]
Fix VideoCodecStats to allow for collecting encoder and decoder stats

Approved for uplift to aurora.
Attachment #8600948 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
ESR 38.0 already was built on Monday. So unless this is a security fix it may need to wait for the next release of ESR (38.1).
Attachment #8600948 - Flags: approval-mozilla-esr38? → approval-mozilla-esr38+
You need to log in before you can comment on or make changes to this bug.