Julien Wajsberg [:julienw] Reporter
	Description • 8 years ago

[Tracking Requested - why for this release]:

STR:
1. Go to https://www.paris-web.fr/association/membres.php
2. put your mouse on a non empty space
3. Try to scroll using the mousewheel (or anything similar like the trackpad scroll -- on Linux at least)
=> It doesn't work at all.

This works when doing the same gesture on the blue background on the left or the right of the page.
This also works when the page is in "mobile" mode (you can press ctrl + + repeatidly to enter this mode, or enter responsive mode).

Reproducing in Aurora 46 and Nightly 47 (Build ID: 20160211030242).
NOT reproducing in Beta 45 and Stable 44.

Nothing looks unusual in this page's CSS. The obvious difference between the mobile and desktop versions of the site is the use of a "float" in the desktop version.

	Alice0775 White
	Comment 1 • 8 years ago

Regression window: 
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=2ec54b38a33da939a8255612999d9e867eb11664&tochange=bb5becd378f40a9be14e4e635d7034f2835fc7b5

Via local build
Last Good: db3a7c3208e7
First Bad: 7d23d87b02f8

	Alice0775 White
	Comment 2 • 8 years ago

Reproduced on Windows7 and Ubuntu14.04

	Kartikaya Gupta (email:kats@mozilla.staktrace.com)
	Comment 3 • 8 years ago

Also happens on OS X aurora and nightly. Doesn't happen if APZ is disabled.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com)
	Comment 6 • 8 years ago

This is a P1 blocking APZ rollout since we're generally recommending Keith's pure-CSS parallax approach as being APZ-friendly and it's currently broken because of this bug.

	Markus Stange [:mstange] Assignee
	Comment 8 • 8 years ago

I've decided to fix this in a very explicit way. The only "magic" part that's
left is how we decide that the AGR of the perspective item is outside the
scrolled frame (and I'm not sure myself how that works).

I didn't want to change what scroll clips we set on what items, because the
scroll clip really belongs on the perspective item, because that's the item
that needs to be clipped, and it should also be the item that should be
scrolled if it weren't for the fact that APZ wouldn't know that it should
apply the perspective transform before the APZ transform.

Review commit: https://reviewboard.mozilla.org/r/36833/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/36833/

	Markus Stange [:mstange] Assignee
	Comment 9 • 8 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a1ddbe6525b4

	Matt Woodrow (:mattwoodrow)
	Comment 10 • 8 years ago

Comment on attachment 8724047 [details]
MozReview Request: Bug 1247854 - Apply the correct scroll clips to the nsDisplayTransform and nsDisplayPerspective of a scrolled perspective item. r?mattwoodrow

https://reviewboard.mozilla.org/r/36833/#review33773

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

Tracking, regression. 
Markus, once this lands, want to request uplift?

	Markus Stange [:mstange] Assignee
	Comment 12 • 8 years ago

Yes, absolutely, this needs to be uplifted once it lands. It's currently waiting for the patches in bug 1238564 to be reviewed and landed.

	Pulsebot
	Comment 13 • 8 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/e0b2fffc1360

	Carsten Book [:Tomcat]
	Comment 14 • 8 years ago
bugherder

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

	Markus Stange [:mstange] Assignee
	Comment 15 • 8 years ago

Comment on attachment 8724047 [details]
MozReview Request: Bug 1247854 - Apply the correct scroll clips to the nsDisplayTransform and nsDisplayPerspective of a scrolled perspective item. r?mattwoodrow

Approval Request Comment
[Feature/regressing bug #]: APZ / bug 1147673
[User impact if declined]: scrolling impossible on some pages, and a certain parallax technique won't work properly
[Describe test coverage new/current, TreeHerder]: this patch adds a test
[Risks and why]: low
[String/UUID change made/needed]: none

This patch requires the changes in bug 1238564.

	Chris Peterson [:cpeterson]
	Comment 16 • 8 years ago

kats says we'll want this fix for the APZ experiment on Beta 46.

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

Comment on attachment 8724047 [details]
MozReview Request: Bug 1247854 - Apply the correct scroll clips to the nsDisplayTransform and nsDisplayPerspective of a scrolled perspective item. r?mattwoodrow

This sounds like a must fix, has been in Nightly for a week, taking it. Aurora47+, Beta46+

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

Julien, could you please verify this issue is fixed as expected on the latest Nightly build (03-10-2016 build on wards has the fix)? Thanks!

	Julien Wajsberg [:julienw] Reporter
	Comment 19 • 8 years ago

Yes this is fixed on Nightly :)

Thanks for the hard work !

	Carsten Book [:Tomcat]
	Comment 20 • 8 years ago

has conflicts uplifting to aurora:

grafting 332726:e0b2fffc1360 "Bug 1247854 - Apply the correct scroll clips to the nsDisplayTransform and nsDisplayPerspective of a scrolled perspective item. r=mattwoodrow"
merging layout/base/FrameLayerBuilder.cpp
merging layout/reftests/async-scrolling/reftest.list
warning: conflicts while merging layout/base/FrameLayerBuilder.cpp! (edit, then use 'hg resolve --mark')
warning: conflicts while merging layout/reftests/async-scrolling/reftest.list! (edit, then use 'hg resolve --mark')
abort: unresolved conflicts, can't continue
(use hg resolve and hg graft --continue)

	Markus Stange [:mstange] Assignee
	Comment 21 • 8 years ago

This patch requires the patches in bug 1238564. We can't uplift it until those patches are also uplifted.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com)
	Comment 22 • 8 years ago

Ritu, could you please also approve bug 1238564 for uplift?

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

wes, can you try to land this again after landing the uplifts in bug 1238564? Thanks!

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

Liz and I chatted about the risks involved in uplifting bug 1238564. The fix was deemed must-have for 46 and 47. She A+'d in the other bug.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com)
	Comment 25 • 8 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/c690a0802e00

	Carsten Book [:Tomcat]
	Comment 26 • 8 years ago

has problems uplifting to beta:

merging layout/base/FrameLayerBuilder.cpp
merging layout/base/FrameLayerBuilder.h
merging layout/reftests/async-scrolling/reftest.list
warning: conflicts while merging layout/base/FrameLayerBuilder.cpp! (edit, then use 'hg resolve --mark')
warning: conflicts while merging layout/reftests/async-scrolling/reftest.list! (edit, then use 'hg resolve --mark')
abort: unresolved conflicts, can't continue
(use hg resolve and hg graft --continue)

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 27 • 8 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/24b6aaac8973

	Petruta Horea [:phorea], Desktop QA
	Comment 28 • 8 years ago

Verified as fixed using Firefox 47 beta 1 (with e10s enabled) and latest Aurora 48.0a2.