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

Unified Diff: chrome/browser/extensions/permission_message_combinations_unittest.cc

Issue 2379573008: Add SetImageData api to chrome.clipboard. (Closed)
Patch Set: Fix nits. Created 3 years, 11 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/browser/extensions/permission_message_combinations_unittest.cc
diff --git a/chrome/browser/extensions/permission_message_combinations_unittest.cc b/chrome/browser/extensions/permission_message_combinations_unittest.cc
index 4619b83ed24e34b9d4dfc40599d897fffbcbbd85..0adaffca82854183cd7b98768760833a29b35eed 100644
--- a/chrome/browser/extensions/permission_message_combinations_unittest.cc
+++ b/chrome/browser/extensions/permission_message_combinations_unittest.cc
@@ -6,6 +6,7 @@
#include "base/command_line.h"
#include "base/macros.h"
+#include "base/strings/stringprintf.h"
#include "base/test/values_test_util.h"
#include "chrome/browser/extensions/test_extension_environment.h"
#include "chrome/common/extensions/permissions/chrome_permission_message_provider.h"
@@ -1162,6 +1163,30 @@ TEST_F(PermissionMessageCombinationsUnittest, PluginPermission) {
ASSERT_TRUE(CheckManifestProducesPermissions());
}
+TEST_F(PermissionMessageCombinationsUnittest, ClipboardPermissionMessages) {
+ const char kManifest[] =
+ "{"
+ " 'app': {"
+ " 'background': {"
+ " 'scripts': ['background.js']"
+ " }"
+ " },"
+ " 'permissions': [%s]"
+ "}";
+
+ CreateAndInstall(base::StringPrintf(kManifest, "'clipboardRead'"));
+ ASSERT_TRUE(CheckManifestProducesPermissions("Read data you copy and paste"));
+
+ CreateAndInstall(
+ base::StringPrintf(kManifest, "'clipboardRead', 'clipboardWrite'"));
+ ASSERT_TRUE(CheckManifestProducesPermissions(
+ "Read and modify data you copy and paste"));
+
+ CreateAndInstall(base::StringPrintf(kManifest, "'clipboardWrite'"));
+ ASSERT_TRUE(
+ CheckManifestProducesPermissions("Modify data you copy and paste"));
+}
+
// TODO(sashab): Add a test that checks that messages are generated correctly
// for withheld permissions, when an app is granted the 'all sites' permission.

Powered by Google App Engine
This is Rietveld 408576698