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

chrome/browser/extensions/declarative_user_script_manager.h

 // Copyright 2014 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 CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MANAGER_H_
 #define CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MANAGER_H_
 
 #include <map>
 
 #include "base/macros.h"
 #include "base/memory/linked_ptr.h"
+#include "extensions/common/consumer.h"
 
 class Profile;
 
 namespace extensions {
 class DeclarativeUserScriptMaster;
 
 // Manages a set of DeclarativeUserScriptMaster objects for script injections.
 class DeclarativeUserScriptManager {
  public:
   explicit DeclarativeUserScriptManager(Profile* profile);
   ~DeclarativeUserScriptManager();
 
-  // Get the user script master for declarative scripts; if one does not exist,
-  // a new object will be created.
-  DeclarativeUserScriptMaster* GetDeclarativeUserScriptMasterByID(
-      const std::string& id);
+  // Gets the user script master for declarative scripts by the given
+  // ConsumerID; if one does not exist, a new object will be created.
+  linked_ptr<DeclarativeUserScriptMaster> GetDeclarativeUserScriptMasterByID(

[Devlin, 2015/01/14 16:45:08]

Again, no linked_ptr (assume this goes for all DeclarativeUserScriptMaster
uses).

[Xi Han, 2015/01/14 23:46:02]

Updated.

+      const ConsumerID& id);
 
  private:
   using UserScriptMasterMap =
-      std::map<std::string, linked_ptr<DeclarativeUserScriptMaster>>;
+      std::map<ConsumerID, linked_ptr<DeclarativeUserScriptMaster>>;
 
-  // A map of DeclarativeUserScriptMasters for ids; each master is lazily
-  // initialized.
+  // Creates a DeclarativeUserScriptMaster object.
+  linked_ptr<DeclarativeUserScriptMaster> CreateDeclarativeUserScriptMaster(
+      const ConsumerID& consumer_id);
+
+  // A map of DeclarativeUserScriptMasters for ConsumerIDs; each master
+  // is lazily initialized.
   UserScriptMasterMap declarative_user_script_masters_;
 
   Profile* profile_;
 
   DISALLOW_COPY_AND_ASSIGN(DeclarativeUserScriptManager);
 };
 
-}  // extensions
+}  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MANAGER_H_