authPrompt: cancel user verification if verifying when reset
authPrompt.reset() currently only leaves the authPrompt in a sane state if the user isn't verifying. This commit makes sure to cancel verification if a reset happens while verification is in process. https://bugzilla.gnome.org/show_bug.cgi?id=683437
This commit is contained in:
parent
c4ca17d127
commit
ebce362c05
@ -232,8 +232,10 @@ const AuthPrompt = new Lang.Class({
|
||||
},
|
||||
|
||||
_onReset: function() {
|
||||
if (this.verificationStatus != AuthPromptStatus.VERIFICATION_SUCCEEDED)
|
||||
if (this.verificationStatus != AuthPromptStatus.VERIFICATION_SUCCEEDED) {
|
||||
this.verificationStatus = AuthPromptStatus.NOT_VERIFYING;
|
||||
this.reset();
|
||||
}
|
||||
},
|
||||
|
||||
_onShowLoginHint: function(verifier, message) {
|
||||
@ -405,7 +407,12 @@ const AuthPrompt = new Lang.Class({
|
||||
},
|
||||
|
||||
reset: function() {
|
||||
let oldStatus = this.verificationStatus;
|
||||
this.verificationStatus = AuthPromptStatus.NOT_VERIFYING;
|
||||
|
||||
if (oldStatus == AuthPromptStatus.VERIFYING)
|
||||
this._userVerifier.cancel();
|
||||
|
||||
this._queryingService = null;
|
||||
this.clear();
|
||||
this._message.opacity = 0;
|
||||
@ -452,9 +459,6 @@ const AuthPrompt = new Lang.Class({
|
||||
},
|
||||
|
||||
cancel: function() {
|
||||
if (this.verificationStatus == AuthPromptStatus.VERIFYING)
|
||||
this._userVerifier.cancel();
|
||||
|
||||
this.reset();
|
||||
}
|
||||
});
|
||||
|
Loading…
Reference in New Issue
Block a user