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

Unified Diff: chrome/browser/resources/options/manage_profile_overlay.js

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: Several fixes Created 7 years, 11 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/resources/options/manage_profile_overlay.js
diff --git a/chrome/browser/resources/options/manage_profile_overlay.js b/chrome/browser/resources/options/manage_profile_overlay.js
index 6e68ef41b0742470a821c3d76039afaf8eac2ced..6a6163f450a46b73c069b0e22b37f31f6a47fb8a 100644
--- a/chrome/browser/resources/options/manage_profile_overlay.js
+++ b/chrome/browser/resources/options/manage_profile_overlay.js
@@ -61,7 +61,8 @@ cr.define('options', function() {
if (loadTimeData.getBoolean('managedUsersEnabled')) {
$('create-profile-managed-container').hidden = false;
$('managed-user-settings-button').onclick = function(event) {
- OptionsPage.navigateToPage('managedUser');
+ chrome.send('displayPassphraseDialog',
+ ['ManageProfileOverlay.showManagedUserSettingsDialog']);
chrome.send('coreOptionsUserMetricsAction',
['Options_ManagedUserPassphraseOverlay']);
};
@@ -322,6 +323,15 @@ cr.define('options', function() {
showCreateDialog_: function() {
OptionsPage.navigateToPage('createProfile');
},
+ /** Display the "Managed User Settings" dialog.
+ * This method is only called if the entered password is correct.
+ * @private
+ */
+ showManagedUserSettingsDialog_: function() {
+ OptionsPage.navigateToPage('managedUser');
+ chrome.send('coreOptionsUserMetricsAction',
+ ['Options_FamilyControlSettings']);
+ },
};
// Forward public APIs to private implementations.
@@ -334,6 +344,7 @@ cr.define('options', function() {
'showManageDialog',
'showDeleteDialog',
'showCreateDialog',
+ 'showManagedUserSettingsDialog',
].forEach(function(name) {
ManageProfileOverlay[name] = function() {
var instance = ManageProfileOverlay.getInstance();

Powered by Google App Engine
This is Rietveld 408576698