1296270 - e10s / non-e10s different mouse events behavior on <select> elements

Status: RESOLVED DUPLICATE of bug 1300784

(Core :: DOM: Core & HTML, defect, P2)

Description

[Iulia Cristescu, QA [:JuliaC] (away, please needinfo? [e-mail])]

[Note]:
- This is a follow up issue filed for Bug 1291078.

[Affected versions]:
- 51.0a1 (2016-08-17)
- 50.0a2 (2016-08-18)
- 49.0b4 build1 (20160814184416)
- 48.0.1 build3 (20160817112116)

[Affected platforms]:
- Windows 10 x64
- Ubuntu 14.04 x86 
- Mac OS X 10.10.4

[Steps to reproduce]:
1. Launch Firefox.
2. Enable e10s (if it isn't already).
3. Open https://bug1291078.bmoattachments.org/attachment.cgi?id=8776788.
4. Click the right menu button in order to open the menu.
5. Click again the right menu button in order to close the menu.
- inspect the last fired events

[Expected result]:
- e10s and non-e10s behaviors correspond:  
 On step 5
 * mousedown
 * mouseup
 * click 
 are displayed (like in non-e10s case).

[Actual result]:
- on Windows, when e10s is on and on Ubuntu (e10s on/off), 
  * mouseup
  * click 
  are displayed;
- on Mac (e10s on/off), nothing is displayed.

[Regression range]:
- I will investigate this as soon as possible.

Comment 1

[Tracy Walker [:tracy]]

The menu opens (step 4) and closes (step 5) as expected. There is nothing user facing that is unexpected here.

Comment 2

[Alice0775 White]

Regression window(Win32 build on Windows10 x64):
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=dc24497dbda4&tochange=3e382ae494a8

Triggered by:3e382ae494a8	Mike Conley — Bug 1108761 - Make the parent process use a menulist popup for <select>'s in the content process. feedback=dao,r=Enn.

Comment 3

[Sylvestre Ledru [:Sylvestre]]

I don't think this is worth taking into a dot release. Wontfix for 48

Comment 4

[Mike Conley (:mconley) (:⚙️)]

(In reply to Sylvestre Ledru [:sylvestre] from comment #3)
> I don't think this is worth taking into a dot release. Wontfix for 48

I agree, this is minor. This is another one of those things that the other browsers are all inconsistent on as well.

Comment 5

[Mike Conley (:mconley) (:⚙️)]

(In reply to Sylvestre Ledru [:sylvestre] from comment #3)
> I don't think this is worth taking into a dot release. Wontfix for 48

I agree, this is an interesting find, but pretty minor, imo.

Comment 6

[Andrew Overholt [:overholt]]

Are we going to try to fix this for >= 49, Mike? (Asking same question as in bug 1296264 just to keep these even more similar ;)

Comment 7

[Mike Conley (:mconley) (:⚙️)]

Yes. jaws and I are mentoring some MSU students who are going to try to combine the e10s and non-e10s select dropdown implementations. I'll see if I can make this part of the stack of work there.

Comment 8

[Andrew Overholt [:overholt]]

(In reply to Mike Conley (:mconley) from comment #7)
> Yes. jaws and I are mentoring some MSU students who are going to try to
> combine the e10s and non-e10s select dropdown implementations. I'll see if I
> can make this part of the stack of work there.

Were you able to do that, Mike?

Comment 9

[Mike Conley (:mconley) (:⚙️)]

(In reply to Andrew Overholt [:overholt] from comment #8)
> (In reply to Mike Conley (:mconley) from comment #7)
> > Yes. jaws and I are mentoring some MSU students who are going to try to
> > combine the e10s and non-e10s select dropdown implementations. I'll see if I
> > can make this part of the stack of work there.
> 
> Were you able to do that, Mike?

The work is likely to land in 53 (bug 1300784), though disabled until we've had QA bang the tires a bit to make sure it works as expected.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

AFAIK, we have no intention of backporting bug 1300784 to Fx52, so calling this wontfix for that release.

Comment 11

[Hsin-Yi Tsai (she/her) [:hsinyi]]

Hi Mike, anything we need to do here? Or is this simply dupe. of bug 1300784?

Comment 12

[Mike Conley (:mconley) (:⚙️)]

Yeah, let's dupe this over.

Comment 13

[Liz Henry (:lizzard) (relman/hg->git project)]

Too late for a fix for 53, as we are in the last week of the 53 beta cycle.