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

Unified Diff: chrome/browser/extensions/extension_service.h

Issue 500043003: Add PolicyProvider to ExtensionManagement (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ext-1
Patch Set: rebase, format correction Created 6 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_service.h
diff --git a/chrome/browser/extensions/extension_service.h b/chrome/browser/extensions/extension_service.h
index b31157b118041e83cd102565694ca8974f8eff89..d3670710d351cb919b2482b65222425a32e54673 100644
--- a/chrome/browser/extensions/extension_service.h
+++ b/chrome/browser/extensions/extension_service.h
@@ -15,9 +15,9 @@
#include "base/gtest_prod_util.h"
#include "base/memory/ref_counted.h"
#include "base/memory/weak_ptr.h"
-#include "base/prefs/pref_change_registrar.h"
#include "base/strings/string16.h"
#include "chrome/browser/extensions/blacklist.h"
+#include "chrome/browser/extensions/extension_management.h"
#include "chrome/browser/extensions/pending_extension_manager.h"
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
@@ -175,7 +175,8 @@ class ExtensionService
: public ExtensionServiceInterface,
public extensions::ExternalProviderInterface::VisitorInterface,
public content::NotificationObserver,
- public extensions::Blacklist::Observer {
+ public extensions::Blacklist::Observer,
+ public extensions::ExtensionManagement::Observer {
public:
// Attempts to uninstall an extension from a given ExtensionService. Returns
// true iff the target extension exists.
@@ -249,6 +250,9 @@ class ExtensionService
virtual void OnExternalProviderReady(
const extensions::ExternalProviderInterface* provider) OVERRIDE;
+ // ExtensionManagement::Observer implementation:
+ virtual void OnExtensionManagementPrefChanged() OVERRIDE;
+
// Initialize and start all installed extensions.
void Init();
@@ -497,9 +501,6 @@ class ExtensionService
// external extensions.
void OnAllExternalProvidersReady();
- // Return true if the sync type of |extension| matches |type|.
- void OnExtensionInstallPrefChanged();
-
// Adds the given extension to the list of terminated extensions if
// it is not already there and unloads it.
void TrackTerminatedExtension(const extensions::Extension* extension);
@@ -645,7 +646,6 @@ class ExtensionService
OrphanedDevTools orphaned_dev_tools_;
content::NotificationRegistrar registrar_;
- PrefChangeRegistrar pref_change_registrar_;
// Keeps track of loading and unloading component extensions.
scoped_ptr<extensions::ComponentLoader> component_loader_;

Powered by Google App Engine
This is Rietveld 408576698