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

Unified Diff: chrome/browser/permissions/permission_manager.cc

Issue 2334613003: Re-write many calls to WrapUnique() with MakeUnique() (Closed)
Patch Set: Changes from review by sky Created 4 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/permissions/permission_manager.cc
diff --git a/chrome/browser/permissions/permission_manager.cc b/chrome/browser/permissions/permission_manager.cc
index fdb5d34f98358b6ee40bb893627145fad2f82c3d..fae00133077224ca301fea205e6f2d4b7abd1961 100644
--- a/chrome/browser/permissions/permission_manager.cc
+++ b/chrome/browser/permissions/permission_manager.cc
@@ -218,36 +218,36 @@ PermissionManager::PermissionManager(Profile* profile)
: profile_(profile),
weak_ptr_factory_(this) {
permission_contexts_[PermissionType::MIDI_SYSEX] =
- base::WrapUnique(new MidiPermissionContext(profile));
+ base::MakeUnique<MidiPermissionContext>(profile);
permission_contexts_[PermissionType::PUSH_MESSAGING] =
- base::WrapUnique(new NotificationPermissionContext(
- profile, PermissionType::PUSH_MESSAGING));
+ base::MakeUnique<NotificationPermissionContext>(
+ profile, PermissionType::PUSH_MESSAGING);
permission_contexts_[PermissionType::NOTIFICATIONS] =
- base::WrapUnique(new NotificationPermissionContext(
- profile, PermissionType::NOTIFICATIONS));
+ base::MakeUnique<NotificationPermissionContext>(
+ profile, PermissionType::NOTIFICATIONS);
#if !BUILDFLAG(ANDROID_JAVA_UI)
permission_contexts_[PermissionType::GEOLOCATION] =
- base::WrapUnique(new GeolocationPermissionContext(profile));
+ base::MakeUnique<GeolocationPermissionContext>(profile);
#else
permission_contexts_[PermissionType::GEOLOCATION] =
- base::WrapUnique(new GeolocationPermissionContextAndroid(profile));
+ base::MakeUnique<GeolocationPermissionContextAndroid>(profile);
#endif
#if defined(OS_CHROMEOS) || defined(OS_ANDROID)
permission_contexts_[PermissionType::PROTECTED_MEDIA_IDENTIFIER] =
- base::WrapUnique(new ProtectedMediaIdentifierPermissionContext(profile));
+ base::MakeUnique<ProtectedMediaIdentifierPermissionContext>(profile);
#endif
permission_contexts_[PermissionType::DURABLE_STORAGE] =
- base::WrapUnique(new DurableStoragePermissionContext(profile));
+ base::MakeUnique<DurableStoragePermissionContext>(profile);
permission_contexts_[PermissionType::AUDIO_CAPTURE] =
- base::WrapUnique(new MediaStreamDevicePermissionContext(
+ base::MakeUnique<MediaStreamDevicePermissionContext>(
profile, content::PermissionType::AUDIO_CAPTURE,
- CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC));
+ CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC);
permission_contexts_[PermissionType::VIDEO_CAPTURE] =
- base::WrapUnique(new MediaStreamDevicePermissionContext(
+ base::MakeUnique<MediaStreamDevicePermissionContext>(
profile, content::PermissionType::VIDEO_CAPTURE,
- CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA));
+ CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
permission_contexts_[PermissionType::BACKGROUND_SYNC] =
- base::WrapUnique(new BackgroundSyncPermissionContext(profile));
+ base::MakeUnique<BackgroundSyncPermissionContext>(profile);
}
PermissionManager::~PermissionManager() {

Powered by Google App Engine
This is Rietveld 408576698