Update change log and spec for wrt-plugins-tizen_0.4.57

[platform/framework/web/wrt-plugins-tizen.git] / src / Content / JSContent.cpp

diff --git a/src/Content/JSContent.cpp b/src/Content/JSContent.cpp
index 2a2aee4..35cf614 100755 (executable)
--- a/src/Content/JSContent.cpp
+++ b/src/Content/JSContent.cpp
@@ -18,8 +18,7 @@
 #include <CommonsJavaScript/PrivateObject.h>
 #include <CommonsJavaScript/Converter.h>
 #include <CommonsJavaScript/JSUtils.h>
-#include <JSTizenException.h>
-#include <JSTizenExceptionFactory.h>
+#include <JSWebAPIErrorFactory.h>
 #include "JSContent.h"
 #include <Logger.h>
 
@@ -281,7 +280,7 @@ JSValueRef JSMedia::getPropertyDescription(
     {
         Converter converter(context);
         MediacontentMediaPtr media = getMediaObject(object);
-        if(!(media->getDescription().empty()))
+//        if(!(media->getDescription().empty()))
         {
             return converter.toJSValueRef(media->getDescription());
         }
@@ -450,28 +449,26 @@ bool      JSMedia::setPropertyRating(
        MediacontentMediaPtr objMedia = getMediaObject(object);
        int rating = converter.toInt(value);
 
-       if(rating < 0)
+       if(rating < 0 || rating > 10)
        {
-               rating = 0;
+            Throw(WrtDeviceApis::Commons::InvalidArgumentException);
        }
        if ( objMedia->getRating() != rating)
        {
-               objMedia->setRating(rating, true);
+            objMedia->setRating(rating, true);
        }
-               
-        return true;
+       return true;
     }
     Catch(Exception)
     {
         LoggerW("trying to set incorrect value");
-        DeviceAPI::Common::JSTizenExceptionFactory::postException(context, exception, DeviceAPI::Common::JSTizenException::TYPE_MISMATCH_ERROR);
     }
 
     return false;
 
 }
 
-bool           JSMedia::setPropertyDisplayName(
+bool JSMedia::setPropertyDisplayName(
                                        JSContextRef context,
                                        JSObjectRef object,
                                        JSStringRef propertyName,
@@ -484,23 +481,22 @@ bool              JSMedia::setPropertyDisplayName(
                MediacontentMediaPtr objMedia = getMediaObject(object);
                string displayName = converter.toString(value);
 
-               if ((objMedia->getDescription()).compare(displayName) != 0)
+               if ((objMedia->getDisplayName()).compare(displayName) != 0)
                {
                        objMedia->setDisplayName(displayName, true);
                }
-               
+
                return true;
        }
        Catch(Exception)
        {
                LoggerW("trying to set incorrect value");
-               DeviceAPI::Common::JSTizenExceptionFactory::postException(context, exception, DeviceAPI::Common::JSTizenException::TYPE_MISMATCH_ERROR);
        }
 
        return false;
 }
 
-bool           JSMedia::setPropertyDescription(
+bool JSMedia::setPropertyDescription(
                                        JSContextRef context,
                                        JSObjectRef object,
                                        JSStringRef propertyName,
@@ -523,9 +519,7 @@ bool                JSMedia::setPropertyDescription(
        Catch(Exception)
        {
                LoggerW("trying to set incorrect value");
-               DeviceAPI::Common::JSTizenExceptionFactory::postException(context, exception, DeviceAPI::Common::JSTizenException::TYPE_MISMATCH_ERROR);
        }
-
        return false;
 }