Reland of RTCVideoEncoder: Report H264 profile information to WebRTC

chrome/test/data/webrtc/munge_sdp.js

 /**
  * Copyright 2016 The Chromium Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 /**
  * See |setSdpDefaultCodec|.
  */
 function setSdpDefaultAudioCodec(sdp, codec) {
-  return setSdpDefaultCodec(sdp, 'audio', codec);
+  return setSdpDefaultCodec(sdp, 'audio', codec, false /* preferHwCodec */);
 }
 
 /**
  * See |setSdpDefaultCodec|.
  */
-function setSdpDefaultVideoCodec(sdp, codec) {
-  return setSdpDefaultCodec(sdp, 'video', codec);
+function setSdpDefaultVideoCodec(sdp, codec, preferHwCodec) {
+  return setSdpDefaultCodec(sdp, 'video', codec, preferHwCodec);
 }
 
 /**
  * Returns a modified version of |sdp| where the opus DTX flag has been
  * enabled.
  */
 function setOpusDtxEnabled(sdp) {
   var sdpLines = splitSdpLines(sdp);
 
   // Get default audio codec
@@ skipping 19 matching lines @@
   } else {
     // Modify the line to enable Opus Dtx.
     sdpLines[fmtLineNo] += ';usedtx=1'
   }
   return mergeSdpLines(sdpLines);
 }
 
 /**
  * Returns a modified version of |sdp| where the |codec| has been promoted to be
  * the default codec, i.e. the codec whose ID is first in the list of codecs on
- * the 'm=|type|' line, where |type| is 'audio' or 'video'.
+ * the 'm=|type|' line, where |type| is 'audio' or 'video'. If |preferHwCodec|
+ * is true, it will select the last codec with the given name, and if false, it
+ * will select the first codec with the given name.

[emircan, 2017/08/02 03:42:22]

Can you also note that HW codecs are after SW codecs in this case to make the
assumption clear?

[magjed_chromium, 2017/08/02 08:51:15]

Done.

  * @private
  */
