From a5b3b23a7fcee681a24e974c1b1a10a7b876d9b5 Mon Sep 17 00:00:00 2001
From: Andrzej Popowski <a.popowski@samsung.com>
Date: Mon, 4 May 2015 08:45:59 +0200
Subject: [PATCH] [power] - checking privileges moved to JS layer

Change-Id: I7eae2d95ca7d218f42db5167253fa8f52056beae
Signed-off-by: Andrzej Popowski <a.popowski@samsung.com>
---
 src/power/power_api.js      |  9 +++++++++
 src/power/power_instance.cc | 11 -----------
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/src/power/power_api.js b/src/power/power_api.js
index 2316922e..ca001250 100755
--- a/src/power/power_api.js
+++ b/src/power/power_api.js
@@ -3,6 +3,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 var validator_ = xwalk.utils.validator;
+var privilege_ = xwalk.utils.privilege;
 var types_ = validator_.Types;
 var native_ = new xwalk.utils.NativeManager(extension);
 
@@ -108,6 +109,8 @@ function PowerManager() {
  *     is desired to be.
  */
 PowerManager.prototype.request = function(resource, state) {
+   xwalk.utils.checkPrivilegeAccess(privilege_.POWER);
+
    var args = validator_.validateArgs(arguments, [
         {'name' : 'resource', 'type': types_.ENUM, 'values' : ['SCREEN', 'CPU']},
         {'name' : 'state', 'type': types_.ENUM, 'values' : ['SCREEN_OFF', 'SCREEN_DIM', 'SCREEN_NORMAL', 'SCREEN_BRIGHT', 'CPU_AWAKE']}
@@ -201,6 +204,8 @@ PowerManager.prototype.getScreenBrightness = function() {
  * @param {!number} brightness The screen brightness value to set.
  */
 PowerManager.prototype.setScreenBrightness = function(brightness) {
+    xwalk.utils.checkPrivilegeAccess(privilege_.POWER);
+
     var args = validator_.validateArgs(arguments, [
         {'name' : 'brightness', 'type': types_.DOUBLE}
     ]);
@@ -257,6 +262,8 @@ PowerManager.prototype.restoreScreenBrightness = function() {
  * Turns on the screen.
  */
 PowerManager.prototype.turnScreenOn = function() {
+    xwalk.utils.checkPrivilegeAccess(privilege_.POWER);
+
     var nativeParam = {
     };
 
@@ -272,6 +279,8 @@ PowerManager.prototype.turnScreenOn = function() {
  * Turns off the screen.
  */
 PowerManager.prototype.turnScreenOff = function() {
+    xwalk.utils.checkPrivilegeAccess(privilege_.POWER);
+
     var nativeParam = {
     };
 
diff --git a/src/power/power_instance.cc b/src/power/power_instance.cc
index 5ce125da..7363c910 100755
--- a/src/power/power_instance.cc
+++ b/src/power/power_instance.cc
@@ -20,9 +20,6 @@ namespace extension {
 namespace power {
 
 namespace {
-// The privileges that required in Power API
-const std::string kPrivilegePower = "http://tizen.org/privilege/power";
-
 const std::map<std::string, PowerResource> kPowerResourceMap = {
     {"SCREEN", POWER_RESOURCE_SCREEN},
     {"CPU", POWER_RESOURCE_CPU}
@@ -140,8 +137,6 @@ static void ReplyAsync(PowerInstance* instance, PowerCallbacks cbfunc,
     }
 
 void PowerInstance::PowerManagerRequest(const picojson::value& args, picojson::object& out) {
-  CHECK_PRIVILEGE_ACCESS(kPrivilegePower, &out);
-
   const std::string& resource = args.get("resource").get<std::string>();
   const std::string& state = args.get("state").get<std::string>();
 
@@ -177,8 +172,6 @@ void PowerInstance::PowerManagerGetscreenbrightness(const picojson::value& args,
 
 void PowerInstance::PowerManagerSetscreenbrightness(const picojson::value& args,
                                                     picojson::object& out) {
-  CHECK_PRIVILEGE_ACCESS(kPrivilegePower, &out);
-
   CHECK_EXIST(args, "brightness", out)
 
   double brightness = args.get("brightness").get<double>();
@@ -204,8 +197,6 @@ void PowerInstance::PowerManagerRestorescreenbrightness(const picojson::value& a
 }
 
 void PowerInstance::PowerManagerTurnscreenon(const picojson::value& args, picojson::object& out) {
-  CHECK_PRIVILEGE_ACCESS(kPrivilegePower, &out);
-
   PlatformResult result = PowerManager::GetInstance()->SetScreenState(true);
   if (result.IsError())
     ReportError(result, &out);
@@ -214,8 +205,6 @@ void PowerInstance::PowerManagerTurnscreenon(const picojson::value& args, picojs
 }
 
 void PowerInstance::PowerManagerTurnscreenoff(const picojson::value& args, picojson::object& out) {
-  CHECK_PRIVILEGE_ACCESS(kPrivilegePower, &out);
-
   PlatformResult result = PowerManager::GetInstance()->SetScreenState(false);
   if (result.IsError())
     ReportError(result, &out);
-- 
2.34.1