Index: chromecast/browser/metrics/cast_metrics_service_client.cc |
diff --git a/chromecast/browser/metrics/cast_metrics_service_client.cc b/chromecast/browser/metrics/cast_metrics_service_client.cc |
index 0d6aed76cbbd1e274099ec5bb8f17a59fb16989c..a6041cb60dcfa336a2651845a596cea994c165b7 100644 |
--- a/chromecast/browser/metrics/cast_metrics_service_client.cc |
+++ b/chromecast/browser/metrics/cast_metrics_service_client.cc |
@@ -8,6 +8,7 @@ |
#include "base/guid.h" |
#include "base/i18n/rtl.h" |
#include "base/logging.h" |
+#include "base/memory/ptr_util.h" |
#include "base/strings/string_number_conversions.h" |
#include "base/thread_task_runner_handle.h" |
#include "build/build_config.h" |
@@ -87,11 +88,11 @@ GetReleaseChannelFromUpdateChannelName(const std::string& channel_name) { |
} // namespace |
// static |
-scoped_ptr<CastMetricsServiceClient> CastMetricsServiceClient::Create( |
+std::unique_ptr<CastMetricsServiceClient> CastMetricsServiceClient::Create( |
base::TaskRunner* io_task_runner, |
PrefService* pref_service, |
net::URLRequestContextGetter* request_context) { |
- return make_scoped_ptr(new CastMetricsServiceClient( |
+ return base::WrapUnique(new CastMetricsServiceClient( |
io_task_runner, pref_service, request_context)); |
} |
@@ -125,8 +126,9 @@ void CastMetricsServiceClient::StoreClientInfo( |
SetMetricsClientId(client_id); |
} |
-scoped_ptr< ::metrics::ClientInfo> CastMetricsServiceClient::LoadClientInfo() { |
- scoped_ptr< ::metrics::ClientInfo> client_info(new ::metrics::ClientInfo); |
+std::unique_ptr<::metrics::ClientInfo> |
+CastMetricsServiceClient::LoadClientInfo() { |
+ std::unique_ptr<::metrics::ClientInfo> client_info(new ::metrics::ClientInfo); |
client_info_loaded_ = true; |
// kMetricsIsNewClientID would be missing if either the device was just |
@@ -156,7 +158,7 @@ scoped_ptr< ::metrics::ClientInfo> CastMetricsServiceClient::LoadClientInfo() { |
LOG(ERROR) << "Invalid client id from platform: " << force_client_id_ |
<< " from platform."; |
} |
- return scoped_ptr< ::metrics::ClientInfo>(); |
+ return std::unique_ptr<::metrics::ClientInfo>(); |
} |
bool CastMetricsServiceClient::IsOffTheRecordSessionActive() { |
@@ -179,7 +181,7 @@ bool CastMetricsServiceClient::GetBrand(std::string* brand_code) { |
} |
::metrics::SystemProfileProto::Channel CastMetricsServiceClient::GetChannel() { |
- scoped_ptr<CastSysInfo> sys_info = CreateSysInfo(); |
+ std::unique_ptr<CastSysInfo> sys_info = CreateSysInfo(); |
#if defined(OS_ANDROID) |
switch (sys_info->GetBuildType()) { |
@@ -238,7 +240,7 @@ void CastMetricsServiceClient::CollectFinalMetricsForLog( |
done_callback.Run(); |
} |
-scoped_ptr< ::metrics::MetricsLogUploader> |
+std::unique_ptr<::metrics::MetricsLogUploader> |
CastMetricsServiceClient::CreateUploader( |
const base::Callback<void(int)>& on_upload_complete) { |
std::string uma_server_url(::metrics::kDefaultMetricsServerUrl); |
@@ -248,9 +250,8 @@ CastMetricsServiceClient::CreateUploader( |
command_line->GetSwitchValueASCII(switches::kOverrideMetricsUploadUrl)); |
} |
DCHECK(!uma_server_url.empty()); |
- return scoped_ptr< ::metrics::MetricsLogUploader>( |
- new ::metrics::NetMetricsLogUploader(request_context_, |
- uma_server_url, |
+ return std::unique_ptr<::metrics::MetricsLogUploader>( |
+ new ::metrics::NetMetricsLogUploader(request_context_, uma_server_url, |
::metrics::kDefaultMetricsMimeType, |
on_upload_complete)); |
} |
@@ -347,25 +348,25 @@ void CastMetricsServiceClient::Initialize(CastService* cast_service) { |
CastStabilityMetricsProvider* stability_provider = |
new CastStabilityMetricsProvider(metrics_service_.get()); |
metrics_service_->RegisterMetricsProvider( |
- scoped_ptr< ::metrics::MetricsProvider>(stability_provider)); |
+ std::unique_ptr<::metrics::MetricsProvider>(stability_provider)); |
base::CommandLine* command_line = base::CommandLine::ForCurrentProcess(); |
if (!command_line->HasSwitch(switches::kDisableGpu)) { |
metrics_service_->RegisterMetricsProvider( |
- scoped_ptr< ::metrics::MetricsProvider>( |
+ std::unique_ptr<::metrics::MetricsProvider>( |
new ::metrics::GPUMetricsProvider)); |
// TODO(gfhuang): Does ChromeCast actually need metrics about screen info? |
// crbug.com/541577 |
metrics_service_->RegisterMetricsProvider( |
- scoped_ptr< ::metrics::MetricsProvider>( |
+ std::unique_ptr<::metrics::MetricsProvider>( |
new ::metrics::ScreenInfoMetricsProvider)); |
} |
metrics_service_->RegisterMetricsProvider( |
- scoped_ptr< ::metrics::MetricsProvider>( |
+ std::unique_ptr<::metrics::MetricsProvider>( |
new ::metrics::NetworkMetricsProvider(io_task_runner_))); |
metrics_service_->RegisterMetricsProvider( |
- scoped_ptr< ::metrics::MetricsProvider>( |
+ std::unique_ptr<::metrics::MetricsProvider>( |
new ::metrics::ProfilerMetricsProvider)); |
shell::CastBrowserProcess::GetInstance()->browser_client()-> |
RegisterMetricsProviders(metrics_service_.get()); |