1334155 - [e10s] selectbox onfocus="this.blur();" strange incompatibility

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect, P2)

Description

[ilisepe1]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:43.0) Gecko/20100101 Firefox/43.0
Build ID: 20160106234723

Steps to reproduce:

<select name="foo" onfocus="this.blur();"><option>1</option><option>2</option></select>



Actual results:

With firefox ver <50 the user won't be able to open the select box.

With firefox ver >50 (51 for sure) the user can expand the select box and select anything. This happens *only* with windows 7 or 10. With windows XP or linux (ubuntu 14.04) the user can't open it.


Expected results:

The user should not be able to open the select box. That's what was happening since firefox version 2.

Comment 1

[Loic]

It happens only with multiprocess enabled. I think it's a dupe of bug 910022.

Comment 2

[ilisepe1]

I can confirm that disabling multiprocess solves the problem.

Disabling was tricky though.
I ended up trying some of the following solutions:
about:config -> browser.tabs.remote.autostart set false
about:config -> browser.tabs.remote.autostart.2 set false (or delete)

and then restart firefox
about:support -> check that Multiprocess Windows is 0/1

Apart from that I think that you need to fix this bug so firefox is consistent with this onfocus="this.blur()" feature! Some people may use this feature for restricting access to their web apps...
I vote for major.

Comment 3

[ilisepe1]

Any news?

Comment 4

[Neil Deakin]

Attachment: Hide popup on blur

I also moved the event listeners to the system group.

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8834898 [details] [diff] [review]
Hide popup on blur

Review of attachment 8834898 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/base/content/test/general/browser_selectpopup.js
@@ +795,5 @@
>    yield hideSelectPopup(selectPopup, "escape");
>    yield BrowserTestUtils.removeTab(tab);
>  });
> +
> +// This test checks that the popup is closed when the select element is blurred. 

note, the trailing space here will fail eslint.

Comment 6

[Jared Wein [:jaws] (please needinfo? me)]

This will need to be rebased as I just landed bug 1336125 and bug 1336301 on autoland.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f3bf536f5b98
close popup when select element is blurred, r=jaws

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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

Comment 9

[Andrei Vaida [:avaida]]

Since the patch pushed for this bug also contains an automated test, I don't think manual testing would be of much value here.

Neil, if you think Manual QA should instead be looking at this, feel free to flip the qe-verify flag or ni? me directly.