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

Unified Diff: chrome/test/data/extensions/api_test/content_scripts/extension_iframe/iframe.js

Issue 2850793005: Remove command line/field trial support and configs for Isolate Extensions. (Closed)
Patch Set: Remove more cases of "isolate.*extension" Created 3 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/content_scripts/extension_iframe/iframe.js
diff --git a/chrome/test/data/extensions/api_test/content_scripts/extension_iframe/iframe.js b/chrome/test/data/extensions/api_test/content_scripts/extension_iframe/iframe.js
index 7e1b7a5184fef2dd3805752db179856100bd8e72..935d330502986a7987dfb6e1776bc2dbda245988 100644
--- a/chrome/test/data/extensions/api_test/content_scripts/extension_iframe/iframe.js
+++ b/chrome/test/data/extensions/api_test/content_scripts/extension_iframe/iframe.js
@@ -9,7 +9,6 @@ chrome.test.getConfig(function(config) {
'use strict';
let success = true;
- let areExtensionsIsolated = config.isolateExtensions;
// chrome.storage should exist, since the extension has the permission, and
// the storage api is allowed in content scripts.
@@ -19,17 +18,11 @@ chrome.test.getConfig(function(config) {
}
let checkPrivilegedApi = function(api, name) {
- if (api && !areExtensionsIsolated) {
- console.log("Error: " + name +
- " exists, but shouldn't without isolated extensions.");
- return false;
- }
- if (!api && areExtensionsIsolated) {
- console.log("Error: " + name +
- " doesn't exist, but should with isolated extensions.");
- return false;
- }
- return true;
+ if (api)
+ return true;
+
+ console.log('Error: ' + name + ' doesn\'t exist, but should.');
+ return false;
};
// For other permissions, they should only be available if this is a trusted

Powered by Google App Engine
This is Rietveld 408576698