Comment 75 for bug 940631

Revision history for this message
In , Kai Engert (kaie) wrote :

(In reply to Matt from comment #61)
> I was reading in Bug 1361838#c2 that .uiBusy == true might be suitable for mitigating the issue

As I understand it, that's simply a flag that can be queried, whether the MP dialog is currently shown.

Detecting this state is only part of the problem. You still need to change all places that obtain logins to query for that state, and try again at a later time, if busy.

Instead of "query is MP prompt currently shown", the potential approach to change "give me the login" to return a "would block" state might be better, as it can prevent a race. (E.g. code queries if MP is shown, gets reply "no", decides to query for login, but in the meantime another thread has triggered the MP. This could still result in two MP prompts being shown.)