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

Unified Diff: chrome/browser/ui/webui/options/password_manager_handler.h

Issue 28713002: [Mac] Add option to reauthenticate the OS user before revealing passwords. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rerebase. Created 7 years, 2 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: chrome/browser/ui/webui/options/password_manager_handler.h
diff --git a/chrome/browser/ui/webui/options/password_manager_handler.h b/chrome/browser/ui/webui/options/password_manager_handler.h
index 8dc188edb0840b036bf6946bed0b225a03e7b789..ee392457394e48d1d0f8d1f58d9eb0c13333325f 100644
--- a/chrome/browser/ui/webui/options/password_manager_handler.h
+++ b/chrome/browser/ui/webui/options/password_manager_handler.h
@@ -64,6 +64,10 @@ class PasswordManagerHandler : public OptionsPageUIHandler,
void SetPasswordList();
void SetPasswordExceptionList();
+ // Returns true if the user needs to be authenticated before a plaintext
+ // password is revealed.
+ bool IsAuthenticationRequired();
+
// A short class to mediate requests to the password store.
class ListPopulater : public PasswordStoreConsumer {
public:
@@ -123,9 +127,13 @@ class PasswordManagerHandler : public OptionsPageUIHandler,
// Whether to show stored passwords or not.
BooleanPrefMember show_passwords_;
- // Indicates whether or not the user has recently been authenticated.
- // Used to determine whether or not to reveal plain text passwords.
- bool is_user_authenticated_;
+ // Indicates whether or not the password manager should require the user to
+ // reauthenticate before revealing plaintext passwords.
+ bool require_reauthentication_;
+
+ // The last time the user was successfully authenticated.
+ // Used to determine whether or not to reveal plaintext passwords.
+ base::TimeTicks last_authentication_time_;
DISALLOW_COPY_AND_ASSIGN(PasswordManagerHandler);
};

Powered by Google App Engine
This is Rietveld 408576698