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

Side by Side Diff: content/public/common/push_messaging_status.h

Issue 2562513002: PushMessaging.UnregistrationReason UMA should count GCM Store resets (Closed)
Patch Set: Created 4 years 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 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 #ifndef CONTENT_PUBLIC_COMMON_PUSH_MESSAGING_STATUS_H_ 5 #ifndef CONTENT_PUBLIC_COMMON_PUSH_MESSAGING_STATUS_H_
6 #define CONTENT_PUBLIC_COMMON_PUSH_MESSAGING_STATUS_H_ 6 #define CONTENT_PUBLIC_COMMON_PUSH_MESSAGING_STATUS_H_
7 7
8 namespace content { 8 namespace content {
9 9
10 // Push registration success/error codes for internal use & reporting in UMA. 10 // Push registration success/error codes for internal use & reporting in UMA.
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 83
84 // Automatic - incoming message's app id was unknown. 84 // Automatic - incoming message's app id was unknown.
85 PUSH_UNREGISTRATION_REASON_DELIVERY_UNKNOWN_APP_ID = 3, 85 PUSH_UNREGISTRATION_REASON_DELIVERY_UNKNOWN_APP_ID = 3,
86 86
87 // Automatic - incoming message's origin no longer has permission. 87 // Automatic - incoming message's origin no longer has permission.
88 PUSH_UNREGISTRATION_REASON_DELIVERY_PERMISSION_DENIED = 4, 88 PUSH_UNREGISTRATION_REASON_DELIVERY_PERMISSION_DENIED = 4,
89 89
90 // Automatic - incoming message's service worker was not found. 90 // Automatic - incoming message's service worker was not found.
91 PUSH_UNREGISTRATION_REASON_DELIVERY_NO_SERVICE_WORKER = 5, 91 PUSH_UNREGISTRATION_REASON_DELIVERY_NO_SERVICE_WORKER = 5,
92 92
93 // Automatic - GCM Store reset due to corruption.
94 PUSH_UNREGISTRATION_REASON_GCM_STORE_RESET = 6,
95
93 // NOTE: Do not renumber these as that would confuse interpretation of 96 // NOTE: Do not renumber these as that would confuse interpretation of
94 // previously logged data. When making changes, also update the enum list 97 // previously logged data. When making changes, also update the enum list
95 // in tools/metrics/histograms/histograms.xml to keep it in sync, and 98 // in tools/metrics/histograms/histograms.xml to keep it in sync, and
96 // update PUSH_UNREGISTRATION_REASON_LAST below. 99 // update PUSH_UNREGISTRATION_REASON_LAST below.
97 100
98 PUSH_UNREGISTRATION_REASON_LAST = 101 PUSH_UNREGISTRATION_REASON_LAST = PUSH_UNREGISTRATION_REASON_GCM_STORE_RESET
99 PUSH_UNREGISTRATION_REASON_DELIVERY_NO_SERVICE_WORKER
100 }; 102 };
101 103
102 // Push unregistration success/error codes for internal use & reporting in UMA. 104 // Push unregistration success/error codes for internal use & reporting in UMA.
103 // Enum values can be added, but must never be renumbered or deleted and reused. 105 // Enum values can be added, but must never be renumbered or deleted and reused.
104 enum PushUnregistrationStatus { 106 enum PushUnregistrationStatus {
105 // The unregistration was successful. 107 // The unregistration was successful.
106 PUSH_UNREGISTRATION_STATUS_SUCCESS_UNREGISTERED = 0, 108 PUSH_UNREGISTRATION_STATUS_SUCCESS_UNREGISTERED = 0,
107 109
108 // Unregistration was unnecessary, as the registration was not found. 110 // Unregistration was unnecessary, as the registration was not found.
109 PUSH_UNREGISTRATION_STATUS_SUCCESS_WAS_NOT_REGISTERED = 1, 111 PUSH_UNREGISTRATION_STATUS_SUCCESS_WAS_NOT_REGISTERED = 1,
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
226 PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_GRACE_EXCEEDED 228 PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_GRACE_EXCEEDED
227 }; 229 };
228 230
229 const char* PushRegistrationStatusToString(PushRegistrationStatus status); 231 const char* PushRegistrationStatusToString(PushRegistrationStatus status);
230 232
231 const char* PushUnregistrationStatusToString(PushUnregistrationStatus status); 233 const char* PushUnregistrationStatusToString(PushUnregistrationStatus status);
232 234
233 } // namespace content 235 } // namespace content
234 236
235 #endif // CONTENT_PUBLIC_COMMON_PUSH_MESSAGING_STATUS_H_ 237 #endif // CONTENT_PUBLIC_COMMON_PUSH_MESSAGING_STATUS_H_
OLDNEW
« no previous file with comments | « chrome/browser/push_messaging/push_messaging_service_impl.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698