From bd6220c0450ab117c851fb1d21ca35ceb76739f8 Mon Sep 17 00:00:00 2001

From: Ray Strode <rstrode@redhat.com>

Date: Wed, 30 Sep 2015 12:51:24 -0400

Subject: [PATCH 1/3] authPrompt: don't fade out auth messages if user types

 password up front

Right now we fade out any stale auth messages as soon as the user starts

typing. This behavior doesn't really make sense if the user is typing up

front, before a password is asked.

---

 js/gdm/authPrompt.js | 2 +-

 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/js/gdm/authPrompt.js b/js/gdm/authPrompt.js

index 481cd3a77..0ad3d2338 100644

--- a/js/gdm/authPrompt.js

+++ b/js/gdm/authPrompt.js

@@ -176,7 +176,7 @@ var AuthPrompt = new Lang.Class({

         this._updateNextButtonSensitivity(this._entry.text.length > 0);

         this._entry.clutter_text.connect('text-changed', () => {

-            if (!this._userVerifier.hasPendingMessages)

+            if (!this._userVerifier.hasPendingMessages && this._queryingService && !this._preemptiveAnswer)

                 this._fadeOutMessage();

             this._updateNextButtonSensitivity(this._entry.text.length > 0 || this.verificationStatus == AuthPromptStatus.VERIFYING);

-- 

2.20.1

From 1aeff45d29983f8b0e970eb9bf15e5d19aa27903 Mon Sep 17 00:00:00 2001

From: Ray Strode <rstrode@redhat.com>

Date: Wed, 30 Sep 2015 14:36:33 -0400

Subject: [PATCH 2/3] authPrompt: don't spin unless answering question

---

 js/gdm/authPrompt.js | 2 +-

 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/js/gdm/authPrompt.js b/js/gdm/authPrompt.js

index 0ad3d2338..f18ef41f2 100644

--- a/js/gdm/authPrompt.js

+++ b/js/gdm/authPrompt.js

@@ -66,8 +66,8 @@ var AuthPrompt = new Lang.Class({

         this.connect('next', () => {

                 this.updateSensitivity(false);

-                this.startSpinning();

                 if (this._queryingService) {

+                    this.startSpinning();

                     this._userVerifier.answerQuery(this._queryingService, this._entry.text);

                 } else {

                     this._preemptiveAnswer = this._entry.text;

-- 

2.20.1

From 6afeaae067b5ad280a0899bd6b10931a9b2f02e4 Mon Sep 17 00:00:00 2001

From: Ray Strode <rstrode@redhat.com>

Date: Mon, 5 Oct 2015 15:26:18 -0400

Subject: [PATCH 3/3] authPrompt: stop accepting preemptive answer if user

 stops typing

We only want to allow the user to type the preemptive password in

one smooth motion.  If they start to type, and then stop typing,

we should discard their preemptive password as expired.

Typing ahead the password is just a convenience for users who don't

want to manually lift the shift before typing their passwords, after

all.

---

 js/gdm/authPrompt.js | 37 +++++++++++++++++++++++++++++++++++++

 1 file changed, 37 insertions(+)

diff --git a/js/gdm/authPrompt.js b/js/gdm/authPrompt.js

index f18ef41f2..89cef4d5d 100644

--- a/js/gdm/authPrompt.js

+++ b/js/gdm/authPrompt.js

@@ -10,6 +10,7 @@ const St = imports.gi.St;

 const Animation = imports.ui.animation;

 const Batch = imports.gdm.batch;

 const GdmUtil = imports.gdm.util;

+const Meta = imports.gi.Meta;

 const Params = imports.misc.params;

 const ShellEntry = imports.ui.shellEntry;

 const Tweener = imports.ui.tweener;

@@ -47,6 +48,8 @@ var AuthPrompt = new Lang.Class({

         this._gdmClient = gdmClient;

         this._mode = mode;

+        this._idleMonitor = Meta.IdleMonitor.get_core();

+

         let reauthenticationOnly;

         if (this._mode == AuthPromptMode.UNLOCK_ONLY)

             reauthenticationOnly = true;

@@ -71,6 +74,11 @@ var AuthPrompt = new Lang.Class({

                     this._userVerifier.answerQuery(this._queryingService, this._entry.text);

                 } else {

                     this._preemptiveAnswer = this._entry.text;

+

+                    if (this._preemptiveAnswerWatchId) {

+                        this._idleMonitor.remove_watch(this._preemptiveAnswerWatchId);

+                        this._preemptiveAnswerWatchId = 0;

+                    }

                 }

             });

@@ -135,6 +143,11 @@ var AuthPrompt = new Lang.Class({

     },

     _onDestroy() {

+        if (this._preemptiveAnswerWatchId) {

+            this._idleMonitor.remove_watch(this._preemptiveAnswerWatchId);

+            this._preemptiveAnswerWatchId = 0;

+        }

+

         this._userVerifier.destroy();

         this._userVerifier = null;

     },

@@ -349,6 +362,11 @@ var AuthPrompt = new Lang.Class({

     },

     setQuestion(question) {

+        if (this._preemptiveAnswerWatchId) {

+            this._idleMonitor.remove_watch(this._preemptiveAnswerWatchId);

+            this._preemptiveAnswerWatchId = 0;

+        }

+

         this._label.set_text(question);

         this._label.show();

@@ -434,12 +452,31 @@ var AuthPrompt = new Lang.Class({

         }

     },

+    _onUserStoppedTypePreemptiveAnswer() {

+        if (!this._preemptiveAnswerWatchId ||

+            this._preemptiveAnswer ||

+            this._queryingService)

+            return;

+

+        this._idleMonitor.remove_watch(this._preemptiveAnswerWatchId);

+        this._preemptiveAnswerWatchId = 0;

+

+        this._entry.text = '';

+        this.updateSensitivity(false);

+    },

+

     reset() {

         let oldStatus = this.verificationStatus;

         this.verificationStatus = AuthPromptStatus.NOT_VERIFYING;

         this.cancelButton.reactive = true;

         this.nextButton.label = _("Next");

+        if (this._preemptiveAnswerWatchId) {

+            this._idleMonitor.remove_watch(this._preemptiveAnswerWatchId);

+        }

+        this._preemptiveAnswerWatchId = this._idleMonitor.add_idle_watch (500,

+                                                                          this._onUserStoppedTypePreemptiveAnswer.bind(this));

+

         if (this._userVerifier)

             this._userVerifier.cancel();

-- 

2.20.1