Allow whitelisted content scripts to be injected in WebViews.

chrome/renderer/extensions/renderer_permissions_policy_delegate.cc

Index: chrome/renderer/extensions/renderer_permissions_policy_delegate.cc
diff --git a/chrome/renderer/extensions/renderer_permissions_policy_delegate.cc b/chrome/renderer/extensions/renderer_permissions_policy_delegate.cc
index dabf649a49360e9b069f6e05c13927d38df8e013..8f3ec0d8b18bff3d355262d1228cd804cf8fa129 100644
--- a/chrome/renderer/extensions/renderer_permissions_policy_delegate.cc
+++ b/chrome/renderer/extensions/renderer_permissions_policy_delegate.cc
@@ -30,12 +30,8 @@ bool RendererPermissionsPolicyDelegate::CanExecuteScriptOnPage(
     int tab_id,
     int process_id,
     std::string* error) {
-  const ExtensionsClient::ScriptingWhitelist& whitelist =
-      ExtensionsClient::Get()->GetScriptingWhitelist();
-  if (std::find(whitelist.begin(), whitelist.end(), extension->id()) !=
-      whitelist.end()) {
+  if (PermissionsData::IsExtensionIdWhitelisted(extension->id()))

[dmazzoni, 2015/05/15 17:33:57]

Note: should this be PermissionsData::CanExecuteScriptEverywhere() instead? That
would also return true for component extensions, which would be a change in
behavior. Would that be more correct or not?

[not at google - send to devlin, 2015/05/15 17:38:11]

I think basically everything should use CanExecuteScriptEverywhere.
ExtensionsClient is just a giant global delegate interface, better to keep that
as much as possible an implementation detail of whoever needs it rather than an
interface to rely on (I would hope).

[dmazzoni, 2015/05/15 17:56:28]

Done.

     return true;
-  }
 
   if (dispatcher_->IsExtensionActive(kWebStoreAppId)) {
     if (error)