Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(165)

Side by Side Diff: chrome/browser/extensions/declarative_user_script_master.h

Issue 822453002: Introduce HostID and de-couple Extensions from "script injection System" [browser side] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nits Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MASTER_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MASTER_H_
6 #define CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MASTER_H_ 6 #define CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MASTER_H_
7 7
8 #include "base/scoped_observer.h" 8 #include "base/scoped_observer.h"
9 #include "chrome/browser/extensions/user_script_loader.h" 9 #include "chrome/browser/extensions/extension_user_script_loader.h"
10 #include "extensions/browser/extension_registry_observer.h"
11 #include "extensions/common/extension.h"
12 10
13 class Profile; 11 class Profile;
14 12
15 namespace extensions { 13 namespace extensions {
16 14
17 class ExtensionRegistry;
18 class UserScript; 15 class UserScript;
19 16
20 // Manages declarative user scripts for a single extension. Owns a 17 // Manages declarative user scripts for a single extension. Owns a
21 // UserScriptLoader to which file loading and shared memory management 18 // UserScriptLoader to which file loading and shared memory management
22 // operations are delegated, and provides an interface for adding, removing, 19 // operations are delegated, and provides an interface for adding, removing,
23 // and clearing scripts. 20 // and clearing scripts.
24 class DeclarativeUserScriptMaster : public ExtensionRegistryObserver { 21 class DeclarativeUserScriptMaster {
25 public: 22 public:
26 DeclarativeUserScriptMaster(Profile* profile, 23 DeclarativeUserScriptMaster(Profile* profile, const HostID& host_id);
27 const ExtensionId& extension_id); 24 ~DeclarativeUserScriptMaster();
28 ~DeclarativeUserScriptMaster() override;
29 25
30 // Adds script to shared memory region. This may not happen right away if a 26 // Adds script to shared memory region. This may not happen right away if a
31 // script load is in progress. 27 // script load is in progress.
32 void AddScript(const UserScript& script); 28 void AddScript(const UserScript& script);
33 29
34 // Removes script from shared memory region. This may not happen right away if 30 // Removes script from shared memory region. This may not happen right away if
35 // a script load is in progress. 31 // a script load is in progress.
36 void RemoveScript(const UserScript& script); 32 void RemoveScript(const UserScript& script);
37 33
38 // Removes all scripts from shared memory region. This may not happen right 34 // Removes all scripts from shared memory region. This may not happen right
39 // away if a script load is in progress. 35 // away if a script load is in progress.
40 void ClearScripts(); 36 void ClearScripts();
41 37
42 const ExtensionId& extension_id() const { return extension_id_; } 38 const HostID& host_id() const { return host_id_; }
43 39
44 private: 40 private:
45 // ExtensionRegistryObserver implementation. 41 // ID of host that owns scripts that this component manages.
46 void OnExtensionUnloaded(content::BrowserContext* browser_context, 42 HostID host_id_;
47 const Extension* extension,
48 UnloadedExtensionInfo::Reason reason) override;
49
50 // ID of extension that owns scripts that this component manages.
51 ExtensionId extension_id_;
52 43
53 // Script loader that handles loading contents of scripts into shared memory 44 // Script loader that handles loading contents of scripts into shared memory
54 // and notifying renderers of script updates. 45 // and notifying renderers of script updates.
55 UserScriptLoader loader_; 46 ExtensionUserScriptLoader loader_;
56
57 ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
58 extension_registry_observer_;
59 47
60 DISALLOW_COPY_AND_ASSIGN(DeclarativeUserScriptMaster); 48 DISALLOW_COPY_AND_ASSIGN(DeclarativeUserScriptMaster);
61 }; 49 };
62 50
63 } // namespace extensions 51 } // namespace extensions
64 52
65 #endif // CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MASTER_H_ 53 #endif // CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MASTER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698