From: Andrzej Popowski <a.popowski@samsung.com>
Date: Thu, 30 Apr 2015 12:46:54 +0000 (+0200)
Subject: [sound] - Checking privileges moved to JS layer
X-Git-Tag: submit/tizen_tv/20150603.064601~1^2~105
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c113c9108f09cabcded9a743e1bde62b131ad5d3;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[sound] - Checking privileges moved to JS layer

Change-Id: I34297aa480a1feb267efff17b9a9cde122c17f28
Signed-off-by: Andrzej Popowski <a.popowski@samsung.com>
---

diff --git a/src/sound/sound_api.js b/src/sound/sound_api.js
index 3e2ebd93..24606573 100644
--- a/src/sound/sound_api.js
+++ b/src/sound/sound_api.js
@@ -7,6 +7,7 @@ var utils_ = xwalk.utils;
 var type_ = utils_.type;
 var converter_ = utils_.converter;
 var validator_ = utils_.validator;
+var privilege_ = utils_.privilege;
 var types_ = validator_.Types;
 var native_ = new xwalk.utils.NativeManager(extension);
 
@@ -94,6 +95,8 @@ SoundManager.prototype.getSoundMode = function() {
 };
 
 SoundManager.prototype.setVolume = function(type, volume) {
+  utils_.checkPrivilegeAccess(privilege_.VOLUME_SET);
+
   var args = validator_.validateArgs(arguments, [
     {name: 'type', type: types_.ENUM, values: Object.keys(SoundType)},
     {name: 'volume', type: types_.DOUBLE}
diff --git a/src/sound/sound_instance.cc b/src/sound/sound_instance.cc
index c364255c..0c7c8bf3 100644
--- a/src/sound/sound_instance.cc
+++ b/src/sound/sound_instance.cc
@@ -14,12 +14,6 @@
 namespace extension {
 namespace sound {
 
-namespace {
-// The privileges that required in Sound API
-const std::string kPrivilegeSound = "http://tizen.org/privilege/volume.set";
-
-} // namespace
-
 using namespace common;
 using namespace extension::sound;
 
@@ -67,7 +61,6 @@ void SoundInstance::SoundManagerGetSoundMode(const picojson::value& args,
 
 void SoundInstance::SoundManagerSetVolume(const picojson::value& args,
                                           picojson::object& out) {
-  CHECK_PRIVILEGE_ACCESS(kPrivilegeSound, &out);
   PlatformResult status = manager_.SetVolume(args.get<picojson::object>());
 
   if (status.IsSuccess())