Make UserScript non-copyable.

extensions/browser/declarative_user_script_master.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 EXTENSIONS_BROWSER_DECLARATIVE_USER_SCRIPT_MASTER_H_
 #define EXTENSIONS_BROWSER_DECLARATIVE_USER_SCRIPT_MASTER_H_
 
 #include <memory>
 #include <set>
 
@@ skipping 17 matching lines @@
 // operations are delegated, and provides an interface for adding, removing,
 // and clearing scripts.
 class DeclarativeUserScriptMaster {
  public:
   DeclarativeUserScriptMaster(content::BrowserContext* browser_context,
                               const HostID& host_id);
   ~DeclarativeUserScriptMaster();
 
   // Adds script to shared memory region. This may not happen right away if a
   // script load is in progress.
-  void AddScript(const UserScript& script);
+  void AddScript(std::unique_ptr<UserScript> script);
 
   // Adds a set of scripts to shared meomory region. The fetch of the content
   // of the script on WebUI requires to start URL request to the associated
   // render specified by |render_process_id, render_frame_id|.
   // This may not happen right away if a script load is in progress.
-  void AddScripts(const std::vector<UserScript>& scripts,
+  void AddScripts(std::vector<std::unique_ptr<UserScript>>& scripts,

[Devlin, 2016/08/17 16:39:31]

UserScriptList?  Also I think style disallows passing by (non-const) ref.

[lazyboy, 2016/08/17 18:55:52]

UserScriptList is in user_script.h, which we're not including here.

That's right about non-const ref, I've changed it to (unique) pointer as we need
to move elements out of this container.

                   int render_process_id,
                   int render_frame_id);
 
   // Removes script from shared memory region. This may not happen right away if
   // a script load is in progress.
-  void RemoveScript(const UserScript& script);
+  void RemoveScript(const UserScriptIDPair& script);
 
   // Removes a set of scripts from shared memory region. This may not happen
   // right away if a script load is in progress.
   void RemoveScripts(const std::set<UserScriptIDPair>& scripts);
 
   // Removes all scripts from shared memory region. This may not happen right
   // away if a script load is in progress.
   void ClearScripts();
 
   const HostID& host_id() const { return host_id_; }
 
   UserScriptLoader* loader() { return loader_.get(); }
 
  private:
   // ID of host that owns scripts that this component manages.
   HostID host_id_;
 
   // Script loader that handles loading contents of scripts into shared memory
   // and notifying renderers of script updates.
   std::unique_ptr<UserScriptLoader> loader_;
 
   DISALLOW_COPY_AND_ASSIGN(DeclarativeUserScriptMaster);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_DECLARATIVE_USER_SCRIPT_MASTER_H_