Introduce HostID and de-couple Extensions from "script injection System" [browser side]

chrome/browser/extensions/user_script_loader.h

Index: chrome/browser/extensions/user_script_loader.h
diff --git a/chrome/browser/extensions/user_script_loader.h b/chrome/browser/extensions/user_script_loader.h
index 687bd9c3144aea81f1bc20cca7531a2fb0cdc7fb..370333aaace316afef2ea7903a2b37e595e4ac8a 100644
--- a/chrome/browser/extensions/user_script_loader.h
+++ b/chrome/browser/extensions/user_script_loader.h
@@ -10,13 +10,12 @@
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/shared_memory.h"
 #include "base/memory/weak_ptr.h"
 #include "base/scoped_observer.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
-#include "extensions/browser/extension_registry_observer.h"
-#include "extensions/common/extension.h"
-#include "extensions/common/extension_set.h"
+#include "extensions/common/consumer_id.h"
 #include "extensions/common/user_script.h"
 
 namespace base {
@@ -33,10 +32,6 @@ class Profile;
 namespace extensions {
 
 class ContentVerifier;
-class ExtensionRegistry;
-
-typedef std::map<ExtensionId, ExtensionSet::ExtensionPathAndDefaultLocale>
-    ExtensionsInfo;
 
 // Manages one "logical unit" of user scripts in shared memory by constructing a
 // new shared memory region when the set of scripts changes. Also notifies
@@ -45,22 +40,31 @@ typedef std::map<ExtensionId, ExtensionSet::ExtensionPathAndDefaultLocale>
 // of this class are embedded within classes with names ending in
 // UserScriptMaster. These "master" classes implement the strategy for which
 // scripts to load/unload on this logical unit of scripts.
-class UserScriptLoader : public content::NotificationObserver,
-                         public ExtensionRegistryObserver {
+class UserScriptLoader : public content::NotificationObserver {
  public:
+  using PathAndDefaultLocale = std::pair<base::FilePath, std::string>;

[Devlin, 2015/01/21 23:25:20]

nit: can we move these usings to the protected section?

[Xi Han, 2015/01/22 17:19:36]

I would love to move them, but they are used by some functions in the anonymous
namespace in user_script_loader.cc.

+  using ConsumersInfo = std::map<ConsumerID, PathAndDefaultLocale>;
+
+  using SubstitutionMap = std::map<std::string, std::string>;
+  using LoadUserScriptsContentFunction =
+      base::Callback<bool(const ConsumerID&,
+                          UserScript::File*,
+                          const SubstitutionMap*,
+                          const scoped_refptr<ContentVerifier>&)>;
+
   // Parses the includes out of |script| and returns them in |includes|.
   static bool ParseMetadataHeader(const base::StringPiece& script_text,
                                   UserScript* script);
 
-  // A wrapper around the method to load user scripts, which is normally run on
-  // the file thread. Exposed only for tests.
-  static void LoadScriptsForTest(UserScriptList* user_scripts);
-
   UserScriptLoader(Profile* profile,
-                   const ExtensionId& owner_extension_id,
-                   bool listen_for_extension_system_loaded);
+                   const ConsumerID& consumer_id,
+                   const scoped_refptr<ContentVerifier>& content_verifier);
   ~UserScriptLoader() override;
 
+  // A wrapper around the method to load user scripts, which is normally run on
+  // the file thread. Exposed only for tests.
+  void LoadScriptsForTest(UserScriptList* user_scripts);
+
   // Add |scripts| to the set of scripts managed by this loader.
   void AddScripts(const std::set<UserScript>& scripts);
 
@@ -73,30 +77,48 @@ class UserScriptLoader : public content::NotificationObserver,
   // Initiates procedure to start loading scripts on the file thread.
   void StartLoad();
 
-  // Return true if we have any scripts ready.
+  // Returns true if we have any scripts ready.
   bool scripts_ready() const { return shared_memory_.get() != NULL; }
 
+ protected:
+  using ConsumerIDSet = std::set<ConsumerID>;

[Devlin, 2015/01/21 23:25:20]

nit: I don't think this adds anything for readability.  Let's just use
std::set<ConsumerID>.

[Xi Han, 2015/01/22 17:19:36]

Done.

+
+  // Updates |consumers_info_| to contain info for each element of
+  // |changed_consumers_|.
+  virtual void UpdateConsumersInfo(const ConsumerIDSet& changed_consumers) = 0;
+
+  // Returns a function pointer of a static funcion to load user scripts.
+  // Base and derived classes can specify their ways to load.

[Devlin, 2015/01/21 23:25:20]

nit: The base class doesn't specify its way to load, so you can take that part
out of the comment.
nit: Please mention that this has to be safe to call multiple times.

[Xi Han, 2015/01/22 17:19:36]

Done.

+  virtual LoadUserScriptsContentFunction GetLoadUserScriptsFunction() = 0;
+
+  // Adds the |consumer_id, location| to the |consumers_info_| map.
+  // Only inserts the entry to the map when the given consumer_id doesn't
+  // exists.
+  void AddConsumerInfo(const ConsumerID& consumer_id,
+                       const PathAndDefaultLocale& location);
+
+  // Removes the entries with the given host_id from the |consumers_info_| map.
+  void RemoveConsumerInfo(const std::string& host_id);
+
+  // Sets the flag if the initial set of consumers has finished loading; if it's
+  // set to be true, calls AttempLoad() to bootstrap.
+  void SetReady(bool ready);
+
+  Profile* profile() const { return profile_; }
+  const ConsumerID& consumer_id() const { return consumer_id_; }
+
  private:
   // content::NotificationObserver implementation.
   void Observe(int type,
                const content::NotificationSource& source,
                const content::NotificationDetails& details) override;
 
-  // ExtensionRegistryObserver implementation.
-  void OnExtensionUnloaded(content::BrowserContext* browser_context,
-                           const Extension* extension,
-                           UnloadedExtensionInfo::Reason reason) override;
-
-  // Initiates script load when we have been waiting for the extension system
-  // to be ready.
-  void OnExtensionSystemReady();
-
   // Returns whether or not it is possible that calls to AddScripts(),
   // RemoveScripts(), and/or ClearScripts() have caused any real change in the
   // set of scripts to be loaded.
   bool ScriptsMayHaveChanged() const;
 
-  // Attempt to initiate a load.
+  // Attempts to initiate a load.
   void AttemptLoad();
 
   // Called once we have finished loading the scripts on the file thread.
@@ -104,19 +126,15 @@ class UserScriptLoader : public content::NotificationObserver,
                        scoped_ptr<base::SharedMemory> shared_memory);
 
   // Sends the renderer process a new set of user scripts. If
-  // |changed_extensions| is not empty, this signals that only the scripts from
-  // those extensions should be updated. Otherwise, all extensions will be
+  // |changed_consumers| is not empty, this signals that only the scripts from
+  // those consumers should be updated. Otherwise, all consumers will be
   // updated.
   void SendUpdate(content::RenderProcessHost* process,
                   base::SharedMemory* shared_memory,
-                  const std::set<ExtensionId>& changed_extensions);
-
-  // Add to |changed_extensions_| those extensions referred to by |scripts|.
-  void ExpandChangedExtensions(const std::set<UserScript>& scripts);
+                  const ConsumerIDSet& changed_consumers);
 
-  // Update |extensions_info_| to contain info for each element of
-  // |changed_extensions_|.
-  void UpdateExtensionsInfo();
+  // Adds to |changed_consumers_| those consumers referred to by |scripts|.
+  void ExpandChangedConsumers(const std::set<UserScript>& scripts);
 
   bool is_loading() const {
     // Ownership of |user_scripts_| is passed to the file thread when loading.
@@ -132,8 +150,8 @@ class UserScriptLoader : public content::NotificationObserver,
   // List of scripts from currently-installed extensions we should load.
   scoped_ptr<UserScriptList> user_scripts_;
 
-  // Maps extension info needed for localization to an extension ID.
-  ExtensionsInfo extensions_info_;
+  // Maps consumer info needed for localization to a host ID.
+  ConsumersInfo consumers_info_;
 
   // The mutually-exclusive sets of scripts that were added or removed since the
   // last script load.
@@ -146,11 +164,10 @@ class UserScriptLoader : public content::NotificationObserver,
 
   // The IDs of the extensions which changed in the last update sent to the
   // renderer.
-  ExtensionIdSet changed_extensions_;
+  ConsumerIDSet changed_consumers_;
 
-  // If the extensions service has finished loading its initial set of
-  // extensions.
-  bool extension_system_ready_;
+  // If the initial set of consumers has finished loading.
+  bool ready_;
 
   // If list of user scripts is modified while we're loading it, we note
   // that we're currently mid-load and then start over again once the load
@@ -163,12 +180,12 @@ class UserScriptLoader : public content::NotificationObserver,
   // The profile for which the scripts managed here are installed.
   Profile* profile_;
 
-  // ID of the extension that owns these scripts, if any. This is only set to a
+  // ID of the consumer that owns these scripts, if any. This is only set to a
   // non-empty value for declarative user script shared memory regions.
-  ExtensionId owner_extension_id_;
+  ConsumerID consumer_id_;
 
-  ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
-      extension_registry_observer_;
+  // Manages content verification of the loaded user scripts.
+  scoped_refptr<ContentVerifier> content_verifier_;
 
   base::WeakPtrFactory<UserScriptLoader> weak_factory_;