Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9069)

Unified Diff: chrome/test/data/webrtc/munge_sdp.js

Issue 2195293002: Revert of Adds a WebRTC browser_test with opus dtx enabled. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/media/webrtc_video_quality_browsertest.cc ('k') | chrome/test/data/webrtc/peerconnection.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webrtc/munge_sdp.js
diff --git a/chrome/test/data/webrtc/munge_sdp.js b/chrome/test/data/webrtc/munge_sdp.js
index c2c7d9b6e86af648cc7a8bb16b42a4db3016e87f..9a8a37e6d78a754b5c195dc119360c131b5711e1 100644
--- a/chrome/test/data/webrtc/munge_sdp.js
+++ b/chrome/test/data/webrtc/munge_sdp.js
@@ -16,40 +16,6 @@
*/
function setSdpDefaultVideoCodec(sdp, codec) {
return setSdpDefaultCodec(sdp, 'video', codec);
-}
-
-/**
- * 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
- var defaultCodec = getSdpDefaultAudioCodec(sdp);
- if (defaultCodec !== 'opus') {
- failure('setOpusDtxEnabled',
- 'Default audio codec is not set to \'opus\'.');
- }
-
- // Find codec ID for Opus, e.g. 111 if 'a=rtpmap:111 opus/48000/2'.
- var codecId = findRtpmapId(sdpLines, 'opus');
- if (codecId === null) {
- failure('setOpusDtxEnabled', 'Unknown ID for |codec| = \'opus\'.');
- }
-
- // Find 'a=fmtp:111' line, where 111 is the codecId
- var fmtLineNo = findFmtpLine(sdpLines, codecId);
- if (fmtLineNo === null) {
- // Add the line to the SDP.
- newLine = 'a=fmtp:' + codecId + ' usedtx=1'
- rtpMapLine = findRtpmapLine(sdpLines, codecId);
- sdpLines.splice(rtpMapLine + 1, 0, newLine);
- } else {
- // Modify the line to enable Opus Dtx.
- sdpLines[fmtLineNo] += ';usedtx=1'
- }
- return mergeSdpLines(sdpLines);
}
/**
@@ -184,16 +150,6 @@
}
/**
- * 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);
-
-}
-
-/**
* Returns a modified version of |mLine| that has |codecId| first in the list of
* codec IDs. For example, setMLineDefaultCodec(
* 'm=video 9 UDP/TLS/RTP/SAVPF 100 101 107 116 117 96', 107)
« no previous file with comments | « chrome/browser/media/webrtc_video_quality_browsertest.cc ('k') | chrome/test/data/webrtc/peerconnection.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698