Allow whitelisted content scripts to be injected in WebViews.

chrome/renderer/extensions/renderer_permissions_policy_delegate.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/renderer/extensions/renderer_permissions_policy_delegate.h"
 
 #include "chrome/common/extensions/extension_constants.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extensions_client.h"
 #include "extensions/common/manifest_constants.h"
@@ skipping 12 matching lines @@
   PermissionsData::SetPolicyDelegate(NULL);
 }
 
 bool RendererPermissionsPolicyDelegate::CanExecuteScriptOnPage(
     const Extension* extension,
     const GURL& document_url,
     const GURL& top_document_url,
     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)
       *error = errors::kCannotScriptGallery;
     return false;
   }
 
   return true;
 }
 
 }  // namespace extensions