Alice0775 White Reporter
	Description • 9 years ago

[Tracking Requested - why for this release]: Regression since Firefox41

I found a bug when I text Bug 1210784.

Steps To Reproduce:
1. Open attached

Actual Results:
z-order is wrong on CSS 3D animation.
See attached screenshot.


Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=ef53c6c25fb1&tochange=eed5d2d610e2

Regressed by:
eed5d2d610e2	L. David Baron — Bug 980770 - Enable off-main-thread animations on all platforms other than Linux (and other X11 based platforms) with off-main-thread compositing, for nightly/aurora. r=birtles

	Alice0775 White Reporter
	Comment 2 • 9 years ago

Setting layers.offmainthreadcomposition.async-animations = false fixes the z-order problem.

	David Baron :dbaron:
	Comment 3 • 9 years ago

I haven't looked closely yet, but it might be that the right fix for this is bug 1097464.

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

Regression, tracking.

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

Looks like we will be shipping 42 with this regression.
David, do you think it is worth tracking it in 43? Thanks

	Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)
	Comment 6 • 9 years ago

[Tracking Requested - why for this release]:

Is this still affecting 43 and 44?  If it's looking ok in 44, maybe bug 1097464 did fix it. 

Either way, I think we will be wontfixing this for 43.  If anyone disagrees or has a patch that looks upliftable, please let me know.

	Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo
	Comment 7 • 9 years ago

Unable to reproduce this in 45.0a1 (2015-11-11) MacOS 10.11.1 (15B42)

	Alice0775 White Reporter
	Comment 8 • 9 years ago

(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #6)
> [Tracking Requested - why for this release]:
> 
> Is this still affecting 43 and 44?  If it's looking ok in 44, maybe bug
> 1097464 did fix it. 
> 
> Either way, I think we will be wontfixing this for 43.  If anyone disagrees
> or has a patch that looks upliftable, please let me know.

I think OMTA should be simply disabled for 43.

	David Baron :dbaron:
	Comment 9 • 9 years ago

(In reply to Alice0775 White from comment #8)
> I think OMTA should be simply disabled for 43.

I disagree, at least on the basis of this bug (given that I believe we still haven't shipped correct z-ordering for 3-D transforms in any release).

	Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo
	Comment 10 • 9 years ago

I just re-read this and now see what Alice0774 was describing and it is *still* happening in 45 nightly.

	Cornel Ionce [:noni] [Hubs QA]
	Comment 12 • 9 years ago

I'm still seeing this on Windows 7 64-bit using:
- latest Aurora 44.0a2, build ID 20151111004055
- Firefox 43 beta 2, build ID 20151109145326.

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

Thinker, I am trying to follow up on 44+ tracked bugs and wondering if you might be able to help with this one. I know you worked on bug 1210882 that DBaron pointed to, so any help would be awesome.

	Thinker Li [:sinker]
	Comment 14 • 9 years ago

Sure! I would look into it.

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

Stop tracking this one as we shipped already several versions with it.
Please resubmit when we are ready to take that in a branch. Thanks