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

Unified Diff: chrome/browser/prefs/preferences_manager.h

Issue 2635093002: Switch Preferences to use Factory (Closed)
Patch Set: Missed test Created 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/prefs/preferences_manager.h
diff --git a/chrome/browser/prefs/preferences_manager.h b/chrome/browser/prefs/preferences_manager.h
index 9ffc67aa4c2e4c51a18437d5ee72430977f5ba59..8f2272a0f8f03bb48f35b1ac3450294b16446d8b 100644
--- a/chrome/browser/prefs/preferences_manager.h
+++ b/chrome/browser/prefs/preferences_manager.h
@@ -25,7 +25,8 @@ class Profile;
// the requested preferences, notifying the client of all changes.
class PreferencesManager : public prefs::mojom::PreferencesManager {
public:
- explicit PreferencesManager(Profile* profile);
+ PreferencesManager(prefs::mojom::PreferencesObserverPtr client,
+ Profile* profile);
~PreferencesManager() override;
private:
@@ -35,7 +36,6 @@ class PreferencesManager : public prefs::mojom::PreferencesManager {
void PreferenceChanged(const std::string& preference_name);
// mojom::PreferencesManager:
- void AddObserver(prefs::mojom::PreferencesObserverPtr client) override;
void SetPreferences(
std::unique_ptr<base::DictionaryValue> preferences) override;
void Subscribe(const std::vector<std::string>& preferences) override;

Powered by Google App Engine
This is Rietveld 408576698