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

Unified Diff: chromeos/login_event_recorder.h

Issue 409113002: Refactoring: get rid of notificataions in ParallelAuthenticator (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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_event_recorder.h
diff --git a/chromeos/login_event_recorder.h b/chromeos/login_event_recorder.h
index d2adbb2d5246d7e298c93048150569d3eb40177e..2b489e2dd04aef686e8360e8fe96b99c33ca718d 100644
--- a/chromeos/login_event_recorder.h
+++ b/chromeos/login_event_recorder.h
@@ -20,6 +20,8 @@ class CHROMEOS_EXPORT LoginEventRecorder {
public:
virtual void AddLoginTimeMarker(const std::string& marker_name,
bool send_to_uma) = 0;
+ virtual void RecordAuthenticationSuccess() = 0;
+ virtual void RecordAuthenticationFailure() = 0;
};
LoginEventRecorder();
virtual ~LoginEventRecorder();
@@ -34,6 +36,9 @@ class CHROMEOS_EXPORT LoginEventRecorder {
// the identifier BootTime.|marker_name|.
void AddLoginTimeMarker(const std::string& marker_name, bool send_to_uma);
+ void RecordAuthenticationSuccess();
Nikita (slow) 2014/07/22 15:50:26 nit: Add short comment (single one for both method
Denis Kuznetsov (DE-MUC) 2014/08/20 13:50:19 Done.
+ void RecordAuthenticationFailure();
+
private:
Delegate* delegate_;

Powered by Google App Engine
This is Rietveld 408576698