From c113c9108f09cabcded9a743e1bde62b131ad5d3 Mon Sep 17 00:00:00 2001 From: Andrzej Popowski Date: Thu, 30 Apr 2015 14:46:54 +0200 Subject: [PATCH] [sound] - Checking privileges moved to JS layer Change-Id: I34297aa480a1feb267efff17b9a9cde122c17f28 Signed-off-by: Andrzej Popowski --- src/sound/sound_api.js | 3 +++ src/sound/sound_instance.cc | 7 ------- 2 files changed, 3 insertions(+), 7 deletions(-) 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()); if (status.IsSuccess()) -- 2.34.1