From: Ji Yong Min <jiyong.min@samsung.com>
Date: Wed, 13 May 2015 12:40:19 +0000 (+0900)
Subject: The color space that is used by thumbnail server is changed to RGB, because gdk suppo... 
X-Git-Tag: accepted/tizen/common/20150605.124444~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6ece5e54778f9a8bea3f23f3a932a884be3c9f87;p=platform%2Fcore%2Fmultimedia%2Flibmedia-thumbnail.git

The color space that is used by thumbnail server is changed to RGB, because gdk support RGB color space.
(Fix broken thumbnail issue)

Change-Id: Id3b5dfa9df57e35a3785eb0b191b020cba995e95
Signed-off-by: jiyong.min <jiyong.min@samsung.com>
---

diff --git a/packaging/libmedia-thumbnail.spec b/packaging/libmedia-thumbnail.spec
index 285dc92..be63478 100644
--- a/packaging/libmedia-thumbnail.spec
+++ b/packaging/libmedia-thumbnail.spec
@@ -1,5 +1,5 @@
 Name:           libmedia-thumbnail
-Version:        0.1.79
+Version:        0.1.80
 Release:        0
 License:        Apache-2.0
 Summary:        Media thumbnail service Library
diff --git a/src/ipc/media-thumb-ipc.c b/src/ipc/media-thumb-ipc.c
index 6968101..e9d7c0c 100755
--- a/src/ipc/media-thumb-ipc.c
+++ b/src/ipc/media-thumb-ipc.c
@@ -543,7 +543,8 @@ _media_thumb_process(thumbMsg *req_msg, thumbMsg *res_msg, uid_t uid)
 	media_thumb_type thumb_type = req_msg->thumb_type;
 	const char *origin_path = req_msg->org_path;
 
-	media_thumb_format thumb_format = MEDIA_THUMB_BGRA;
+	// Currently, The color space that is supported by the gdk-pixbuf is only RGB.
+	media_thumb_format thumb_format = MEDIA_THUMB_RGB888;
 
 	thumb_path = res_msg->dst_path;
 	thumb_path[0] = '\0';
diff --git a/src/media-thumb-internal.c b/src/media-thumb-internal.c
index b8ddb9c..e6b3f12 100755
--- a/src/media-thumb-internal.c
+++ b/src/media-thumb-internal.c
@@ -767,7 +767,7 @@ int _media_thumb_png(const char *origin_path,
 		return err;
 	}
 
-	err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_BGRA, format);
+	err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_RGB888, format);
 	if (err < 0) {
 		thumb_err("_media_thumb_convert_format falied: %d", err);
 		SAFE_FREE(thumb_info->data);
@@ -791,7 +791,7 @@ int _media_thumb_bmp(const char *origin_path,
 		return err;
 	}
 
-	err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_BGRA, format);
+	err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_RGB888, format);
 	if (err < 0) {
 		thumb_err("_media_thumb_convert_format falied: %d", err);
 		SAFE_FREE(thumb_info->data);
@@ -815,7 +815,7 @@ int _media_thumb_wbmp(const char *origin_path,
 		return err;
 	}
 
-	err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_BGRA, format);
+	err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_RGB888, format);
 	if (err < 0) {
 		thumb_err("_media_thumb_convert_format falied: %d", err);
 		SAFE_FREE(thumb_info->data);
@@ -839,7 +839,7 @@ int _media_thumb_gif(const char *origin_path,
 		return err;
 	}
 
-	err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_BGRA, format);
+	err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_RGB888, format);
 	if (err < 0) {
 		thumb_err("_media_thumb_convert_format falied: %d", err);
 		SAFE_FREE(thumb_info->data);
@@ -908,7 +908,7 @@ int _media_thumb_jpeg(const char *origin_path,
 			return err;
 		}
 
-		err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_BGRA, format);
+		err = _media_thumb_convert_format(thumb_info, MEDIA_THUMB_RGB888, format);
 		if (err < 0) {
 			thumb_err("_media_thumb_convert_format falied: %d", err);
 			SAFE_FREE(thumb_info->data);
diff --git a/test/test-thumb.c b/test/test-thumb.c
index 2d0c191..5e37255 100755
--- a/test/test-thumb.c
+++ b/test/test-thumb.c
@@ -72,8 +72,8 @@ int main(int argc, char *argv[])
 
 		media_thumb_type thumb_type = MEDIA_THUMB_LARGE;
 		//media_thumb_type thumb_type = MEDIA_THUMB_SMALL;
-		media_thumb_format thumb_format = MEDIA_THUMB_BGRA;
-		//media_thumb_format thumb_format = MEDIA_THUMB_RGB888;
+		//media_thumb_format thumb_format = MEDIA_THUMB_BGRA;
+		media_thumb_format thumb_format = MEDIA_THUMB_RGB888;
 		int is_bgra = 1;
 		//int is_bgra = 0;