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

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

Issue 10750010: Add an installType property to the management API (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Forgot to remove param from LoadExtensionWithOptions (sorry) Created 8 years, 5 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
« no previous file with comments | « chrome/test/data/extensions/api_test/management/internal_extension/manifest.json ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 087e1958347caf7187f4edcedc42dc339c52b9db..21563f4acb757eaece063197baa5d8e2ef68555e 100644
--- a/chrome/test/data/extensions/api_test/management/test/basics.js
+++ b/chrome/test/data/extensions/api_test/management/test/basics.js
@@ -45,22 +45,30 @@ function checkHostPermission(item, perm) {
var tests = [
function simple() {
chrome.management.getAll(callback(function(items) {
- chrome.test.assertEq(7, items.length);
+ chrome.test.assertEq(10, items.length);
checkItemInList(items, "Extension Management API Test", true, false);
checkItemInList(items, "description", true, false,
- { "description": "a short description" });
+ { "description": "a short description" });
checkItemInList(items, "enabled_app", true, true,
- { "appLaunchUrl": "http://www.google.com/",
- "offlineEnabled": true,
- "updateUrl": "http://example.com/update.xml" });
+ { "appLaunchUrl": "http://www.google.com/",
+ "offlineEnabled": true,
+ "updateUrl": "http://example.com/update.xml" });
checkItemInList(items, "disabled_app", false, true,
- { "disabledReason": "unknown" });
+ { "disabledReason": "unknown" });
checkItemInList(items, "enabled_extension", true, false,
- { "homepageUrl": "http://example.com/" });
+ { "homepageUrl": "http://example.com/" });
checkItemInList(items, "disabled_extension", false, false,
- { "optionsUrl": "chrome-extension://<ID>/pages/options.html",
- "disabledReason": "unknown" });
+ { "optionsUrl": "chrome-extension://<ID>/pages/options.html",
+ "disabledReason": "unknown" });
+ checkItemInList(items, "description", true, false,
+ { "installType": "development" });
+ checkItemInList(items, "internal_extension", true, false,
+ { "installType": "normal" });
+ checkItemInList(items, "external_extension", true, false,
+ { "installType": "sideload" });
+ checkItemInList(items, "admin_extension", true, false,
+ { "installType": "admin" });
// Check that we got the icons correctly
var extension = getItemNamed(items, "enabled_extension");
« no previous file with comments | « chrome/test/data/extensions/api_test/management/internal_extension/manifest.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698