Revert "Allow video codec preference when publishing with RTCRtpEncodingParameters."

This reverts commit 64125ee.

Author: jianjunz

src/sdk/base/publication.js

@@ -185,14 +185,14 @@ export class PublishOptions {
      * @member {?Array<Owt.Base.AudioEncodingParameters> | ?Array<RTCRtpEncodingParameters>} audio
      * @instance
      * @memberof Owt.Base.PublishOptions
-     * @desc Parameters for audio RtpSender. Publishing with RTCRtpEncodingParameters is an experimental feature. It is subject to change.
+     * @desc Parameters for audio RtpSender.
      */
     this.audio = audio;
     /**
      * @member {?Array<Owt.Base.VideoEncodingParameters> | ?Array<RTCRtpEncodingParameters>} video
      * @instance
      * @memberof Owt.Base.PublishOptions
-     * @desc Parameters for video RtpSender. Publishing with RTCRtpEncodingParameters is an experimental feature. It is subject to change.
+     * @desc Parameters for video RtpSender.
      */
     this.video = video;
     /**

src/sdk/conference/channel.js

@@ -35,7 +35,6 @@ export class ConferencePeerConnectionChannel extends EventDispatcher {
     super();
     this._config = config;
     this._options = null;
-    this._videoCodecs = undefined;
     this._signaling = signaling;
     this._pc = null;
     this._internalId = null; // It's publication ID or subscription ID.
@@ -79,7 +78,7 @@ export class ConferencePeerConnectionChannel extends EventDispatcher {
     }
   }
 
-  publish(stream, options, videoCodecs) {
+  publish(stream, options) {
     if (options === undefined) {
       options = {
         audio: !!stream.mediaStream.getAudioTracks().length,
@@ -230,7 +229,6 @@ export class ConferencePeerConnectionChannel extends EventDispatcher {
           };
           if (this._isRtpEncodingParameters(options.video)) {
             transceiverInit.sendEncodings = options.video;
-            this._videoCodecs = videoCodecs;
           }
           const transceiver = this._pc.addTransceiver(
               stream.mediaStream.getVideoTracks()[0], transceiverInit);
@@ -745,18 +743,6 @@ export class ConferencePeerConnectionChannel extends EventDispatcher {
   }
 
   _setRtpReceiverOptions(sdp, options) {
-    // Add legacy simulcast in SDP for safari.
-    if (this._isRtpEncodingParameters(options.video) && Utils.isSafari()) {
-      if (options.video.length > 1) {
-        sdp = SdpUtils.addLegacySimulcast(sdp, 'video', options.video.length);
-      }
-    }
-
-    // _videoCodecs is a workaround for setting video codecs. It will be moved to RTCRtpSendParameters.
-    if (this._isRtpEncodingParameters(options.video) && this._videoCodecs) {
-      sdp = SdpUtils.reorderCodecs(sdp, 'video', this._videoCodecs);
-      return sdp;
-    }
     if (this._isRtpEncodingParameters(options.audio) ||
         this._isRtpEncodingParameters(options.video)) {
       return sdp;

src/sdk/conference/client.js

@@ -408,10 +408,9 @@ export const ConferenceClient = function(config, signalingImpl) {
    * @desc Publish a LocalStream to conference server. Other participants will be able to subscribe this stream when it is successfully published.
    * @param {Owt.Base.LocalStream} stream The stream to be published.
    * @param {Owt.Base.PublishOptions} options Options for publication.
-   * @param {string[]} videoCodecs Video codec names for publishing. Valid values are 'VP8', 'VP9' and 'H264'. This parameter only valid when options.video is RTCRtpEncodingParameters. Publishing with RTCRtpEncodingParameters is an experimental feature. This parameter is subject to change.
    * @return {Promise<Publication, Error>} Returned promise will be resolved with a newly created Publication once specific stream is successfully published, or rejected with a newly created Error if stream is invalid or options cannot be satisfied. Successfully published means PeerConnection is established and server is able to process media data.
    */
-  this.publish = function(stream, options, videoCodecs) {
+  this.publish = function(stream, options) {
     if (!(stream instanceof StreamModule.LocalStream)) {
       return Promise.reject(new ConferenceError('Invalid stream.'));
     }
@@ -420,7 +419,7 @@ export const ConferenceClient = function(config, signalingImpl) {
           'Cannot publish a published stream.'));
     }
     const channel = createPeerConnectionChannel();
-    return channel.publish(stream, options, videoCodecs);
+    return channel.publish(stream, options);
   };
 
   /**