Closed Bug 1569815 Opened 6 years ago Closed 5 years ago

Fix broken audio in WebEx WebRTC calls

Categories

(Web Compatibility :: Interventions, enhancement, P1)

enhancement

Tracking

(firefox68+ wontfix, firefox69 wontfix, firefox70 wontfix, firefox71 wontfix)

RESOLVED INCOMPLETE
Tracking Status
firefox68 + wontfix
firefox69 --- wontfix
firefox70 --- wontfix
firefox71 --- wontfix

People

(Reporter: drno, Assigned: drno)

References

Details

Attachments

(5 files)

In bug 1545144 we found that a recent fix in Firefox 68 results in WebEx calls via WebRTC to no longer having audio.

I just confirmed that we can in fact build an intervention add-on which needs to intercept RTCPeerConnection.setRemoteDescription() on all *.webex.com domains and change the normal behavior of that function.

This is a JsFiddle which demonstrates the workaround: https://jsfiddle.net/4w27eu5m/2/

Summary: Fix WebEx WebRTC calls → Fix broken audio in WebEx WebRTC calls
Priority: -- → P1
Assignee: nobody → drno

This seems to work ok: https://jsfiddle.net/v6z1nk04/9/

Tom, could you potentially help me figure out how to properly get RTCPeerConnection.setRemoteDescription() overloaded (what I'm trying in the attached patch does not appear to work)?

Flags: needinfo?(wisniewskit)

(I spoke with Nils on Slack and hopefully helped get things to the point where he can focus on just the most important parts of the patch).

Flags: needinfo?(wisniewskit)

Status update: we got the initial version working which overloads the setRemoteDescription() method. But this results in no audio packets getting exchanged at all any more. My guess is the super fast re-negotiating triggers some other internal bugs.

Trying now to do the re-negotiation only after the connection has been established.

Depends on D40536

Depends on D40571

Nils, do you have plans to keep working on this? Or should we close?

Flags: needinfo?(drno)

Thomas and me tried for quite some time, but gave up in the end. Unfortunate.

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(drno)
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: