Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4698)

Unified Diff: chrome/test/data/extensions/api_test/bookmark_manager/standard/test.js

Issue 308273002: Made the bookmarks extension APIs aware of managed bookmarks. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/extensions/api_test/bookmark_manager/standard/test.js
diff --git a/chrome/test/data/extensions/api_test/bookmark_manager/standard/test.js b/chrome/test/data/extensions/api_test/bookmark_manager/standard/test.js
index 7667a7b243f8d6e14bc9d3c61f939b0a690b4b95..e09929ebc35af5637cd32dd8d833e86fb9368b0a 100644
--- a/chrome/test/data/extensions/api_test/bookmark_manager/standard/test.js
+++ b/chrome/test/data/extensions/api_test/bookmark_manager/standard/test.js
@@ -9,6 +9,7 @@ const pass = chrome.test.callbackPass;
const fail = chrome.test.callbackFail;
const assertEq = chrome.test.assertEq;
const assertTrue = chrome.test.assertTrue;
+const assertFalse = chrome.test.assertFalse;
const bookmarks = chrome.bookmarks;
const bookmarkManager = chrome.bookmarkManagerPrivate;
var fooNode, fooNode2, barNode, gooNode, count, emptyFolder, emptyFolder2;
@@ -294,6 +295,26 @@ var tests = [
}));
},
+ function clipboard6() {
+ // Verify that we can't cut managed folders.
+ bookmarks.getChildren('4', pass(function(result) {
+ assertEq(2, result.length);
+ const error = "Can't modify managed bookmarks.";
+ bookmarkManager.cut([ result[0].id ], fail(error));
+
+ // Copying is fine.
+ bookmarkManager.copy([ result[0].id ], pass());
+
+ // Pasting to a managed folder is not allowed.
+ assertTrue(result[1].url === undefined);
+ bookmarkManager.canPaste(result[1].id, pass(function(result) {
+ assertFalse(result, 'Should not be able to paste to managed folders.');
+ }));
+
+ bookmarkManager.paste(result[1].id, fail(error));
+ }));
+ },
+
function canEdit() {
bookmarkManager.canEdit(pass(function(result) {
assertTrue(result, 'Should be able to edit bookmarks');
« no previous file with comments | « chrome/browser/extensions/api/bookmarks/bookmarks_api.cc ('k') | chrome/test/data/extensions/api_test/bookmarks/test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698