Closed Bug 1334155 Opened 8 years ago Closed 8 years ago

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

Categories

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

50 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: ilisepe1, Assigned: enndeakin)

References

(Depends on 1 open bug, Blocks 1 open bug, )

Details

Attachments

(1 file)

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.
Component: Untriaged → Layout: Form Controls
Product: Firefox → Core
It happens only with multiprocess enabled. I think it's a dupe of bug 910022.
Blocks: e10s-select
Summary: selectbox onfocus="this.blur();" strange incompatibility → [e10s] selectbox onfocus="this.blur();" strange incompatibility
Whiteboard: DUPEME
Status: UNCONFIRMED → NEW
Ever confirmed: true
Whiteboard: DUPEME
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.
Severity: normal → major
Priority: -- → P2
Any news?
Attached patch Hide popup on blur (deleted) — Splinter Review
I also moved the event listeners to the system group.
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Attachment #8834898 - Flags: review?(jaws)
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.
Attachment #8834898 - Flags: review?(jaws) → review+
This will need to be rebased as I just landed bug 1336125 and bug 1336301 on autoland.
Pushed by neil@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/f3bf536f5b98 close popup when select element is blurred, r=jaws
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Depends on: 1352817
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.
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: