146575 - LDAP autocomplete password dialog triggers weird loss of focus

Status: RESOLVED WORKSFORME

(Core :: XUL, defect)

Description

[Dan Mosedale (:dmosedale, :dmose)]

Apply the patch in bug 135778.

Configure mail/news autocompletion to work against local addressbooks and an
LDAP server which requires authentication (ie has a bind DN set in the prefs page).

In the compose window addressing widget, type a string which will cause at least
1 match in a local addressbook.

The autocomplete menu may or may not drop down, depending on the number of matches.

An auth prompt dialog will pop up.  The textbox will not have have focus, and
will be extremely difficult to give focus to.  Clicking in the textbox won't do
it.  The only way I've been able to give focus to the textbox at all is by using
my window manager shortcut (alt-tab) to cycle through windows.

I'm using sawfish with click-to-focus on linux.

Comment 1

[Brian Ryner (not reading)]

Is this linux-only?

Comment 2

[Samir Gehani]

Nav triage team needs info: 

dmose, 
Does this occur on all platforms?  Does this problem occur when users aren't
using LDAP (i.e., only local addressbooks)?

Comment 3

[Dan Mosedale (:dmosedale, :dmose)]

I don't know whether this is linux only; i'll endeavor to figure that out
tommorrow.  It is specific to authenticated LDAP; if that's not in use, this bug
won't be seen.  However, if it is in use, it makes authenticated LDAP extremely
difficult to use.

Comment 4

[Dan Mosedale (:dmosedale, :dmose)]

This does not happen on Windows.

Comment 5

[Brian Ryner (not reading)]

blizzard, would you be able to look at this?

Comment 6

[Dan Mosedale (:dmosedale, :dmose)]

Lightly edited log of conversation about this bug:

<dmose> blizzard: any chance you could take a look at 
http://bugzilla.mozilla.org/show_bug.cgi?id=146575 ?
<dmose> it no longer requires applying a patch to reproduce; the code is now
checked in
<blizzard> dmose: is there a grab in effect when that auth dialog pops up?
<blizzard> dmose: I wonder.
<blizzard> dmose: you should track mozarea_focus_in and mozarea_focus_out
<blizzard> dmose: and CaptureRollupEvents()
<blizzard> dmose: in nsWindow.cpp
<dmose> blizzard: there could well be a grab in effect, yeah
<blizzard> dmose: "don't do that"
<dmose> heh
<blizzard> dmose: serious
<blizzard> dmose: don't launch a dialog when you've got a grab
<dmose> every other platform seems to handle this situation fine
<blizzard> dmose: a grab is a grab is a grab
* dmose sighs
<dmose> well, let me verify about the grab
<dmose> how would i tell?
<blizzard> dmose: Whether or not there's a PR_TRUE argument to CaptureRollupEvents
<dmose> alright
<blizzard> dmose: other platforms might be a little less sensitive to changes
like this, that's all
<blizzard> dmose: it's not for lack of trying
<blizzard> dmose: I've spent months of my life trying to get this crap right
<dmose> mmmm, fun fun
<dmose> alright, well i will investigate further and see what i can figure out
<dmose> thanks for the hints
<blizzard> dmose: no problem

Comment 7

[Samir Gehani]

adt: nsbeta1-

Comment 8

[Andrei Purice]

Hey Dan,
Is this issue still reproducible for you or should we close it?

Comment 9

[Dan Mosedale (:dmosedale, :dmose)]

I'd say close it.

Comment 10

[Andrei Purice]

Marking this as Resolved > Worksforme based on the last comment.