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

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

Issue 11783008: Add a lock to the managed user settings page and require authentication for unlocking. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address review comments. Created 7 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: chrome/browser/ui/webui/options/managed_user_passphrase_handler.h
diff --git a/chrome/browser/ui/webui/options/managed_user_passphrase_handler.h b/chrome/browser/ui/webui/options/managed_user_passphrase_handler.h
index 9c1a0769195c1ba202960ee06abb8763f588ee80..fd35f872415f006ea45ee37258a4a526a33daf73 100644
--- a/chrome/browser/ui/webui/options/managed_user_passphrase_handler.h
+++ b/chrome/browser/ui/webui/options/managed_user_passphrase_handler.h
@@ -28,11 +28,34 @@ class ManagedUserPassphraseHandler : public OptionsPageUIHandler {
base::DictionaryValue* localized_strings) OVERRIDE;
private:
- // This function is used to set the passphrase of the manager of the managed
- // account. The passphrase is expected as the parameter and is passed in from
- // the UI.
+ // Used to set the passphrase of the manager of the managed account. The
James Hawkins 2013/02/20 17:24:56 nit: // Sets the passphrase...
Adrian Kuegel 2013/02/20 18:05:02 Done.
+ // passphrase is expected as the parameter and is passed in from the UI.
void SetLocalPassphrase(const base::ListValue* args);
+ // Displays the passphrase dialog where the manager of the managed account
+ // can enter the passphrase which allows him to modify the settings. It
+ // expects as parameter the name of the Javascript function which should be
+ // called after the passphrase has been checked.
+ void DisplayPassphraseDialog(const base::ListValue* args);
+
+ // Will be called after the user either clicked Cancel or successfully entered
James Hawkins 2013/02/20 17:24:56 nit: Document what it does, not when it's called.
Adrian Kuegel 2013/02/20 18:05:02 Done.
+ // the passphrase and clicked Unlock in the Passphrase Dialog.
+ void PassphraseDialogCallback(bool success);
+
+ // Resets the authentication state of the manager of the managed account.
+ void EndAuthentication(const base::ListValue* args);
+
+ // Checks if there is already a passphrase specified. It expects as parameter
+ // the name of the Javascript function which should be called with the results
+ // of this check.
+ void IsPassphraseSet(const base::ListValue* args);
+
+ // Resets the passphrase to the empty string.
+ void ResetPassphrase(const base::ListValue* args);
+
+ // Used to save the name of the Javascript function which should be called
James Hawkins 2013/02/20 17:24:56 nit: // The name of the...
Adrian Kuegel 2013/02/20 18:05:02 Done.
+ // after the passphrase has been checked.
+ std::string callback_function_name_;
base::WeakPtrFactory<ManagedUserPassphraseHandler> weak_ptr_factory_;
DISALLOW_COPY_AND_ASSIGN(ManagedUserPassphraseHandler);

Powered by Google App Engine
This is Rietveld 408576698