Comment 58 for bug 181575

Revision history for this message
In , Enn (enndeakin) wrote :

(In reply to comment #55)
> I don't see how that patch is related to this bug. The URL bar uses
> completeselectedindex="true" so that patch isn't going to affect it,

Because completeselectedindex is true, the call to GetResultValueAt which fills in the selected value from the popup when enter is pressed will be skipped:

- if (selectedIndex >= 0)
+ if (selectedIndex >= 0 && !completeSelection)
     GetResultValueAt(selectedIndex, PR_TRUE, value);

> the problem really is at step 4 of comment 0, which is caused by bug 297080 (and

I can't actually reproduce the problem given the steps in comment 0. I do see the problem of the wrong url being used when enter is pressed.

Bug 417053 is a possible fix, although it is arguably invalid, as menus do not behave this way (with the mouse not affecting the selected item) and one could say that the autocomplete popup should behave in the same way.