Closed Bug 1001272 Opened 6 years ago Closed 6 years ago
AEC distorted (especially on Mac) due to 10ms rechunking
A last-second change to patch 3 of bug 694814 (Move AEC to GUM), adding far-end observers, introduced a bug missed in review. When rechunking to 10ms (due to webrtc.org restrictions) the far-end observer code was converted from a hand-coded loop to do sample conversion, to a call to ConvertAudioSamples(). It was reviewed, but missed in the conversion (or accidentally deleted) was the need to now separately bump the input pointer so it would be correct for the next iteration of the for() loop. This led to snippets of repeated audio replacing correct audio, depending on the chunk sizes processed by MSG and passed over. These could be far worse on some hardware/driver setups (such as certain macs). The distorted far-end audio was used only for AEC, so the AEC was only partially functional (again, very system-dependent).
Attachment #8412365 - Flags: review?(paul)
Target Milestone: --- → mozilla31
Priority: -- → P1
Whiteboard: [priority, p=1, ft:webrtc] → [p=1, 1.5:p1, ft:webrtc]
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [p=1, 1.5:p1, ft:webrtc] → [p=1, 1.5:p1, ft:webrtc][s=fx32]
You need to log in before you can comment on or make changes to this bug.