From b82d4282df8640c944f3f87d029de6b0f9ff2d4e Mon Sep 17 00:00:00 2001
From: Florent Castelli <orphis@chromium.org>
Date: Fri, 3 May 2019 07:00:59 -0700
Subject: [PATCH] Implement and ship RTCRtpTransceiver.setCodecPreferences

Intent thread: https://groups.google.com/a/chromium.org/d/msg/blink-dev/DRz-uHqPCLw/ZNDiwL0ZAwAJ

Bug: 891556
Change-Id: I3a851ca721f6ec29971e9a9ca76df1966a6b1ad1
---
 ...RTCRtpTransceiver-setCodecPreferences.html | 69 ++++++++++++++++---
 1 file changed, 58 insertions(+), 11 deletions(-)

diff --git a/webrtc/RTCRtpTransceiver-setCodecPreferences.html b/webrtc/RTCRtpTransceiver-setCodecPreferences.html
index a1f7854e1a2f18..c553c9d8bc7964 100644
--- a/webrtc/RTCRtpTransceiver-setCodecPreferences.html
+++ b/webrtc/RTCRtpTransceiver-setCodecPreferences.html
@@ -26,7 +26,7 @@
           RTCRtpTransceiver on which the method is called. Additionally, the
           RTCRtpCodecParameters dictionary members cannot be modified. If
           codecs does not fulfill these requirements, the user agent MUST throw
-          an InvalidAccessError.
+          an InvalidModificationError.
    */
 
   test(() => {
@@ -81,9 +81,23 @@
     const pc = new RTCPeerConnection();
     const transceiver = pc.addTransceiver('audio');
     const capabilities = RTCRtpSender.getCapabilities('video');
-    assert_throws(() => transceiver.setCodecPreferences(capabilities.codecs));
+    assert_throws('InvalidModificationError', () => transceiver.setCodecPreferences(capabilities.codecs));
 
-  }, `setCodecPreferences() on audio transceiver with codecs returned from getCapabilities('video') should throw InvalidAccessError`);
+  }, `setCodecPreferences() on audio transceiver with codecs returned from getCapabilities('video') should throw InvalidModificationError`);
+
+  test(() => {
+    const pc = new RTCPeerConnection();
+    const transceiver = pc.addTransceiver('audio');
+    const codecs = [{
+      mimeType: 'data',
+      clockRate: 2000,
+      channels: 2,
+      sdpFmtpLine: '0-15'
+    }];
+
+    assert_throws('InvalidModificationError', () => transceiver.setCodecPreferences(codecs));
+
+  }, `setCodecPreferences() with user defined codec with invalid mimeType should throw InvalidModificationError`);
 
   test(() => {
     const pc = new RTCPeerConnection();
@@ -92,12 +106,12 @@
       mimeType: 'audio/piepiper',
       clockRate: 2000,
       channels: 2,
-      sdpFmtpLine: 'a=fmtp:98 0-15'
+      sdpFmtpLine: '0-15'
     }];
 
-    assert_throws(() => transceiver.setCodecPreferences(codecs));
+    assert_throws('InvalidModificationError', () => transceiver.setCodecPreferences(codecs));
 
-  }, `setCodecPreferences() with user defined codec should throw InvalidAccessError`);
+  }, `setCodecPreferences() with user defined codec should throw InvalidModificationError`);
 
   test(() => {
     const pc = new RTCPeerConnection();
@@ -109,12 +123,45 @@
         mimeType: 'audio/piepiper',
         clockRate: 2000,
         channels: 2,
-        sdpFmtpLine: 'a=fmtp:98 0-15'
+        sdpFmtpLine: '0-15'
       }];
 
-    assert_throws(() => transceiver.setCodecPreferences(codecs));
+    assert_throws('InvalidModificationError', () => transceiver.setCodecPreferences(codecs));
+
+  }, `setCodecPreferences() with user defined codec together with codecs returned from getCapabilities() should throw InvalidModificationError`);
+
+  test(() => {
+    const pc = new RTCPeerConnection();
+    const transceiver = pc.addTransceiver('audio');
+    const capabilities = RTCRtpSender.getCapabilities('audio');
+    const codecs = [capabilities.codecs[0]];
+    codecs[0].clockRate = codecs[0].clockRate / 2;
+
+    assert_throws('InvalidModificationError', () => transceiver.setCodecPreferences(codecs));
+
+  }, `setCodecPreferences() with modified codec clock rate should throw InvalidModificationError`);
+
+  test(() => {
+    const pc = new RTCPeerConnection();
+    const transceiver = pc.addTransceiver('audio');
+    const capabilities = RTCRtpSender.getCapabilities('audio');
+    const codecs = [capabilities.codecs[0]];
+    codecs[0].channels = codecs[0].channels + 11;
+
+    assert_throws('InvalidModificationError', () => transceiver.setCodecPreferences(codecs));
+
+  }, `setCodecPreferences() with modified codec channel count should throw InvalidModificationError`);
+
+  test(() => {
+    const pc = new RTCPeerConnection();
+    const transceiver = pc.addTransceiver('audio');
+    const capabilities = RTCRtpSender.getCapabilities('audio');
+    const codecs = [capabilities.codecs[0]];
+    codecs[0].sdpFmtpLine = "modifiedparameter=1";
+
+    assert_throws('InvalidModificationError', () => transceiver.setCodecPreferences(codecs));
 
-  }, `setCodecPreferences() with user defined codec together with codecs returned from getCapabilities() should throw InvalidAccessError`);
+  }, `setCodecPreferences() with modified codec parameters should throw InvalidModificationError`);
 
   test(() => {
     const pc = new RTCPeerConnection();
@@ -129,8 +176,8 @@
     const { channels=2 } = codec;
     codec.channels = channels+1;
 
-    assert_throws(() => transceiver.setCodecPreferences(codecs));
+    assert_throws('InvalidModificationError', () => transceiver.setCodecPreferences(codecs));
 
-  }, `setCodecPreferences() with modified codecs returned from getCapabilities() should throw InvalidAccessError`);
+  }, `setCodecPreferences() with modified codecs returned from getCapabilities() should throw InvalidModificationError`);
 
  </script>