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

Unified Diff: chrome/browser/resources/chromeos/login/screen_signin.js

Issue 7382001: [ChromeOS] WebUI OOBE/Login refactoring. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync and merge Created 9 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
« no previous file with comments | « chrome/browser/chromeos/login/webui_login_display_host.cc ('k') | chrome/browser/resources/login.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/chromeos/login/screen_signin.js
diff --git a/chrome/browser/resources/chromeos/login/screen_signin.js b/chrome/browser/resources/chromeos/login/screen_signin.js
index d456765cbc14b0f0873715f0004e3c99c7087e15..40e6b60c679ed85efc7eafc34777ea7c29ebc095 100644
--- a/chrome/browser/resources/chromeos/login/screen_signin.js
+++ b/chrome/browser/resources/chromeos/login/screen_signin.js
@@ -32,6 +32,8 @@ cr.define('login', function() {
/** @inheritDoc */
decorate: function() {
+ $('email').addEventListener('keydown', this.handleKeyDown_.bind(this));
+ $('password').addEventListener('keydown', this.handleKeyDown_.bind(this));
},
/**
@@ -93,14 +95,51 @@ cr.define('login', function() {
},
/**
+ * Validates input fields.
+ */
+ checkInput: function() {
+ // Validate input.
+ if (!$('email').value) {
+ $('email').focus();
+ return false;
+ }
+
+ if (!$('password').value) {
+ $('password').focus();
+ return false;
+ }
+
+ return true;
+ },
+
+ /**
* Handles sign in button click.
* @private
*/
handleSigninClick_: function(e) {
+ if (!this.checkInput())
+ return;
+
this.state = SigninScreen.STATE_AUTHENTICATING;
chrome.send('authenticateUser',
[$('email').value, $('password').value]);
+ },
+
+ /**
+ * Handles keyboard event.
+ * @private
+ */
+ handleKeyDown_: function(e) {
+ // Handle 'Enter' key for 'email' and 'password' field.
+ if (e.keyIdentifier == 'Enter') {
+ if (e.target.id == 'email') {
+ if (e.target.value)
+ $('password').focus();
+ } else if (e.target.id == 'password') {
+ this.handleSigninClick_();
+ }
+ }
}
};
« no previous file with comments | « chrome/browser/chromeos/login/webui_login_display_host.cc ('k') | chrome/browser/resources/login.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698