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/20211216.092652^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a057dfb3790f76e67237032764d18077186722e2;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.3.23
[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 dc32f7a5..765344d3 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.3.22
+Version:    0.3.23
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
diff --git a/src/webrtc_ini.c b/src/webrtc_ini.c
index de89eb2d..844c5d09 100644
--- a/src/webrtc_ini.c
+++ b/src/webrtc_ini.c
@@ -88,7 +88,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 */
@@ -124,7 +124,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