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

extensions/common/user_script.h

Index: extensions/common/user_script.h
diff --git a/extensions/common/user_script.h b/extensions/common/user_script.h
index 5a2731536b2237699a1c76f54aa778d2af9643e8..542ea203e1b6dcf41b58d1f7e5e098c072548bdb 100644
--- a/extensions/common/user_script.h
+++ b/extensions/common/user_script.h
@@ -10,7 +10,9 @@
 
 #include "base/basictypes.h"
 #include "base/files/file_path.h"
+#include "base/memory/linked_ptr.h"
 #include "base/strings/string_piece.h"
+#include "extensions/common/consumer.h"
 #include "extensions/common/url_pattern.h"
 #include "extensions/common/url_pattern_set.h"
 #include "url/gurl.h"
@@ -194,8 +196,10 @@ class UserScript {
   FileList& css_scripts() { return css_scripts_; }
   const FileList& css_scripts() const { return css_scripts_; }
 
-  const std::string& extension_id() const { return extension_id_; }
-  void set_extension_id(const std::string& id) { extension_id_ = id; }
+  const ConsumerID* consumer_id() const { return consumer_id_.get(); }
+  void set_consumer_id(const ConsumerID& consumer_id) {
+    consumer_id_.reset(new ConsumerID(consumer_id));
+  }
 
   int id() const { return user_script_id_; }
   void set_id(int id) { user_script_id_ = id; }
@@ -203,7 +207,9 @@ class UserScript {
   bool is_incognito_enabled() const { return incognito_enabled_; }
   void set_incognito_enabled(bool enabled) { incognito_enabled_ = enabled; }
 
-  bool is_standalone() const { return extension_id_.empty(); }
+  bool is_standalone() const { return GetExtensionID().empty(); }
+
+  const std::string& GetExtensionID() const;
 
   // Returns true if the script should be applied to the specified URL, false
   // otherwise.
@@ -222,6 +228,8 @@ class UserScript {
   // Pickle helper functions used to pickle the individual types of components.
   void PickleGlobs(::Pickle* pickle,
                    const std::vector<std::string>& globs) const;
+  void PickleConsumerID(::Pickle* pickle,
+                        linked_ptr<ConsumerID> consumer_id) const;
   void PickleURLPatternSet(::Pickle* pickle,
                            const URLPatternSet& pattern_list) const;
   void PickleScripts(::Pickle* pickle, const FileList& scripts) const;
@@ -229,6 +237,8 @@ class UserScript {
   // Unpickle helper functions used to unpickle individual types of components.
   void UnpickleGlobs(const ::Pickle& pickle, PickleIterator* iter,
                      std::vector<std::string>* globs);
+  linked_ptr<ConsumerID> UnpickleConsumerID(const ::Pickle& pickle,
+                                            PickleIterator* iter);
   void UnpickleURLPatternSet(const ::Pickle& pickle, PickleIterator* iter,
                              URLPatternSet* pattern_list);
   void UnpickleScripts(const ::Pickle& pickle, PickleIterator* iter,
@@ -267,9 +277,9 @@ class UserScript {
   // List of css scripts defined in content_scripts
   FileList css_scripts_;
 
-  // The ID of the extension this script is a part of, if any. Can be empty if
-  // the script is a "standlone" user script.
-  std::string extension_id_;
+  // The ID of the consumer this script is a part of. The host_id of the
+  // consumer_id can be empty if the script is a "standlone" user script.
+  linked_ptr<ConsumerID> consumer_id_;

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

Let's not have these be linked_ptrs.

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

Well, it has to be a pointer here since the consumer_id needs to be updated
after the construction of UserScript. Initially we want to use scoped_ptr, but
it makes impossible to store UserScript in STL container  since scoped_ptr isn't
copyable. 

[Devlin, 2015/01/16 17:05:04]

This should be fixable once you can just assign ConsumerIDs.

[Xi Han, 2015/01/19 22:40:05]

Done.

 
   // The globally-unique id associated with this user script. Defaults to
   // -1 for invalid.