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

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

Issue 922403002: [Extensions] Adjust script injection reporting to the browser side (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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/browser/extensions/extension_util.h ('k') | chrome/browser/extensions/permissions_updater.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/extension_util.cc
diff --git a/chrome/browser/extensions/extension_util.cc b/chrome/browser/extensions/extension_util.cc
index 29479c49fd36413729793797c2cf13f73bd7eaf9..7ac37621974d87a3fe622d84ec60129c026a87df 100644
--- a/chrome/browser/extensions/extension_util.cc
+++ b/chrome/browser/extensions/extension_util.cc
@@ -252,20 +252,6 @@ bool DefaultAllowedScriptingOnAllUrls() {
return !FeatureSwitch::scripts_require_action()->IsEnabled();
}
-bool ScriptsMayRequireActionForExtension(
- const Extension* extension,
- const PermissionSet* permissions) {
- // An extension may require user action to execute scripts iff the extension
- // shows up in chrome:extensions (so the user can grant withheld permissions),
- // is not part of chrome or corporate policy, not on the scripting whitelist,
- // and requires enough permissions that we should withhold them.
- return extension->ShouldDisplayInExtensionSettings() &&
- !Manifest::IsPolicyLocation(extension->location()) &&
- !Manifest::IsComponentLocation(extension->location()) &&
- !PermissionsData::CanExecuteScriptEverywhere(extension) &&
- permissions->ShouldWarnAllHosts();
-}
-
bool IsAppLaunchable(const std::string& extension_id,
content::BrowserContext* context) {
int reason = ExtensionPrefs::Get(context)->GetDisableReasons(extension_id);
« no previous file with comments | « chrome/browser/extensions/extension_util.h ('k') | chrome/browser/extensions/permissions_updater.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698