Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Fix: Firefox console errors TypeError: this.preferredCodec.split is not a function #310

Merged
merged 3 commits into from
Jul 18, 2023

Conversation

Belchy06
Copy link
Collaborator

Relevant components:

  • Signalling server
  • Frontend library
  • Frontend UI library
  • Matchmaker
  • Platform scripts
  • SFU

Problem statement:

Addresses #309

Solution

As of 06/2023, Firefox has added RTCRtpReceiver.getCapabilities, but hasn't added the ability to set codec preferences with transceiver.setCodecPreferences. By simply checking if this method exists on a transceiver we can know if we are using Firefox or Chrome.

Copy link
Contributor

@lukehb lukehb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm. Do we indicate in the frontend panel that setting codec preference is Chrome only. That is, do we grey out the box on FF or something like this?

@Belchy06
Copy link
Collaborator Author

Dropdown is disabled and displays 'Chrome Only'.

Copy link
Contributor

@lukehb lukehb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Belchy06 Belchy06 merged commit 503b565 into EpicGames:master Jul 18, 2023
github-actions bot pushed a commit that referenced this pull request Jul 18, 2023
…not a function` (#310)

* Fix select codec on Firefox

* Restore indentation in cirrus

(cherry picked from commit 503b565)
github-actions bot pushed a commit that referenced this pull request Jul 18, 2023
…not a function` (#310)

* Fix select codec on Firefox

* Restore indentation in cirrus

(cherry picked from commit 503b565)
@github-actions
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
UE5.2
UE5.3

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Belchy06 added a commit that referenced this pull request Jul 18, 2023
…not a function` (#310) (#312)

* Fix select codec on Firefox

* Restore indentation in cirrus

(cherry picked from commit 503b565)

Co-authored-by: William Belcher <[email protected]>
Belchy06 added a commit that referenced this pull request Jul 18, 2023
…not a function` (#310) (#313)

* Fix select codec on Firefox

* Restore indentation in cirrus

(cherry picked from commit 503b565)

Co-authored-by: William Belcher <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

[BUG] - Firefox console errors TypeError: this.preferredCodec.split is not a function
2 participants