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

Unified Diff: chrome/test/data/extensions/api_test/management/test/basics.js

Issue 236113002: Suppress tabs permission warning if there is already a browsingHistory warning. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix tests Created 6 years, 8 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/management/test/basics.js
diff --git a/chrome/test/data/extensions/api_test/management/test/basics.js b/chrome/test/data/extensions/api_test/management/test/basics.js
index 915843b5d8ef963712317e6b03162045182e2ada..78c7a6e814d65bebb5e958340e637fae0ca5bccd 100644
--- a/chrome/test/data/extensions/api_test/management/test/basics.js
+++ b/chrome/test/data/extensions/api_test/management/test/basics.js
@@ -107,15 +107,14 @@ var tests = [
chrome.management.getPermissionWarningsByManifest(
manifest_str, callback(function(warnings) {
- chrome.test.assertEq(5, warnings.length);
+ // Warning for "tabs" is suppressed by "history" permission.
+ chrome.test.assertEq(4, warnings.length);
chrome.test.assertEq(
"Access your data on *.flickr.com and api.flickr.com", warnings[0]);
chrome.test.assertEq("Read and modify your bookmarks", warnings[1]);
chrome.test.assertEq("Detect your physical location", warnings[2]);
chrome.test.assertEq("Read and modify your browsing history",
warnings[3]);
- chrome.test.assertEq("Access your tabs and browsing activity",
- warnings[4]);
}));
chrome.management.getAll(callback(function(items) {

Powered by Google App Engine
This is Rietveld 408576698