From 6c2f1e7349fe2fcc1ef366814400cadfe6b6f731 Mon Sep 17 00:00:00 2001 From: Randell Jesup Date: Wed, 23 Jan 2013 16:41:35 -0500 Subject: [PATCH] Bug 832683: Match SRTP policy values to enable NACK mode in webrtc r=ekr --- media/webrtc/signaling/src/mediapipeline/SrtpFlow.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/media/webrtc/signaling/src/mediapipeline/SrtpFlow.cpp b/media/webrtc/signaling/src/mediapipeline/SrtpFlow.cpp index d352c36f48c..552a039fb0c 100644 --- a/media/webrtc/signaling/src/mediapipeline/SrtpFlow.cpp +++ b/media/webrtc/signaling/src/mediapipeline/SrtpFlow.cpp @@ -75,8 +75,8 @@ RefPtr SrtpFlow::Create(int cipher_suite, policy.ssrc.type = inbound ? ssrc_any_inbound : ssrc_any_outbound; policy.ssrc.value = 0; policy.ekt = NULL; - policy.window_size = 0; // Use the default value. - policy.allow_repeat_tx = 0; // TODO(ekr@rtfm.com): revisit? + policy.window_size = 1024; // Use the Chrome value. Needs to be revisited. Default is 128 + policy.allow_repeat_tx = 1; // Use Chrome value; needed for NACK mode to work policy.next = NULL; // Now make the session