From: Pawel Andruszkiewicz
Date: Tue, 21 Apr 2015 11:09:21 +0000 (+0200)
Subject: [Privilege] Fixes for TCT tests.
X-Git-Tag: submit/tizen_tv/20150603.064601~1^2~134
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3e1c1c56d21a098a2dc930b39551fc6ea0b0ef54;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git
[Privilege] Fixes for TCT tests.
Change-Id: Ia86d0a28f48ab53fd17bdf5159a32dd7dea81a96
---
diff --git a/src/contact/js/contact_manager.js b/src/contact/js/contact_manager.js
index 5cedb027..b1ba64d4 100644
--- a/src/contact/js/contact_manager.js
+++ b/src/contact/js/contact_manager.js
@@ -66,7 +66,9 @@ ContactManager.prototype.getAddressBooks = function() {
}
};
- native_.call('ContactManager_getAddressBooks', {}, callback);
+ var result = native_.call('ContactManager_getAddressBooks', {}, callback);
+
+ _checkError(result);
};
// Gets the aggregation of all address books.
@@ -209,6 +211,8 @@ ContactManager.prototype.get = function() {
// Updates a person in the address book synchronously.
ContactManager.prototype.update = function() {
+ xwalk.utils.checkPrivilegeAccess('http://tizen.org/privilege/contact.write');
+
// validation
var args = validator_.validateArgs(arguments, [{
name: 'person',
@@ -261,10 +265,12 @@ ContactManager.prototype.updateBatch = function() {
native_.callIfPossible(args.successCallback);
};
- native_.call('ContactManager_updateBatch', {
+ var result = native_.call('ContactManager_updateBatch', {
addressBook: {},
batchArgs: _toJsonObject(args.persons)
}, callback);
+
+ _checkError(result);
};
// Removes a person from the contact DB synchronously.
@@ -319,10 +325,12 @@ ContactManager.prototype.removeBatch = function() {
native_.callIfPossible(args.successCallback);
};
- native_.call('ContactManager_removeBatch', {
+ var result = native_.call('ContactManager_removeBatch', {
addressBook: {},
batchArgs: _toJsonObject(args.personIds)
}, callback);
+
+ _checkError(result);
};
// Gets an array of all Person objects from the contact DB or the ones that match the
@@ -382,7 +390,9 @@ ContactManager.prototype.find = function() {
}
};
- native_.call('ContactManager_find', data, callback);
+ var result = native_.call('ContactManager_find', data, callback);
+
+ _checkError(result);
};
// Subscribes to receive notifications about persons' changes.
@@ -419,6 +429,8 @@ ContactManager.prototype.addChangeListener = function() {
// Unsubscribes a persons' changes watch operation.
ContactManager.prototype.removeChangeListener = function() {
+ xwalk.utils.checkPrivilegeAccess('http://tizen.org/privilege/contact.read');
+
var args = validator_.validateArgs(arguments, [
{
name: 'watchId',
diff --git a/src/content/js/manager.js b/src/content/js/manager.js
index 18f9871f..7b8f3bb4 100644
--- a/src/content/js/manager.js
+++ b/src/content/js/manager.js
@@ -17,6 +17,8 @@ function ContentManager() {
}
ContentManager.prototype.update = function(content) {
+ xwalk.utils.checkPrivilegeAccess('http://tizen.org/privilege/content.write');
+
var args = validator_.validateArgs(arguments, [
{name: 'content', type: types_.PLATFORM_OBJECT, values: Content}
]);
@@ -51,7 +53,11 @@ ContentManager.prototype.updateBatch = function(contents, successCallback, error
native_.callIfPossible(args.successCallback);
};
- native_.call('ContentManager_updateBatch', data, callback);
+ var result = native_.call('ContentManager_updateBatch', data, callback);
+
+ if (native_.isFailure(result)) {
+ throw native_.getErrorObject(result);
+ }
};
ContentManager.prototype.getDirectories = function(successCallback, errorCallback) {
@@ -122,7 +128,11 @@ ContentManager.prototype.find = function(successCallback, errorCallback, directo
native_.callIfPossible(args.successCallback, out);
};
- native_.call('ContentManager_find', data, callback);
+ var result = native_.call('ContentManager_find', data, callback);
+
+ if (native_.isFailure(result)) {
+ throw native_.getErrorObject(result);
+ }
};
ContentManager.prototype.scanFile = function(contentURI, successCallback, errorCallback) {
@@ -149,7 +159,11 @@ ContentManager.prototype.scanFile = function(contentURI, successCallback, errorC
native_.callIfPossible(args.successCallback, args.contentURI);
};
- native_.call('ContentManager_scanFile', data, callback);
+ var result = native_.call('ContentManager_scanFile', data, callback);
+
+ if (native_.isFailure(result)) {
+ throw native_.getErrorObject(result);
+ }
};
ContentManager.prototype.setChangeListener = function(changeCallback) {
diff --git a/src/package/package_api.js b/src/package/package_api.js
index c5f297eb..64690961 100644
--- a/src/package/package_api.js
+++ b/src/package/package_api.js
@@ -48,7 +48,7 @@ function callNative(cmd, args) {
} else if (result['status'] == 'error') {
var err = result['error'];
if (err) {
- throw new WebAPIException(err.name, err.message);
+ throw new WebAPIException(err);
}
return false;
}
diff --git a/src/power/power_api.js b/src/power/power_api.js
index fe40aefa..6d746a9c 100755
--- a/src/power/power_api.js
+++ b/src/power/power_api.js
@@ -19,21 +19,6 @@ function nextCallbackId() {
return callbackId++;
}
-var ExceptionMap = {
- 'UnknownError' : WebAPIException.UNKNOWN_ERR ,
- 'TypeMismatchError' : WebAPIException.TYPE_MISMATCH_ERR ,
- 'InvalidValuesError' : WebAPIException.INVALID_VALUES_ERR ,
- 'IOError' : WebAPIException.IO_ERR ,
- 'ServiceNotAvailableError' : WebAPIException.SERVICE_NOT_AVAILABLE_ERR ,
- 'SecurityError' : WebAPIException.SECURITY_ERR ,
- 'NetworkError' : WebAPIException.NETWORK_ERR ,
- 'NotSupportedError' : WebAPIException.NOT_SUPPORTED_ERR ,
- 'NotFoundError' : WebAPIException.NOT_FOUND_ERR ,
- 'InvalidAccessError' : WebAPIException.INVALID_ACCESS_ERR ,
- 'AbortError' : WebAPIException.ABORT_ERR ,
- 'QuotaExceededError' : WebAPIException.QUOTA_EXCEEDED_ERR ,
-}
-
function callNative(cmd, args) {
var json = {'cmd':cmd, 'args':args};
var argjson = JSON.stringify(json);
@@ -52,11 +37,7 @@ function callNative(cmd, args) {
} else if (result['status'] == 'error') {
var err = result['error'];
if(err) {
- if(ExceptionMap[err.name]) {
- throw new WebAPIException(ExceptionMap[err.name], err.message);
- } else {
- throw new WebAPIException(WebAPIException.UNKNOWN_ERR, err.message);
- }
+ throw new WebAPIException(err);
}
return false;
}