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

Side by Side Diff: chromecast/browser/metrics/cast_metrics_service_client.cc

Issue 1161133004: Delete code corresponding to old UMA prefs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Moar comments. Created 5 years, 6 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 unified diff | Download patch
« no previous file with comments | « no previous file | components/metrics/metrics_pref_names.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chromecast/browser/metrics/cast_metrics_service_client.h" 5 #include "chromecast/browser/metrics/cast_metrics_service_client.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/guid.h" 8 #include "base/guid.h"
9 #include "base/i18n/rtl.h" 9 #include "base/i18n/rtl.h"
10 #include "base/prefs/pref_service.h" 10 #include "base/prefs/pref_service.h"
11 #include "base/thread_task_runner_handle.h" 11 #include "base/thread_task_runner_handle.h"
12 #include "chromecast/base/chromecast_switches.h" 12 #include "chromecast/base/chromecast_switches.h"
13 #include "chromecast/browser/metrics/cast_stability_metrics_provider.h" 13 #include "chromecast/browser/metrics/cast_stability_metrics_provider.h"
14 #include "chromecast/browser/metrics/platform_metrics_providers.h" 14 #include "chromecast/browser/metrics/platform_metrics_providers.h"
15 #include "chromecast/common/pref_names.h" 15 #include "chromecast/common/pref_names.h"
16 #include "components/metrics/client_info.h" 16 #include "components/metrics/client_info.h"
17 #include "components/metrics/gpu/gpu_metrics_provider.h" 17 #include "components/metrics/gpu/gpu_metrics_provider.h"
18 #include "components/metrics/metrics_pref_names.h"
19 #include "components/metrics/metrics_provider.h" 18 #include "components/metrics/metrics_provider.h"
20 #include "components/metrics/metrics_service.h" 19 #include "components/metrics/metrics_service.h"
21 #include "components/metrics/metrics_state_manager.h" 20 #include "components/metrics/metrics_state_manager.h"
22 #include "components/metrics/net/net_metrics_log_uploader.h" 21 #include "components/metrics/net/net_metrics_log_uploader.h"
23 #include "components/metrics/net/network_metrics_provider.h" 22 #include "components/metrics/net/network_metrics_provider.h"
24 #include "components/metrics/profiler/profiler_metrics_provider.h" 23 #include "components/metrics/profiler/profiler_metrics_provider.h"
25 #include "components/metrics/url_constants.h" 24 #include "components/metrics/url_constants.h"
26 #include "content/public/common/content_switches.h" 25 #include "content/public/common/content_switches.h"
27 26
28 #if defined(OS_LINUX) 27 #if defined(OS_LINUX)
29 #include "chromecast/browser/metrics/external_metrics.h" 28 #include "chromecast/browser/metrics/external_metrics.h"
30 #endif // defined(OS_LINUX) 29 #endif // defined(OS_LINUX)
31 30
32 namespace chromecast { 31 namespace chromecast {
33 namespace metrics { 32 namespace metrics {
34 33
35 namespace { 34 namespace {
35
36 const int kStandardUploadIntervalMinutes = 5; 36 const int kStandardUploadIntervalMinutes = 5;
37
38 const char kMetricsOldClientID[] = "user_experience_metrics.client_id";
39
37 } // namespace 40 } // namespace
38 41
39 // static 42 // static
40 scoped_ptr<CastMetricsServiceClient> CastMetricsServiceClient::Create( 43 scoped_ptr<CastMetricsServiceClient> CastMetricsServiceClient::Create(
41 base::TaskRunner* io_task_runner, 44 base::TaskRunner* io_task_runner,
42 PrefService* pref_service, 45 PrefService* pref_service,
43 net::URLRequestContextGetter* request_context) { 46 net::URLRequestContextGetter* request_context) {
44 return make_scoped_ptr(new CastMetricsServiceClient(io_task_runner, 47 return make_scoped_ptr(new CastMetricsServiceClient(io_task_runner,
45 pref_service, 48 pref_service,
46 request_context)); 49 request_context));
(...skipping 18 matching lines...) Expand all
65 } 68 }
66 69
67 scoped_ptr< ::metrics::ClientInfo> CastMetricsServiceClient::LoadClientInfo() { 70 scoped_ptr< ::metrics::ClientInfo> CastMetricsServiceClient::LoadClientInfo() {
68 scoped_ptr< ::metrics::ClientInfo> client_info(new ::metrics::ClientInfo); 71 scoped_ptr< ::metrics::ClientInfo> client_info(new ::metrics::ClientInfo);
69 72
70 // kMetricsIsNewClientID would be missing if either the device was just 73 // kMetricsIsNewClientID would be missing if either the device was just
71 // FDR'ed, or it is on pre-v1.2 build. 74 // FDR'ed, or it is on pre-v1.2 build.
72 if (!pref_service_->GetBoolean(prefs::kMetricsIsNewClientID)) { 75 if (!pref_service_->GetBoolean(prefs::kMetricsIsNewClientID)) {
73 // If the old client id exists, the device must be on pre-v1.2 build, 76 // If the old client id exists, the device must be on pre-v1.2 build,
74 // instead of just being FDR'ed. 77 // instead of just being FDR'ed.
75 if (!pref_service_->GetString(::metrics::prefs::kMetricsOldClientID) 78 if (!pref_service_->GetString(kMetricsOldClientID).empty()) {
76 .empty()) {
77 // Force old client id to be regenerated. See b/9487011. 79 // Force old client id to be regenerated. See b/9487011.
78 client_info->client_id = base::GenerateGUID(); 80 client_info->client_id = base::GenerateGUID();
79 pref_service_->SetBoolean(prefs::kMetricsIsNewClientID, true); 81 pref_service_->SetBoolean(prefs::kMetricsIsNewClientID, true);
80 return client_info.Pass(); 82 return client_info.Pass();
81 } 83 }
82 // else the device was just FDR'ed, pass through. 84 // else the device was just FDR'ed, pass through.
83 } 85 }
84 86
85 const std::string client_id(GetPlatformClientID(cast_service_)); 87 const std::string client_id(GetPlatformClientID(cast_service_));
86 if (!client_id.empty() && base::IsValidGUID(client_id)) { 88 if (!client_id.empty() && base::IsValidGUID(client_id)) {
(...skipping 186 matching lines...) Expand 10 before | Expand all | Expand 10 after
273 #endif // defined(OS_LINUX) 275 #endif // defined(OS_LINUX)
274 metrics_service_->Stop(); 276 metrics_service_->Stop();
275 } 277 }
276 278
277 bool CastMetricsServiceClient::IsReportingEnabled() { 279 bool CastMetricsServiceClient::IsReportingEnabled() {
278 return PlatformIsReportingEnabled(cast_service_); 280 return PlatformIsReportingEnabled(cast_service_);
279 } 281 }
280 282
281 } // namespace metrics 283 } // namespace metrics
282 } // namespace chromecast 284 } // namespace chromecast
OLDNEW
« no previous file with comments | « no previous file | components/metrics/metrics_pref_names.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698