1357841 - The status panel does not mirror on non-e10s content

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

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

STR:
Open about:newtab and reduce the height of the browser window so one of the new tab thumbnails is clipped by the bottom of the window.
Hover over the left-most (on a LTR setup)

Expected results:
The status panel should not overlap the mouse cursor

Actual results:
The status panel appears on the left-side and beneath the mouse cursor. It should mirror to the other side.

Comment 1

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

Confirmed via mozregression that bug 1356183 introduced this. I used about:about to test.

Comment 2

[Florian Quèze [:florian]]

Given the platforms patches that landed in bug 1307134, should we consider backing out bug 1356183?

Comment 3

[Dão Gottwald [:dao]]

(In reply to Florian Quèze [:florian] [:flo] from comment #2)
> Given the platforms patches that landed in bug 1307134, should we consider
> backing out bug 1356183?

Maybe, but I think we should also investigate what's wrong with clientX. This seems like a platform bug. In fact it's probably why bug 646038 wasn't using clientX in the first place...

Comment 4

[Timothy Nikkel (:tnikkel)]

Is what clientX is relative to in e10s different from non-e10s? The event is dispatched to the same window? I don't really know this event stuff well.

Comment 5

[Dão Gottwald [:dao]]

(In reply to Timothy Nikkel (:tnikkel) from comment #4)
> Is what clientX is relative to in e10s different from non-e10s?

Yeah.

> The event is dispatched to the same window? I don't really know this event stuff well.

For non-e10s the event is dispatched on content nodes, and the numbers make sense for any content listener but seem weird for chrome listeners. For e10s or when moving the mouse pointer over the browser chrome, the numbers become relative to the browser window.

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

Sounds like browser/toolkit bug to me then.

Comment 7

[Dão Gottwald [:dao]]

Okay, backed out: https://hg.mozilla.org/integration/mozilla-inbound/rev/f18e288534fbc91b47a4d11a25409fdae274a1d2

Not going to back this out from 54 unless bug 1307134 gets uplifted too.

Comment 8

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

Looks like the fix is going to apply for 55 but we will still have this bug in 54.