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

extensions/common/extensions_client.h

Index: extensions/common/extensions_client.h
diff --git a/extensions/common/extensions_client.h b/extensions/common/extensions_client.h
index 71a4dfded19587a0eda83c7d52575e091b1dc14b..928347fa4b382bc11241719a22d9fbaae6b3ce5c 100644
--- a/extensions/common/extensions_client.h
+++ b/extensions/common/extensions_client.h
@@ -9,6 +9,8 @@
 #include <string>
 #include <vector>
 
+class GURL;
+
 namespace extensions {
 
 class APIPermissionSet;
@@ -63,6 +65,9 @@ class ExtensionsClient {
       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();