From: Sangchul Lee <sc11.lee@samsung.com>
Date: Tue, 14 Dec 2021 09:42:22 +0000 (+0900)
Subject: webrtc_ini: Revise two default values
X-Git-Tag: submit/tizen_6.5/20211216.092707^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=58cc8e0f56b34a839f74b64223058dc3b39eb614;p=platform%2Fcore%2Fapi%2Fwebrtc.git

webrtc_ini: Revise two default values

1. DEFAULT_USE_ULPFEC_RED is changed to FALSE
 FEC is optional functionality that also affects bitrate and latency,
 so, set it FALSE as a default value.

2. DEFAULT_VPXENC_KEYFRAME_MAX_DIST is changed from 999999 to 10
 The previous value which has been brought from www.webmproject.org
 with no thought of that we are using 'wait-for-keyframe' of VP8
 depayloader. With a high value, received video stream can be shown
 to be freezed when a RTP packet gets lost. Therefore, this value is
 now changed to the reasonable value.

Note that these values affect only if there's empty value in ini file
or there's no ini file in a target.

[Version] 0.2.153
[Issue Type] Improvement

Change-Id: I19dd64b5bdb3fd21f09ab213cf2d8bae9b52a190
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
---

diff --git a/packaging/capi-media-webrtc.spec b/packaging/capi-media-webrtc.spec
index 1e4eecb9..510b04ff 100644
--- a/packaging/capi-media-webrtc.spec
+++ b/packaging/capi-media-webrtc.spec
@@ -1,6 +1,6 @@
 Name:       capi-media-webrtc
 Summary:    A WebRTC library in Tizen Native API
-Version:    0.2.152
+Version:    0.2.153
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
diff --git a/src/webrtc_ini.c b/src/webrtc_ini.c
index efc778f1..52a656de 100644
--- a/src/webrtc_ini.c
+++ b/src/webrtc_ini.c
@@ -85,7 +85,7 @@ bool g_verbose = false;
 #define DEFAULT_VIDEO_CODEC               "vp8"
 #define DEFAULT_AUDIO_CODEC               "opus"
 
-#define DEFAULT_USE_ULPFEC_RED            true
+#define DEFAULT_USE_ULPFEC_RED            false
 #define DEFAULT_FEC_PERCENTAGE            100  /* FEC overhead percentage for the whole stream */
 
 /* items for rendering sink */
@@ -121,7 +121,7 @@ bool g_verbose = false;
 #define DEFAULT_VPXENC_END_USAGE            1
 #define DEFAULT_VPXENC_CPU_USED             5
 #define DEFAULT_VPXENC_TARGET_BITRATE       256000
-#define DEFAULT_VPXENC_KEYFRAME_MAX_DIST    999999
+#define DEFAULT_VPXENC_KEYFRAME_MAX_DIST    10
 #define DEFAULT_VPXENC_MIN_QUANTIZER        4
 #define DEFAULT_VPXENC_MAX_QUANTIZER        56
 #define DEFAULT_VPXENC_UNDERSHOOT           95