From: Piotr Kosko
Date: Wed, 1 Apr 2020 11:25:38 +0000 (+0200)
Subject: [Bookmark] Added deprecation warnings
X-Git-Tag: submit/tizen/20200508.122743~4^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=285d864af35278a96a937e06a8b2499265508164;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git
[Bookmark] Added deprecation warnings
Whole module is deprecated since 6.0 - added warnings for C++ and JS
layers.
[ACR] https://code.sec.samsung.net/jira/browse/TWDAPI-257
[Verification] Code compiles without errors.
Change-Id: Iea2fb030b2876e0546273adf17b0f6e7c43ec296
---
diff --git a/src/bookmark/bookmark_api.js b/src/bookmark/bookmark_api.js
index 8e9513f1..6e38b603 100755
--- a/src/bookmark/bookmark_api.js
+++ b/src/bookmark/bookmark_api.js
@@ -17,6 +17,7 @@
var validator_ = xwalk.utils.validator;
var types_ = validator_.Types;
var native_ = new xwalk.utils.NativeManager(extension);
+var privUtils_ = xwalk.utils;
function EditManager() {
this.canEdit = false;
@@ -35,6 +36,7 @@ var _edit = new EditManager();
function BookmarkManager() {}
BookmarkManager.prototype.get = function() {
+ privUtils_.printDeprecationWarningFor('Bookmark API');
var args = validator_.validateArgs(arguments, [
{
name: 'parentFolder',
@@ -66,6 +68,7 @@ BookmarkManager.prototype.get = function() {
};
BookmarkManager.prototype.add = function() {
+ privUtils_.printDeprecationWarningFor('Bookmark API');
var args = validator_.validateArgs(arguments, [
{
name: 'bookmark',
@@ -98,6 +101,7 @@ BookmarkManager.prototype.add = function() {
};
BookmarkManager.prototype.remove = function() {
+ privUtils_.printDeprecationWarningFor('Bookmark API');
var args = validator_.validateArgs(arguments, [
{
name: 'bookmark',
@@ -136,6 +140,7 @@ BookmarkManager.prototype.remove = function() {
function BookmarkProvider() {}
BookmarkProvider.prototype.addToFolder = function() {
+ privUtils_.printDeprecationWarningFor('Bookmark API');
var args = validator_.validateArgs(arguments, [
{
name: 'bookmark',
@@ -170,6 +175,7 @@ BookmarkProvider.prototype.addToFolder = function() {
};
BookmarkProvider.prototype.getFolder = function() {
+ privUtils_.printDeprecationWarningFor('Bookmark API');
var args = validator_.validateArgs(arguments, [
{
name: 'id',
@@ -202,6 +208,7 @@ BookmarkProvider.prototype.getFolder = function() {
};
BookmarkProvider.prototype.getFolderItems = function() {
+ privUtils_.printDeprecationWarningFor('Bookmark API');
var args = validator_.validateArgs(arguments, [
{
name: 'id',
@@ -253,6 +260,7 @@ BookmarkProvider.prototype.getFolderItems = function() {
};
BookmarkProvider.prototype.getRootId = function() {
+ privUtils_.printDeprecationWarningFor('Bookmark API');
var ret = native_.callSync('BookmarkGetRootId');
if (native_.isFailure(ret)) {
throw native_.getErrorObject(ret);
@@ -264,6 +272,7 @@ BookmarkProvider.prototype.getRootId = function() {
var provider = new BookmarkProvider();
tizen.BookmarkItem = function() {
+ privUtils_.printDeprecationWarningFor('Bookmark API');
validator_.isConstructorCall(this, tizen.BookmarkItem);
var args = validator_.validateArgs(arguments, [
{
@@ -320,6 +329,7 @@ tizen.BookmarkItem = function() {
};
tizen.BookmarkFolder = function() {
+ privUtils_.printDeprecationWarningFor('Bookmark API');
validator_.isConstructorCall(this, tizen.BookmarkFolder);
var args = validator_.validateArgs(arguments, [
{
diff --git a/src/bookmark/bookmark_instance.cc b/src/bookmark/bookmark_instance.cc
index 63402943..eefb2e89 100644
--- a/src/bookmark/bookmark_instance.cc
+++ b/src/bookmark/bookmark_instance.cc
@@ -26,6 +26,7 @@
using common::ErrorCode;
using common::PlatformResult;
+using namespace common::tools;
namespace extension {
namespace bookmark {
@@ -184,6 +185,7 @@ PlatformResult BookmarkInstance::BookmarkTitleExistsInParent(const char* title,
void BookmarkInstance::BookmarkGet(const picojson::value& arg, picojson::object& o) {
ScopeLogger();
+ PrintDeprecationWarningFor("Bookmark API");
CHECK_PRIVILEGE_ACCESS(kPrivilegeBookmarkRead, &o);
Context ctx = {0};
@@ -215,6 +217,7 @@ void BookmarkInstance::BookmarkGet(const picojson::value& arg, picojson::object&
void BookmarkInstance::BookmarkAdd(const picojson::value& arg, picojson::object& o) {
ScopeLogger();
+ PrintDeprecationWarningFor("Bookmark API");
CHECK_PRIVILEGE_ACCESS(kPrivilegeBookmarkWrite, &o);
const auto& title = arg.get(kTitle).get();
@@ -284,6 +287,7 @@ void BookmarkInstance::BookmarkAdd(const picojson::value& arg, picojson::object&
void BookmarkInstance::BookmarkRemove(const picojson::value& arg, picojson::object& o) {
ScopeLogger();
+ PrintDeprecationWarningFor("Bookmark API");
CHECK_PRIVILEGE_ACCESS(kPrivilegeBookmarkWrite, &o);
int id = common::stol(common::FromJson(arg.get(), kId));
@@ -298,6 +302,7 @@ void BookmarkInstance::BookmarkRemove(const picojson::value& arg, picojson::obje
void BookmarkInstance::BookmarkRemoveAll(const picojson::value& msg, picojson::object& o) {
ScopeLogger();
+ PrintDeprecationWarningFor("Bookmark API");
CHECK_PRIVILEGE_ACCESS(kPrivilegeBookmarkWrite, &o);
int ntv_ret = bp_bookmark_adaptor_reset();
@@ -310,6 +315,7 @@ void BookmarkInstance::BookmarkRemoveAll(const picojson::value& msg, picojson::o
void BookmarkInstance::BookmarkGetRootId(const picojson::value& msg, picojson::object& o) {
ScopeLogger();
+ PrintDeprecationWarningFor("Bookmark API");
int rootId(0);
int ntv_ret = bp_bookmark_adaptor_get_root(&rootId);
if (ntv_ret < 0) {