webrtc: Remove state change to READY in webrtc_create_data_channel()

Previously, READY state of webrtcbin was required to create the
data channel. Due to this condition, we raise the state to READY
right before creating a data channel. But this state change invokes
the negotiation needed callback unexpectedly.

In webrtcbin, the state limitation to create a data channel has been
removed.
 : https://review.tizen.org/gerrit/#/c/platform/upstream/gst-plugins-bad/+/256742/

Therefore, this state change is also no longer necessary and removed.

[Version] 0.1.148
[Issue Type] Improvement

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

diff --git a/packaging/capi-media-webrtc.spec b/packaging/capi-media-webrtc.spec
index 1daf96c39819936e8d55c2317bdf3058f8eb1877..b146aefb5d8a0eb1224b7f516857cf20225489e9 100644 (file)
--- 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.1.147
+Version:    0.1.148
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0

diff --git a/src/webrtc.c b/src/webrtc.c
index 2e2a55155069aa13674f6aecc6779d38737296a5..852fee4aab472bc2bd0952ad9f9dd15df963943a 100644 (file)
--- a/src/webrtc.c
+++ b/src/webrtc.c
@@ -895,9 +895,6 @@ int webrtc_create_data_channel(webrtc_h webrtc, const char *label, bundle *optio
 
        RET_VAL_WITH_UNLOCK_IF(_webrtc->state != WEBRTC_STATE_IDLE, WEBRTC_ERROR_INVALID_STATE, &_webrtc->mutex, "the state should be IDLE");
 
-       ret = _gst_pipeline_set_state(_webrtc, GST_STATE_READY);
-       RET_VAL_WITH_UNLOCK_IF(ret != WEBRTC_ERROR_NONE, ret, &_webrtc->mutex, "failed to change GST state to READY");
-
        ret = _create_data_channel(webrtc, label, options, (webrtc_data_channel_s **)channel);
 
        g_mutex_unlock(&_webrtc->mutex);