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

Side by Side Diff: content/browser/background_sync/background_sync_metrics.cc

Issue 1614063002: [BackgroundSync Cleanup] Remove periodic sync code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@purge_power
Patch Set: Address comments from PS3 Created 4 years, 11 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "content/browser/background_sync/background_sync_metrics.h" 5 #include "content/browser/background_sync/background_sync_metrics.h"
6 6
7 #include "base/metrics/histogram_macros.h" 7 #include "base/metrics/histogram_macros.h"
8 #include "base/metrics/user_metrics_action.h" 8 #include "base/metrics/user_metrics_action.h"
9 9
10 namespace { 10 namespace {
(...skipping 15 matching lines...) Expand all
26 } 26 }
27 return finished_in_foreground ? RESULT_PATTERN_FAILED_FOREGROUND 27 return finished_in_foreground ? RESULT_PATTERN_FAILED_FOREGROUND
28 : RESULT_PATTERN_FAILED_BACKGROUND; 28 : RESULT_PATTERN_FAILED_BACKGROUND;
29 } 29 }
30 30
31 } // namespace 31 } // namespace
32 32
33 namespace content { 33 namespace content {
34 34
35 // static 35 // static
36 void BackgroundSyncMetrics::RecordEventStarted(SyncPeriodicity periodicity, 36 void BackgroundSyncMetrics::RecordEventStarted(bool started_in_foreground) {
37 bool started_in_foreground) { 37 UMA_HISTOGRAM_BOOLEAN("BackgroundSync.Event.OneShotStartedInForeground",
38 switch (periodicity) { 38 started_in_foreground);
39 case SYNC_ONE_SHOT:
40 UMA_HISTOGRAM_BOOLEAN("BackgroundSync.Event.OneShotStartedInForeground",
41 started_in_foreground);
42 return;
43 case SYNC_PERIODIC:
44 UMA_HISTOGRAM_BOOLEAN("BackgroundSync.Event.PeriodicStartedInForeground",
45 started_in_foreground);
46 return;
47 }
48 NOTREACHED();
49 } 39 }
50 40
51 // static 41 // static
52 void BackgroundSyncMetrics::RecordEventResult(SyncPeriodicity periodicity, 42 void BackgroundSyncMetrics::RecordEventResult(bool success,
53 bool success,
54 bool finished_in_foreground) { 43 bool finished_in_foreground) {
55 switch (periodicity) { 44 UMA_HISTOGRAM_ENUMERATION(
56 case SYNC_ONE_SHOT: 45 "BackgroundSync.Event.OneShotResultPattern",
57 UMA_HISTOGRAM_ENUMERATION( 46 EventResultToResultPattern(success, finished_in_foreground),
58 "BackgroundSync.Event.OneShotResultPattern", 47 RESULT_PATTERN_MAX + 1);
59 EventResultToResultPattern(success, finished_in_foreground),
60 RESULT_PATTERN_MAX + 1);
61 return;
62 case SYNC_PERIODIC:
63 UMA_HISTOGRAM_ENUMERATION(
64 "BackgroundSync.Event.PeriodicResultPattern",
65 EventResultToResultPattern(success, finished_in_foreground),
66 RESULT_PATTERN_MAX + 1);
67 return;
68 }
69 NOTREACHED();
70 } 48 }
71 49
72 // static 50 // static
73 void BackgroundSyncMetrics::RecordBatchSyncEventComplete( 51 void BackgroundSyncMetrics::RecordBatchSyncEventComplete(
74 const base::TimeDelta& time, 52 const base::TimeDelta& time,
75 int number_of_batched_sync_events) { 53 int number_of_batched_sync_events) {
76 // The total batch handling time should be under 5 minutes; we'll record up to 54 // The total batch handling time should be under 5 minutes; we'll record up to
77 // 6 minutes, to be safe. 55 // 6 minutes, to be safe.
78 UMA_HISTOGRAM_CUSTOM_TIMES("BackgroundSync.Event.Time", time, 56 UMA_HISTOGRAM_CUSTOM_TIMES("BackgroundSync.Event.Time", time,
79 base::TimeDelta::FromMilliseconds(10), 57 base::TimeDelta::FromMilliseconds(10),
80 base::TimeDelta::FromMinutes(6), 50); 58 base::TimeDelta::FromMinutes(6), 50);
81 UMA_HISTOGRAM_COUNTS_100("BackgroundSync.Event.BatchSize", 59 UMA_HISTOGRAM_COUNTS_100("BackgroundSync.Event.BatchSize",
82 number_of_batched_sync_events); 60 number_of_batched_sync_events);
83 } 61 }
84 62
85 // static 63 // static
86 void BackgroundSyncMetrics::CountRegisterSuccess( 64 void BackgroundSyncMetrics::CountRegisterSuccess(
87 SyncPeriodicity periodicity,
88 RegistrationCouldFire registration_could_fire, 65 RegistrationCouldFire registration_could_fire,
89 RegistrationIsDuplicate registration_is_duplicate) { 66 RegistrationIsDuplicate registration_is_duplicate) {
90 switch (periodicity) { 67 UMA_HISTOGRAM_ENUMERATION("BackgroundSync.Registration.OneShot",
91 case SYNC_ONE_SHOT: 68 BACKGROUND_SYNC_STATUS_OK,
92 UMA_HISTOGRAM_ENUMERATION("BackgroundSync.Registration.OneShot", 69 BACKGROUND_SYNC_STATUS_MAX + 1);
93 BACKGROUND_SYNC_STATUS_OK, 70 UMA_HISTOGRAM_BOOLEAN("BackgroundSync.Registration.OneShot.CouldFire",
94 BACKGROUND_SYNC_STATUS_MAX + 1); 71 registration_could_fire == REGISTRATION_COULD_FIRE);
95 UMA_HISTOGRAM_BOOLEAN("BackgroundSync.Registration.OneShot.CouldFire", 72 UMA_HISTOGRAM_BOOLEAN("BackgroundSync.Registration.OneShot.IsDuplicate",
96 registration_could_fire == REGISTRATION_COULD_FIRE); 73 registration_is_duplicate == REGISTRATION_IS_DUPLICATE);
97 UMA_HISTOGRAM_BOOLEAN( 74 return;
98 "BackgroundSync.Registration.OneShot.IsDuplicate",
99 registration_is_duplicate == REGISTRATION_IS_DUPLICATE);
100 return;
101 case SYNC_PERIODIC:
102 UMA_HISTOGRAM_ENUMERATION("BackgroundSync.Registration.Periodic",
103 BACKGROUND_SYNC_STATUS_OK,
104 BACKGROUND_SYNC_STATUS_MAX + 1);
105 UMA_HISTOGRAM_BOOLEAN(
106 "BackgroundSync.Registration.Periodic.IsDuplicate",
107 registration_is_duplicate == REGISTRATION_IS_DUPLICATE);
108 return;
109 }
110 NOTREACHED();
111 } 75 }
112 76
113 // static 77 // static
114 void BackgroundSyncMetrics::CountRegisterFailure(SyncPeriodicity periodicity, 78 void BackgroundSyncMetrics::CountRegisterFailure(BackgroundSyncStatus result) {
115 BackgroundSyncStatus result) { 79 UMA_HISTOGRAM_ENUMERATION("BackgroundSync.Registration.OneShot", result,
116 switch (periodicity) { 80 BACKGROUND_SYNC_STATUS_MAX + 1);
117 case SYNC_ONE_SHOT: 81 return;
118 UMA_HISTOGRAM_ENUMERATION("BackgroundSync.Registration.OneShot", result,
119 BACKGROUND_SYNC_STATUS_MAX + 1);
120 return;
121 case SYNC_PERIODIC:
122 UMA_HISTOGRAM_ENUMERATION("BackgroundSync.Registration.Periodic", result,
123 BACKGROUND_SYNC_STATUS_MAX + 1);
124 return;
125 }
126 NOTREACHED();
127 } 82 }
128 83
129 // static 84 // static
130 void BackgroundSyncMetrics::CountUnregister(SyncPeriodicity periodicity, 85 void BackgroundSyncMetrics::CountUnregister(BackgroundSyncStatus result) {
131 BackgroundSyncStatus result) { 86 UMA_HISTOGRAM_ENUMERATION("BackgroundSync.Unregistration.OneShot", result,
132 switch (periodicity) { 87 BACKGROUND_SYNC_STATUS_MAX + 1);
133 case SYNC_ONE_SHOT: 88 return;
134 UMA_HISTOGRAM_ENUMERATION("BackgroundSync.Unregistration.OneShot", result,
135 BACKGROUND_SYNC_STATUS_MAX + 1);
136 return;
137 case SYNC_PERIODIC:
138 UMA_HISTOGRAM_ENUMERATION("BackgroundSync.Unregistration.Periodic",
139 result, BACKGROUND_SYNC_STATUS_MAX + 1);
140 return;
141 }
142 NOTREACHED();
143 } 89 }
144 90
145 } // namespace content 91 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698