From: Hyunsoo Park <hance.park@samsung.com>
Date: Wed, 13 May 2020 09:50:18 +0000 (+0900)
Subject: Adjust sequence of setting of 'display_visible' attribute
X-Git-Tag: submit/tizen_5.5/20200514.063507^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fheads%2Faccepted%2Ftizen_5.5_unified;p=platform%2Fcore%2Fapi%2Fscreen-mirroring.git

Adjust sequence of setting of 'display_visible' attribute

As the attribute 'display_visible' determines whether a display is used or not,
it must be set at first to make commit functions related to display feature to be work properly.

Change-Id: I551e475ae8cdbcb534599a4de354e9cbd37c6b66
Signed-off-by: Hyunsoo Park <hance.park@samsung.com>
---

diff --git a/packaging/capi-media-screen-mirroring.spec b/packaging/capi-media-screen-mirroring.spec
index 227681c..531cc2c 100644
--- a/packaging/capi-media-screen-mirroring.spec
+++ b/packaging/capi-media-screen-mirroring.spec
@@ -1,6 +1,6 @@
 Name:       capi-media-screen-mirroring
 Summary:    A screen mirroring library in Tizen C API
-Version:    0.2.6
+Version:    0.2.7
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
diff --git a/src/scmirroring_primary_sink.c b/src/scmirroring_primary_sink.c
index caee1d6..98efb76 100644
--- a/src/scmirroring_primary_sink.c
+++ b/src/scmirroring_primary_sink.c
@@ -415,21 +415,21 @@ int scmirroring_primary_sink_set_display(scmirroring_primary_sink_h scmirroring_
 
 	scmirroring_debug("display type(%d)", type);
 
-	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_surface_type", type, NULL);
+	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_visible", TRUE, NULL);
 	if (ret != MM_ERROR_NONE) {
-		scmirroring_error("Fail to Set Display Type");
+		scmirroring_error("Fail to Set Display Visible as TRUE");
 		return __scmirroring_primary_sink_error_convert(__func__, ret);
 	}
 
-	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_overlay", display_surface, sizeof(void *), NULL);
+	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_surface_type", type, NULL);
 	if (ret != MM_ERROR_NONE) {
-		scmirroring_error("Fail to Set Display Overlay");
+		scmirroring_error("Fail to Set Display Type");
 		return __scmirroring_primary_sink_error_convert(__func__, ret);
 	}
 
-	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_visible", TRUE, NULL);
+	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_overlay", display_surface, sizeof(void *), NULL);
 	if (ret != MM_ERROR_NONE) {
-		scmirroring_error("Fail to Set Display Visible as TRUE");
+		scmirroring_error("Fail to Set Display Overlay");
 		return __scmirroring_primary_sink_error_convert(__func__, ret);
 	}
 
diff --git a/src/scmirroring_sink.c b/src/scmirroring_sink.c
index 74fbc25..53feba8 100644
--- a/src/scmirroring_sink.c
+++ b/src/scmirroring_sink.c
@@ -385,21 +385,21 @@ int scmirroring_sink_set_display(scmirroring_sink_h scmirroring_sink, scmirrorin
 
 	scmirroring_debug("display type(%d)", type);
 
-	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_surface_type", type, NULL);
+	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_visible", TRUE, NULL);
 	if (ret != MM_ERROR_NONE) {
-		scmirroring_error("Fail to Set Display Type");
+		scmirroring_error("Fail to Set Display Visible as TRUE");
 		return __scmirroring_sink_error_convert(__func__, ret);
 	}
 
-	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_overlay", display_surface, sizeof(void *), NULL);
+	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_surface_type", type, NULL);
 	if (ret != MM_ERROR_NONE) {
-		scmirroring_error("Fail to Set Display Overlay");
+		scmirroring_error("Fail to Set Display Type");
 		return __scmirroring_sink_error_convert(__func__, ret);
 	}
 
-	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_visible", TRUE, NULL);
+	ret = mm_wfd_sink_set_attribute(handle->mm_handle, NULL, "display_overlay", display_surface, sizeof(void *), NULL);
 	if (ret != MM_ERROR_NONE) {
-		scmirroring_error("Fail to Set Display Visible as TRUE");
+		scmirroring_error("Fail to Set Display Overlay");
 		return __scmirroring_sink_error_convert(__func__, ret);
 	}