From e31ad0e174eb7f4c4a69c670ca40f7868983eb5c Mon Sep 17 00:00:00 2001
From: Travis Ralston <travpc@gmail.com>
Date: Thu, 25 Mar 2021 18:24:58 -0600
Subject: [PATCH] Don't specify sample rates for voice messages

Turns out the browser doesn't actually resample for us, instead opting to explode in sadness.

We'll leave the resampling to the opus encoder.

Fixes https://github.com/vector-im/element-web/issues/16775
---
 src/voice/VoiceRecorder.ts | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/voice/VoiceRecorder.ts b/src/voice/VoiceRecorder.ts
index 06c0d939fc..6bf4189d8e 100644
--- a/src/voice/VoiceRecorder.ts
+++ b/src/voice/VoiceRecorder.ts
@@ -51,9 +51,6 @@ export class VoiceRecorder {
     private async makeRecorder() {
         this.recorderStream = await navigator.mediaDevices.getUserMedia({
             audio: {
-                // specify some audio settings so we're feeding the recorder with the
-                // best possible values. The browser will handle resampling for us.
-                sampleRate: SAMPLE_RATE,
                 channelCount: CHANNELS,
                 noiseSuppression: true, // browsers ignore constraints they can't honour
                 deviceId: CallMediaHandler.getAudioInput(),
@@ -61,7 +58,6 @@ export class VoiceRecorder {
         });
         this.recorderContext = new AudioContext({
             latencyHint: "interactive",
-            sampleRate: SAMPLE_RATE, // once again, the browser will resample for us
         });
         this.recorderSource = this.recorderContext.createMediaStreamSource(this.recorderStream);
         this.recorderFreqNode = this.recorderContext.createAnalyser();