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

Side by Side Diff: chrome/browser/metrics/metrics_service_android.cc

Issue 294043008: Introduce AndroidMetricsProvider class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 6 years, 7 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 | Annotate | Revision Log
« no previous file with comments | « chrome/browser/metrics/metrics_service.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "chrome/browser/metrics/metrics_service.h"
6
7 #include "base/metrics/histogram.h"
8 #include "base/prefs/pref_registry_simple.h"
9 #include "base/prefs/pref_service.h"
10 #include "base/prefs/scoped_user_pref_update.h"
11 #include "base/values.h"
12 #include "chrome/browser/android/activity_type_ids.h"
13 #include "chrome/browser/browser_process.h"
14 #include "chrome/common/pref_names.h"
15
16 namespace {
17
18 // Increments a particular entry in the ListValue.
19 void IncrementListValue(base::ListValue* counts, int index) {
20 int current_count = 0;
21 counts->GetInteger(index, &current_count);
22 counts->Set(index, new base::FundamentalValue(current_count + 1));
23 }
24
25 // Takes an int corresponding to a Type and returns the corresponding flag.
26 int GetActivityFlag(int type_id) {
27 ActivityTypeIds::Type type = ActivityTypeIds::GetActivityType(type_id);
28 DCHECK_LT(type, ActivityTypeIds::ACTIVITY_MAX_VALUE);
29 return (1 << type);
30 }
31
32 } // namespace
33
34 // static
35 void MetricsService::RegisterPrefsAndroid(PrefRegistrySimple* registry) {
36 registry->RegisterIntegerPref(prefs::kStabilityForegroundActivityType,
37 ActivityTypeIds::ACTIVITY_NONE);
38 registry->RegisterIntegerPref(prefs::kStabilityLaunchedActivityFlags, 0);
39 registry->RegisterListPref(prefs::kStabilityLaunchedActivityCounts);
40 registry->RegisterListPref(prefs::kStabilityCrashedActivityCounts);
41 }
42
43 void MetricsService::LogAndroidStabilityToPrefs(PrefService* pref) {
44 // Track which Activities were launched by the user.
45 // A 'launch' is defined as starting the Activity at least once during a
46 // UMA session. Multiple launches are counted only once since it is possible
47 // for users to hop between Activities (e.g. entering and leaving Settings).
48 int launched = pref->GetInteger(prefs::kStabilityLaunchedActivityFlags);
49 ListPrefUpdate update_launches(pref, prefs::kStabilityLaunchedActivityCounts);
50 base::ListValue* launch_counts = update_launches.Get();
51 for (int activity_type = ActivityTypeIds::ACTIVITY_NONE;
52 activity_type < ActivityTypeIds::ACTIVITY_MAX_VALUE;
53 ++activity_type) {
54 if (launched & GetActivityFlag(activity_type))
55 IncrementListValue(launch_counts, activity_type);
56 }
57 pref->SetInteger(prefs::kStabilityLaunchedActivityFlags, 0);
58
59 // Track any Activities that were in the foreground when Chrome died.
60 // These Activities failed to be recorded as leaving the foreground, so Chrome
61 // couldn't have ended the UMA session cleanly. Record them as crashing.
62 int foreground = pref->GetInteger(prefs::kStabilityForegroundActivityType);
63 if (foreground != ActivityTypeIds::ACTIVITY_NONE) {
64 ListPrefUpdate update_crashes(pref, prefs::kStabilityCrashedActivityCounts);
65 base::ListValue* crash_counts = update_crashes.Get();
66 IncrementListValue(crash_counts, foreground);
67 pref->SetInteger(prefs::kStabilityForegroundActivityType,
68 ActivityTypeIds::ACTIVITY_NONE);
69 }
70
71 pref->CommitPendingWrite();
72 }
73
74 void MetricsService::ConvertAndroidStabilityPrefsToHistograms(
75 PrefService* pref) {
76 ListPrefUpdate launch_updater(pref, prefs::kStabilityLaunchedActivityCounts);
77 ListPrefUpdate crash_updater(pref, prefs::kStabilityCrashedActivityCounts);
78
79 base::ListValue* launch_counts = launch_updater.Get();
80 base::ListValue* crash_counts = crash_updater.Get();
81
82 for (int activity_type = ActivityTypeIds::ACTIVITY_NONE;
83 activity_type < ActivityTypeIds::ACTIVITY_MAX_VALUE;
84 ++activity_type) {
85 int launch_count = 0;
86 int crash_count = 0;
87
88 launch_counts->GetInteger(activity_type, &launch_count);
89 crash_counts->GetInteger(activity_type, &crash_count);
90
91 for (int count = 0; count < launch_count; ++count) {
92 UMA_STABILITY_HISTOGRAM_ENUMERATION(
93 "Chrome.Android.Activity.LaunchCounts",
94 activity_type,
95 ActivityTypeIds::ACTIVITY_MAX_VALUE);
96 }
97
98 for (int count = 0; count < crash_count; ++count) {
99 UMA_STABILITY_HISTOGRAM_ENUMERATION("Chrome.Android.Activity.CrashCounts",
100 activity_type,
101 ActivityTypeIds::ACTIVITY_MAX_VALUE);
102 }
103 }
104
105 launch_counts->Clear();
106 crash_counts->Clear();
107 }
108
109 void MetricsService::OnForegroundActivityChanged(PrefService* pref,
110 ActivityTypeIds::Type type) {
111 DCHECK(type < ActivityTypeIds::ACTIVITY_MAX_VALUE);
112
113 int activity_type = pref->GetInteger(prefs::kStabilityForegroundActivityType);
114 if (activity_type == type)
115 return;
116
117 // Record that the Activity is now in the foreground.
118 pref->SetInteger(prefs::kStabilityForegroundActivityType, type);
119
120 // Record that the Activity was launched this sesaion.
121 // The pref stores a set of flags ORed together, where each set flag
122 // corresponds to a launched Activity type.
123 int launched = pref->GetInteger(prefs::kStabilityLaunchedActivityFlags);
124 if (type != ActivityTypeIds::ACTIVITY_NONE) {
125 launched |= GetActivityFlag(type);
126 pref->SetInteger(prefs::kStabilityLaunchedActivityFlags, launched);
127 }
128
129 pref->CommitPendingWrite();
130 }
OLDNEW
« no previous file with comments | « chrome/browser/metrics/metrics_service.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698