arni2033 Reporter
	Description • 9 years ago

STR:
1. Open page data:text/html,<select><option>First<option>Second
2. Click on <select> element
3. Click on dropmarker

RESULT: (on clear Nightly profile)
The drop-down list hasn't collapsed

EXPECTATIONS:
It should've collapsed

	arni2033 Reporter
	Comment 2 • 9 years ago

(In reply to Towkir Ahmed [:Towkir] from comment #1)
> 
> *** This bug has been marked as a duplicate of bug 1153586 ***

I have to disagree with this. That bug was fixed in April, but still is presented in Fx Release 39.0
THIS bug is about <select> behavior in latest Nightly in e10s mode.
Try this page with e10s: data:text/html,<select><option>First

	[:Towkir] Ahmed
	Comment 3 • 9 years ago

Sorry about that arni, but you filed it against e10s and the version was trunk, so I tested on latest nightly and got nothing like that, but actually this exists on release version 39 (not reproducible each time)
Build ID 	20150630154324
User Agent 	Mozilla/5.0 (Windows NT 6.1; rv:39.0) Gecko/20100101 Firefox/39.0

	Loic
	Comment 4 • 9 years ago

It's present in FF42 with e10s enabled, not present with e10s disabled.

About FF39, we don't care because e10s is off by default (and FF39 is almost EOLed, so no bug fix like that).

	Loic
	Comment 5 • 9 years ago

It regressed and it's similar to bug 1182607 which has been recently fixed.
I guess it's an edge case due to style="position:fixed; bottom:200px".

Regression range:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=0093691d3715&tochange=b7ee8e13145a

suspected: bug 1171973

	Neil Deakin Assignee
	Comment 6 • 9 years ago

Is this not fixed, or just a duplicate of 1182607?

	Loic
	Comment 7 • 9 years ago

On my side, testcase of bug 1182607 is not fixed with e10s on, so this current bug is probably a dupe of bug 1182607.

Do you know why bug 1182607 appears unfixed?

	arni2033 Reporter
	Comment 8 • 9 years ago

I tested it more carefully

I have default font size on my Win7 = 125%. When I launch Nightly with layout.css.devPixelsPerPx = -1.0 [default] then I can't close <select> by clicking on it. You can test it by setting that pref to 1.25

If I set layout.css.devPixelsPerPx = 1.0 then I can close <select> if I do the following:
1. Click on <select>
2. Wait 2 seconds
3. Click on <select>
BUT if I DoubleClick <select> (that is initially closed) then it isn't closed. It just causes selected option in list to lose blue background.

	Loic
	Comment 9 • 9 years ago

I confirm that, it's not fixed on HiDPI screens. Mine is Win 7 at 125%, it's not fixed with e10s enabled, if I revert to 100%, it works.

Title modified according to that.

	arni2033 Reporter
	Comment 10 • 9 years ago

(In reply to Loic from comment #9)
> Title modified according to that.

How about DoubleClick on dropmarker not collapsing the list?

	Loic
	Comment 11 • 9 years ago

I guess everybody got that. ^^

	[:Towkir] Ahmed
	Comment 12 • 9 years ago

Are double clicks supposed to collapse a dropdown list ???

	:Felipe Gomes (needinfo for replies!)
	Comment 13 • 9 years ago

Jared, you've got a Win10 Hi-DPI device, right? Could you try to reproduce this bug?

	Jared Wein [:jaws] (please needinfo? me)
	Comment 14 • 9 years ago

(In reply to :Felipe Gomes from comment #13)
> Jared, you've got a Win10 Hi-DPI device, right? Could you try to reproduce
> this bug?

Yeah, I can reproduce this on Windows with Windows set to 150% DPI. Clicking on the text within the <select> element (to open the dropdown) then clicking on the dropdown marker causes the dropdown to flicker (I believe it is closing then reopening very quickly).

	Jared Wein [:jaws] (please needinfo? me)
	Comment 15 • 9 years ago

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #14)
> (In reply to :Felipe Gomes from comment #13)
> > Jared, you've got a Win10 Hi-DPI device, right? Could you try to reproduce
> > this bug?
> 
> Yeah, I can reproduce this on Windows with Windows set to 150% DPI. Clicking
> on the text within the <select> element (to open the dropdown) then clicking
> on the dropdown marker causes the dropdown to flicker (I believe it is
> closing then reopening very quickly).

(repro'd on Windows 8.1 with 150% DPI and Nightly 42 2015-07-28)

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 17 • 9 years ago

Tracked as it's a regression.

	Neil Deakin Assignee
	Comment 18 • 9 years ago

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/fd0e8165c50f8b58e50d59b7abb629cdb62beb72
changeset:  fd0e8165c50f8b58e50d59b7abb629cdb62beb72
user:       Neil Deakin <neil@mozilla.com>
date:       Mon Aug 10 08:31:37 2015 -0400
description:
Bug 1186972, use correct coordinates when comparing mouse position with menu anchor rectangle so that select element popups close properly, r=tn

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 19 • 9 years ago

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

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 20 • 9 years ago

Brad, this bug is marked FF41+ tracking. Given that the fix has been in Nightly for a week, do you think it is safe to uplift the fix (seems simple enough) to Beta and Aurora? Thanks!

	Brad Lassey [:blassey] (use needinfo?)
	Comment 21 • 9 years ago

redirecting to Neil

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 22 • 9 years ago

Removing tracking for FF41 as it is not enabled by default. It is already tracked for FF42.

	Neil Deakin Assignee
	Comment 23 • 9 years ago

Comment on attachment 8640616 [details] [diff] [review]
Convert mouse coordinates from device to css pixels

Approval Request Comment
[Feature/regressing bug #]: revealed by 1171973
[User impact if declined]: select popup reopens when clicking twice
[Describe test coverage new/current, TreeHerder]: mochitest isn't currently able to handle testing this, but tested manually, and fixes an obvious incorrect usage of coordinates
[Risks and why]: e10s only, only affects high DPI screen modes
[String/UUID change made/needed]: none

	Sylvestre Ledru [:Sylvestre]
	Comment 24 • 9 years ago

Comment on attachment 8640616 [details] [diff] [review]
Convert mouse coordinates from device to css pixels

Improve e10s, taking it as it is now the default in aurora.

	Ryan VanderMeulen [:RyanVM]
	Comment 25 • 9 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/76c987fe2b25