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

Unified Diff: chrome/browser/extensions/api/settings_overrides/settings_overrides_api.h

Issue 2639153002: Make extensions DSE persistent in browser prefs (Reland) (Closed)
Patch Set: Rebased and updated after review, use new keyword conflicts resolution Created 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/settings_overrides/settings_overrides_api.h
diff --git a/chrome/browser/extensions/api/settings_overrides/settings_overrides_api.h b/chrome/browser/extensions/api/settings_overrides/settings_overrides_api.h
index 2781665c3a50a89eab72586b1f07d05956c3f1dc..89fd425c5ee5f8079bb1426ba8554900beb14213 100644
--- a/chrome/browser/extensions/api/settings_overrides/settings_overrides_api.h
+++ b/chrome/browser/extensions/api/settings_overrides/settings_overrides_api.h
@@ -5,6 +5,7 @@
#ifndef CHROME_BROWSER_EXTENSIONS_API_SETTINGS_OVERRIDES_SETTINGS_OVERRIDES_API_H_
#define CHROME_BROWSER_EXTENSIONS_API_SETTINGS_OVERRIDES_SETTINGS_OVERRIDES_API_H_
+#include <memory>
#include <set>
#include <string>
@@ -32,14 +33,12 @@ class SettingsOverridesAPI : public BrowserContextKeyedAPI,
private:
friend class BrowserContextKeyedAPIFactory<SettingsOverridesAPI>;
- typedef std::set<scoped_refptr<const Extension> > PendingExtensions;
-
// Wrappers around PreferenceAPI.
void SetPref(const std::string& extension_id,
const std::string& pref_key,
- base::Value* value);
+ std::unique_ptr<base::Value> value) const;
void UnsetPref(const std::string& extension_id,
- const std::string& pref_key);
+ const std::string& pref_key) const;
// ExtensionRegistryObserver implementation.
void OnExtensionLoaded(content::BrowserContext* browser_context,
@@ -48,11 +47,6 @@ class SettingsOverridesAPI : public BrowserContextKeyedAPI,
const Extension* extension,
UnloadedExtensionInfo::Reason reason) override;
- // KeyedService implementation.
- void Shutdown() override;
-
- void OnTemplateURLsLoaded();
-
void RegisterSearchProvider(const Extension* extension) const;
// BrowserContextKeyedAPI implementation.
static const char* service_name() { return "SettingsOverridesAPI"; }
@@ -60,16 +54,10 @@ class SettingsOverridesAPI : public BrowserContextKeyedAPI,
Profile* profile_;
TemplateURLService* url_service_;
- // List of extensions waiting for the TemplateURLService to Load to
- // have search provider registered.
- PendingExtensions pending_extensions_;
-
// Listen to extension load, unloaded notifications.
ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
extension_registry_observer_;
- std::unique_ptr<TemplateURLService::Subscription> template_url_sub_;
-
DISALLOW_COPY_AND_ASSIGN(SettingsOverridesAPI);
};

Powered by Google App Engine
This is Rietveld 408576698