-function setSdpDefaultCodec(sdp, type, codec) {
+function setSdpDefaultCodec(sdp, type, codec, preferHwCodec) {
   var sdpLines = splitSdpLines(sdp);
 
   // Find codec ID, e.g. 100 for 'VP8' if 'a=rtpmap:100 VP8/9000'.
-  var codecId = findRtpmapId(sdpLines, codec);
+  var codecId = findRtpmapId(sdpLines, codec, preferHwCodec);
   if (codecId === null) {
     failure('setSdpDefaultCodec',
             'Unknown ID for |codec| = \'' + codec + '\'.');
   }
 
   // Find 'm=|type|' line, e.g. 'm=video 9 UDP/TLS/RTP/SAVPF 100 101 107 116'.
   var mLineNo = findLine(sdpLines, 'm=' + type);
   if (mLineNo === null) {
     failure('setSdpDefaultCodec',
             '\'m=' + type + '\' line missing from |sdp|.');
@@ skipping 46 matching lines @@
   if (defaultCodec === null) {
     failure('getSdpDefaultCodec',
             'Unknown codec name for default codec ' + defaultCodecId + '.');
   }
   return defaultCodec;
 }
 
 /**
  * Searches through all |sdpLines| for the 'a=rtpmap:' line for the codec of
  * the specified name, returning its ID as an int if found, or null otherwise.
- * |codec| is the case-sensitive name of the codec.
+ * |codec| is the case-sensitive name of the codec. If |searchBackwards|
+ * is true, it will return the last such ID, and if false, it will return the
+ * first such ID.
  * For example, if |sdpLines| contains 'a=rtpmap:100 VP8/9000' and |codec| is
  * 'VP8', this function returns 100.
  * @private
  */
-function findRtpmapId(sdpLines, codec) {
-  var lineNo = findRtpmapLine(sdpLines, codec);
+function findRtpmapId(sdpLines, codec, searchBackwards) {
+  var lineNo = findRtpmapLine(sdpLines, codec, searchBackwards);
   if (lineNo === null)
     return null;
   // Parse <id> from 'a=rtpmap:<id> <codec>/<rate>'.
   var id = sdpLines[lineNo].substring(9, sdpLines[lineNo].indexOf(' '));
   return parseInt(id);
 }
 
 /**
  * Searches through all |sdpLines| for the 'a=rtpmap:' line for the codec of
  * the specified codec ID, returning its name if found, or null otherwise.
  * For example, if |sdpLines| contains 'a=rtpmap:100 VP8/9000' and |id| is 100,
  * this function returns 'VP8'.
  * @private
  */
 function findRtpmapCodec(sdpLines, id) {
   var lineNo = findRtpmapLine(sdpLines, id);
   if (lineNo === null)
     return null;
   // Parse <codec> from 'a=rtpmap:<id> <codec>/<rate>'.
   var from = sdpLines[lineNo].indexOf(' ');
   var to = sdpLines[lineNo].indexOf('/', from);
   if (from === null || to === null || from + 1 >= to)
     failure('findRtpmapCodec', '');
   return sdpLines[lineNo].substring(from + 1, to);
 }
 
 /**
- * Finds the first 'a=rtpmap:' line from |sdpLines| that contains |contains| and
- * returns its line index, or null if no such line was found. |contains| may be
- * the codec ID, codec name or bitrate. An 'a=rtpmap:' line looks like this:
- * 'a=rtpmap:<id> <codec>/<rate>'.
+ * Finds a 'a=rtpmap:' line from |sdpLines| that contains |contains| and returns
+ * its line index, or null if no such line was found. |contains| may be the
+ * codec ID, codec name or bitrate. If |searchBackwards| is true, it will return

[emircan, 2017/08/02 03:42:21]

Replace |searchBackwards| with |lastInstance|

[magjed_chromium, 2017/08/02 08:51:15]

Done.

+ * the last such line index, and if false, it will return the first such line
+ * index.
+ * An 'a=rtpmap:' line looks like this: 'a=rtpmap:<id> <codec>/<rate>'.
  */
-function findRtpmapLine(sdpLines, contains) {
-  for (var i = 0; i < sdpLines.length; i++) {
-    // Is 'a=rtpmap:' line containing |contains| string?
-    if (sdpLines[i].startsWith('a=rtpmap:') &&
-        sdpLines[i].indexOf(contains) != -1) {
-      // Expecting pattern 'a=rtpmap:<id> <codec>/<rate>'.
-      var pattern = new RegExp('a=rtpmap:(\\d+) \\w+\\/\\d+');
-      if (!sdpLines[i].match(pattern))
-        failure('findRtpmapLine', 'Unexpected "a=rtpmap:" pattern.');
-      // Return line index.
-      return i;
+function findRtpmapLine(sdpLines, contains, searchBackwards) {
+  if (searchBackwards === true) {
+    for (var i = sdpLines.length - 1; i >= 0 ; i--) {
+      if (isRtpmapLine(sdpLines[i], contains))
+        return i;
+    }
+  } else {
+    for (var i = 0; i < sdpLines.length; i++) {
+      if (isRtpmapLine(sdpLines[i], contains))
+        return i;
     }
   }
   return null;
 }
 
 /**
+ * Returns true if |sdpLine| contains |contains| and is of pattern
+ * 'a=rtpmap:<id> <codec>/<rate>'.
+ */
+function isRtpmapLine(sdpLine, contains) {
+  // Is 'a=rtpmap:' line containing |contains| string?
+  if (sdpLine.startsWith('a=rtpmap:') &&
+      sdpLine.indexOf(contains) != -1) {
+    // Expecting pattern 'a=rtpmap:<id> <codec>/<rate>'.
+    var pattern = new RegExp('a=rtpmap:(\\d+) \\w+\\/\\d+');
+    if (!sdpLine.match(pattern))
+      failure('isRtpmapLine', 'Unexpected "a=rtpmap:" pattern.');
+    return true;
+  }
+  return false;
+}
+
+/**
  * Finds the fmtp line in |sdpLines| for the given |codecId|, and returns its
  * line number. The line starts with 'a=fmtp:<codecId>'.
  * @private
  */
 function findFmtpLine(sdpLines, codecId) {
   return findLine(sdpLines, 'a=fmtp:' + codecId);
 
 }
 
 /**
@@ skipping 47 matching lines @@
 }
 
 /** @private */
 function findLine(lines, lineStartsWith, startingLine = 0) {
   for (var i = startingLine; i < lines.length; i++) {
     if (lines[i].startsWith(lineStartsWith))
       return i;
   }
   return null;
 }