1106824 - Focus does not change when click outside of select dropdown list

Status: RESOLVED FIXED

(Core :: DOM: UI Events & Focus Handling, defect)

Description

[Alice0775 White]

Attachment: testcase

Focus does not change when click outside of select dropdown.

Steps To Reproduce:
1. Focus <select> element and Press Alt + down arrow key to popup dropdown
2. Click outside of the <select> element and the dropdown list

Actual Results:
The <select> element still has focus.
In other words, onblur event does not fire on the <select> element. This causes Bug 1106812.

Expected Results:
Clicked element should get focus.
I.e, onblur event should fire on the <select> element. 


The problem does not happen on IE11 and Chrome39.

Comment 1

[Mats Palmgren (inactive)]

> Expected Results:
> Clicked element should get focus.

Yes, I think that's expected on Windows, but not on Linux or OSX.

It might be as simple as returning false in nsIRollupListener::Rollup...

Alice, please test this build when it's ready:
https://tbpl.mozilla.org/?usebuildbot=1&tree=Try&rev=2ff5ee883542

Comment 2

[Alice0775 White]

(In reply to Mats Palmgren (:mats) from comment #1)
> > Expected Results:
> > Clicked element should get focus.
> 
> Yes, I think that's expected on Windows, but not on Linux or OSX.
> 
> It might be as simple as returning false in nsIRollupListener::Rollup...
> 
> Alice, please test this build when it's ready:
> https://tbpl.mozilla.org/?usebuildbot=1&tree=Try&rev=2ff5ee883542

The try build works as expected for this testcase, and also works as expected for Bug 1106812.

https://hg.mozilla.org/try/rev/2ff5ee883542
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:37.0) Gecko/20100101 Firefox/37.0 ID:20141202155041

Comment 3

[Mats Palmgren (inactive)]

There was a "layout/xul/test/test_bug987230.xul | Popup should only be shown once"
failure on Try though.  So let's try without the nsXULPopupManager change:
https://tbpl.mozilla.org/?tree=Try&rev=b4e70d64964a

Comment 4

[Mats Palmgren (inactive)]

Attachment: fix

Comment 5

[Mats Palmgren (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d37ced519d4d

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d37ced519d4d