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

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

Issue 899983002: Revert of 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: rebase Created 5 years, 10 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/extension_user_script_loader.h" 9 #include "chrome/browser/extensions/user_script_loader.h"
10 #include "extensions/browser/extension_registry_observer.h"
11 #include "extensions/common/extension.h"
10 12
11 class Profile; 13 class Profile;
12 14
13 namespace extensions { 15 namespace extensions {
14 16
17 class ExtensionRegistry;
15 class UserScript; 18 class UserScript;
16 19
17 // Manages declarative user scripts for a single extension. Owns a 20 // Manages declarative user scripts for a single extension. Owns a
18 // UserScriptLoader to which file loading and shared memory management 21 // UserScriptLoader to which file loading and shared memory management
19 // operations are delegated, and provides an interface for adding, removing, 22 // operations are delegated, and provides an interface for adding, removing,
20 // and clearing scripts. 23 // and clearing scripts.
21 class DeclarativeUserScriptMaster { 24 class DeclarativeUserScriptMaster : public ExtensionRegistryObserver {
22 public: 25 public:
23 DeclarativeUserScriptMaster(Profile* profile, const HostID& host_id); 26 DeclarativeUserScriptMaster(Profile* profile,
24 ~DeclarativeUserScriptMaster(); 27 const ExtensionId& extension_id);
28 ~DeclarativeUserScriptMaster() override;
25 29
26 // Adds script to shared memory region. This may not happen right away if a 30 // Adds script to shared memory region. This may not happen right away if a
27 // script load is in progress. 31 // script load is in progress.
28 void AddScript(const UserScript& script); 32 void AddScript(const UserScript& script);
29 33
30 // Removes script from shared memory region. This may not happen right away if 34 // Removes script from shared memory region. This may not happen right away if
31 // a script load is in progress. 35 // a script load is in progress.
32 void RemoveScript(const UserScript& script); 36 void RemoveScript(const UserScript& script);
33 37
34 // Removes all scripts from shared memory region. This may not happen right 38 // Removes all scripts from shared memory region. This may not happen right
35 // away if a script load is in progress. 39 // away if a script load is in progress.
36 void ClearScripts(); 40 void ClearScripts();
37 41
38 const HostID& host_id() const { return host_id_; } 42 const ExtensionId& extension_id() const { return extension_id_; }
39 43
40 private: 44 private:
41 // ID of host that owns scripts that this component manages. 45 // ExtensionRegistryObserver implementation.
42 HostID host_id_; 46 void OnExtensionUnloaded(content::BrowserContext* browser_context,
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_;
43 52
44 // Script loader that handles loading contents of scripts into shared memory 53 // Script loader that handles loading contents of scripts into shared memory
45 // and notifying renderers of script updates. 54 // and notifying renderers of script updates.
46 ExtensionUserScriptLoader loader_; 55 UserScriptLoader loader_;
56
57 ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
58 extension_registry_observer_;
47 59
48 DISALLOW_COPY_AND_ASSIGN(DeclarativeUserScriptMaster); 60 DISALLOW_COPY_AND_ASSIGN(DeclarativeUserScriptMaster);
49 }; 61 };
50 62
51 } // namespace extensions 63 } // namespace extensions
52 64
53 #endif // CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MASTER_H_ 65 #endif // CHROME_BROWSER_EXTENSIONS_DECLARATIVE_USER_SCRIPT_MASTER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698