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

Unified Diff: chrome/browser/extensions/api/settings_private/settings_private_event_router.cc

Issue 2466523002: Remove some linked_ptr c/b/extension (Closed)
Patch Set: review Created 4 years, 1 month 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_private/settings_private_event_router.cc
diff --git a/chrome/browser/extensions/api/settings_private/settings_private_event_router.cc b/chrome/browser/extensions/api/settings_private/settings_private_event_router.cc
index 4b0db6660fe3717d2d4e5b54db6dfc95878010fd..f42e2ef98f92d0e5a7ff9316f84b53124b5edd27 100644
--- a/chrome/browser/extensions/api/settings_private/settings_private_event_router.cc
+++ b/chrome/browser/extensions/api/settings_private/settings_private_event_router.cc
@@ -94,15 +94,13 @@ void SettingsPrivateEventRouter::StartOrStopListeningForPrefsChanges() {
std::string pref_name = it.first;
if (prefs_util_->IsCrosSetting(pref_name)) {
#if defined(OS_CHROMEOS)
- std::unique_ptr<chromeos::CrosSettings::ObserverSubscription> observer =
- chromeos::CrosSettings::Get()->AddSettingsObserver(
+ std::unique_ptr<chromeos::CrosSettings::ObserverSubscription>
+ subscription = chromeos::CrosSettings::Get()->AddSettingsObserver(
pref_name.c_str(),
base::Bind(&SettingsPrivateEventRouter::OnPreferenceChanged,
base::Unretained(this), pref_name));
- linked_ptr<chromeos::CrosSettings::ObserverSubscription> subscription(
- observer.release());
cros_settings_subscription_map_.insert(
- make_pair(pref_name, subscription));
+ make_pair(pref_name, std::move(subscription)));
#endif
} else {
FindRegistrarForPref(it.first)

Powered by Google App Engine
This is Rietveld 408576698