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

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

Issue 3074010: Revert 53989 - Implement new mock for user options page per chromium-os:5028... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 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: chrome/browser/resources/options/chromeos_accounts_options.js
===================================================================
--- chrome/browser/resources/options/chromeos_accounts_options.js (revision 54126)
+++ chrome/browser/resources/options/chromeos_accounts_options.js (working copy)
@@ -33,35 +33,32 @@
OptionsPage.prototype.initializePage.call(this);
// Set up accounts page.
+ $('addUserButton').onclick = function(e) {
+ OptionsPage.showOverlay('addUserOverlay');
+ };
+ $('removeUserButton').onclick = function(e) {
+ $('userList').removeSelectedUser();
+ };
+
options.accounts.UserList.decorate($('userList'));
- var userNameEdit = $('userNameEdit');
- options.accounts.UserNameEdit.decorate(userNameEdit);
- userNameEdit.addEventListener('add', this.handleAddUser_);
+ this.addEventListener('visibleChange',
+ cr.bind(this.handleVisibleChange_, this));
- this.addEventListener('visibleChange', this.handleVisibleChange_);
+ // Setup add user overlay page.
+ OptionsPage.registerOverlay(AddUserOverlay.getInstance());
},
userListInitalized_: false,
/**
* Handler for OptionsPage's visible property change event.
- * @private
* @param {Event} e Property change event.
*/
- handleVisibleChange_: function(e) {
+ handleVisibleChange_ : function(e) {
if (!this.userListInitalized_ && this.visible) {
this.userListInitalized_ = true;
userList.redraw();
}
- },
-
- /**
- * Handler for "add" event fired from userNameEdit.
- * @private
- * @param {Event} e Add event fired from userNameEdit.
- */
- handleAddUser_: function(e) {
- $('userList').addUser(e.user);
}
};

Powered by Google App Engine
This is Rietveld 408576698