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

Unified Diff: chromeos/login/auth/auth_status_consumer.h

Issue 2699833005: kiosk: UMA for launch error (Closed)
Patch Set: update comment Created 3 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chromeos/login/auth/auth_status_consumer.h
diff --git a/chromeos/login/auth/auth_status_consumer.h b/chromeos/login/auth/auth_status_consumer.h
index ad202a5c8a4bba857b7dc1cc2f72699c0e99ef90..ba2ae4649e7c21c9f73241aacd8a3ba041ae2f3f 100644
--- a/chromeos/login/auth/auth_status_consumer.h
+++ b/chromeos/login/auth/auth_status_consumer.h
@@ -19,24 +19,26 @@ class UserContext;
class CHROMEOS_EXPORT AuthFailure {
public:
+ // Enum used for UMA. Do NOT reorder or remove entry. Don't forget to
+ // update histograms.xml when adding new entries.
enum FailureReason {
- NONE,
- COULD_NOT_MOUNT_CRYPTOHOME,
- COULD_NOT_MOUNT_TMPFS,
- COULD_NOT_UNMOUNT_CRYPTOHOME,
- DATA_REMOVAL_FAILED, // Could not destroy your old data
- LOGIN_TIMED_OUT,
- UNLOCK_FAILED,
- NETWORK_AUTH_FAILED, // Could not authenticate against Google
- OWNER_REQUIRED, // Only the device owner can log-in.
- WHITELIST_CHECK_FAILED, // Login attempt blocked by whitelist. This
+ NONE = 0,
+ COULD_NOT_MOUNT_CRYPTOHOME = 1,
+ COULD_NOT_MOUNT_TMPFS = 2,
+ COULD_NOT_UNMOUNT_CRYPTOHOME = 3,
+ DATA_REMOVAL_FAILED = 4, // Could not destroy your old data
+ LOGIN_TIMED_OUT = 5,
+ UNLOCK_FAILED = 6,
+ NETWORK_AUTH_FAILED = 7, // Could not authenticate against Google
+ OWNER_REQUIRED = 8, // Only the device owner can log-in.
+ WHITELIST_CHECK_FAILED = 9, // Login attempt blocked by whitelist. This
// value is synthesized by the ExistingUserController and passed to the
// login_status_consumer_ in tests only. It is never generated or seen by
// any of the other authenticator classes.
- TPM_ERROR, // Critical TPM error encountered.
- USERNAME_HASH_FAILED, // Could not get username hash.
- FAILED_TO_INITIALIZE_TOKEN, // Could not get OAuth2 Token,
- NUM_FAILURE_REASONS, // This has to be the last item.
+ TPM_ERROR = 10, // Critical TPM error encountered.
+ USERNAME_HASH_FAILED = 11, // Could not get username hash.
+ FAILED_TO_INITIALIZE_TOKEN = 12, // Could not get OAuth2 Token,
+ NUM_FAILURE_REASONS, // This has to be the last item.
};
explicit AuthFailure(FailureReason reason)
« no previous file with comments | « chrome/browser/ui/webui/chromeos/login/kiosk_app_menu_handler.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698