[gnome-shell] loginDialog: don't ever call _reset directly
- From: Ray Strode <halfline src gnome org>
 
- To: commits-list gnome org
 
- Cc: 
 
- Subject: [gnome-shell] loginDialog: don't ever call _reset directly
 
- Date: Thu,  1 Aug 2013 20:35:45 +0000 (UTC)
 
commit 925eaa1db066d50dfd47aba95e22007abfea83f3
Author: Ray Strode <rstrode redhat com>
Date:   Sun Jul 28 15:55:09 2013 -0400
    loginDialog: don't ever call _reset directly
    
    The only time we ever call _reset directly is when
    detecting changes to disable-user-list.  We can implicitly
    trigger a reset for this case, just as easily by calling
    this._authPrompt.reset()
    
    This commit makes that change for consistency and to make
    it easier to adjust the authprompt workflow later.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=683437
 js/gdm/loginDialog.js |    9 +++------
 1 files changed, 3 insertions(+), 6 deletions(-)
---
diff --git a/js/gdm/loginDialog.js b/js/gdm/loginDialog.js
index 1651af9..40a4799 100644
--- a/js/gdm/loginDialog.js
+++ b/js/gdm/loginDialog.js
@@ -451,7 +451,7 @@ const LoginDialog = new Lang.Class({
 
         this._authPrompt = new AuthPrompt.AuthPrompt(gdmClient, AuthPrompt.AuthPromptMode.UNLOCK_OR_LOG_IN);
         this._authPrompt.connect('prompted', Lang.bind(this, this._onPrompted));
-        this._authPrompt.connect('reset', Lang.bind(this, this._reset));
+        this._authPrompt.connect('reset', Lang.bind(this, this._onReset));
         this._authPrompt.hide();
 
         this._authPrompt.actor.add_constraint(new Clutter.AlignConstraint({ source: this.actor,
@@ -530,7 +530,7 @@ const LoginDialog = new Lang.Class({
             this._disableUserList = disableUserList;
 
             if (!this._authPrompt.verifyingUser)
-                this._reset();
+                this._authPrompt.reset();
         }
     },
 
@@ -575,10 +575,7 @@ const LoginDialog = new Lang.Class({
         this._showPrompt();
     },
 
-    _reset: function() {
-        if (this._authPrompt.verifyingUser)
-            return;
-
+    _onReset: function() {
         this._sessionMenuButton.updateSensitivity(true);
 
         this._user = null;
[
Date Prev][
Date Next]   [
Thread Prev][
Thread Next]   
[
Thread Index]
[
Date Index]
[
Author Index]