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

Unified Diff: chrome/browser/metrics/chrome_metrics_services_manager_client.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/metrics/chrome_metrics_services_manager_client.cc
diff --git a/chrome/browser/metrics/chrome_metrics_services_manager_client.cc b/chrome/browser/metrics/chrome_metrics_services_manager_client.cc
index f74bac7a29c21274d67c39b2387e60bd8190ca65..9edab5b01be340c9609cbdfa02e646f07c184a99 100644
--- a/chrome/browser/metrics/chrome_metrics_services_manager_client.cc
+++ b/chrome/browser/metrics/chrome_metrics_services_manager_client.cc
@@ -191,15 +191,15 @@ bool ChromeMetricsServicesManagerClient::GetSamplingRatePerMille(int* rate) {
std::unique_ptr<rappor::RapporService>
ChromeMetricsServicesManagerClient::CreateRapporService() {
DCHECK(thread_checker_.CalledOnValidThread());
- return base::WrapUnique(new rappor::RapporService(
- local_state_, base::Bind(&chrome::IsIncognitoSessionActive)));
+ return base::MakeUnique<rappor::RapporService>(
+ local_state_, base::Bind(&chrome::IsIncognitoSessionActive));
}
std::unique_ptr<variations::VariationsService>
ChromeMetricsServicesManagerClient::CreateVariationsService() {
DCHECK(thread_checker_.CalledOnValidThread());
return variations::VariationsService::Create(
- base::WrapUnique(new ChromeVariationsServiceClient()), local_state_,
+ base::MakeUnique<ChromeVariationsServiceClient>(), local_state_,
GetMetricsStateManager(), switches::kDisableBackgroundNetworking,
chrome_variations::CreateUIStringOverrider());
}
« no previous file with comments | « chrome/browser/media_galleries/linux/mtp_read_file_worker.cc ('k') | chrome/browser/metrics/subprocess_metrics_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698