Refactor PermissionsData to pull out the check for scripting the webstore.

extensions/common/extensions_client.h

 // 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.
 
 #ifndef EXTENSIONS_COMMON_EXTENSIONS_CLIENT_H_
 #define EXTENSIONS_COMMON_EXTENSIONS_CLIENT_H_
 
 #include <set>
 #include <string>
 #include <vector>
 
+class GURL;
+
 namespace extensions {
 
 class APIPermissionSet;
 class Extension;
 class FeatureProvider;
 class PermissionMessage;
 class PermissionMessageProvider;
 class PermissionsProvider;
 class URLPatternSet;
 
@@ skipping 34 matching lines @@
 
   // Return the whitelist of extensions that can run content scripts on
   // any origin.
   virtual const ScriptingWhitelist& GetScriptingWhitelist() const = 0;
 
   // Get the set of chrome:// hosts that |extension| can run content scripts on.
   virtual URLPatternSet GetPermittedChromeSchemeHosts(
       const Extension* extension,
       const APIPermissionSet& api_permissions) const = 0;
 
+  // Returns false if contents scripts are forbidden from running on |url|.

[Daniel Erat, 2013/11/11 23:39:27]

nit: s/contents/content/ (unless i'm mistaken about what they're called) :-)

[Yoyo Zhou, 2013/11/11 23:56:53]

Oops.

+  virtual bool IsScriptableURL(const GURL& url, std::string* error) const = 0;
+
   // Return the extensions client.
   static ExtensionsClient* Get();
 
   // Initialize the extensions system with this extensions client.
   static void Set(ExtensionsClient* client);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_COMMON_EXTENSIONS_CLIENT_H_