1496416 - Elements move across page when scrolling with mouse wheel

Status: RESOLVED FIXED

(Core :: Graphics: WebRender, defect, P1)

Description

[TMart]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:64.0) Gecko/20100101 Firefox/64.0

Steps to reproduce:

Visit 
https://www.golem.de/news/imho-unnoetige-sicherheitsrisiken-mit-linux-1810-136892.html

Start scrolling down with the mouse wheel. 



Actual results:

The copyright notice on the right of the picture of the Linux penguin moves across the page to the left while scrolling, until it disappears from the viewport.

If you scroll up again it stays hidden. After doing a tab switch, the element is there again. Using the arrow keys doesn't move the element.

Comment 1

[TMart]

Another example: https://www.camp-firefox.de/forum/viewforum.php?f=1

The arrow icon of the highlighted box moves across the page: 
https://abload.de/img/unbenanntkuesy.png

Comment 2

[Darkspirit]

mozregression --good 2018-10-03 --bad 2018-10-04 --pref gfx.webrender.all:true -a https://www.golem.de/news/imho-unnoetige-sicherheitsrisiken-mit-linux-1810-136892.html
> 4:19.75 INFO: Last good revision: 831c1cae9dde0bd0f2eff1023fbe80afef8d8e8d
> 4:19.75 INFO: First bad revision: 4526e08d44775dd3c32ce27c16d96e4b995eea60
> 4:19.75 INFO: Pushlog:
> https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=831c1cae9dde0bd0f2eff1023fbe80afef8d8e8d&tochange=4526e08d44775dd3c32ce27c16d96e4b995eea60

> 4526e08d4477	Jeff Muizelaar — Bug 1495228. Re-generate FFI header
> 3e45393406be	Jeff Muizelaar — Bug 1495228 - Api change from PR#3147.
> 1eb163b766fd	Kartikaya Gupta — Bug 1415272 - Enable perspective-scrolling-3 as it passes with WR PR 3154. r=jrmuizel
> a351e7a11b2c	Jeff Muizelaar — Bug 1495228. Update webrender to commit 9536249e3ed920a920346f6cc0a79473cad16099

Regression range: https://github.com/servo/webrender/compare/d7a6d081384ce0da9dd359b0cf4b9f758aab1b67...9536249e3ed920a920346f6cc0a79473cad16099

https://github.com/servo/webrender/pull/3145#issuecomment-425709527
mozregression --repo try --launch a03726cfc8516204b07957a0a38e96d84e1a48f4 --pref gfx.webrender.all:true -a https://www.golem.de/news/imho-unnoetige-sicherheitsrisiken-mit-linux-1810-136892.html
good

https://github.com/servo/webrender/pull/3149#issuecomment-426084544
mozregression --repo try --launch 79ebd8293327b257b8dfeab4696eb699499c88ce --pref gfx.webrender.all:true -a https://www.golem.de/news/imho-unnoetige-sicherheitsrisiken-mit-linux-1810-136892.html
good

https://github.com/servo/webrender/pull/3153#issuecomment-426264904
mozregression --repo try --launch a9456f153a36d064d9ef0a163b7814b3d3adce63 --pref gfx.webrender.all:true -a https://www.golem.de/news/imho-unnoetige-sicherheitsrisiken-mit-linux-1810-136892.html
good

https://github.com/servo/webrender/pull/3154#issuecomment-426325620
mozregression --repo try --launch fd156919846cdc951c47d94e2c8df7b0df64fa47 --pref gfx.webrender.all:true -a https://www.golem.de/news/imho-unnoetige-sicherheitsrisiken-mit-linux-1810-136892.html
bad

Likely regressed by servo/webrender#3154.

Comment 5

[Aria Beingessner [:Gankra]]

twitter video play buttons also have this problem

Comment 6

[Kartikaya Gupta (email:[e-mail])]

I'll see if a backout of servo/webrender#3154 fixes this.

Comment 7

[Darkspirit]

https://www.youtube.com/watch?v=cRSO3RtUOOk Thumbs up/down and verified account badges.

Comment 8

[Darkspirit]

Yes, the backout worked. https://treeherder.mozilla.org/#/jobs?repo=try&revision=746058eee0895ccfaa511c986af1ee60cc51f14c

Comment 9

[Kartikaya Gupta (email:[e-mail])]

I landed the backout on mozilla-inbound directly: https://hg.mozilla.org/integration/mozilla-inbound/rev/4bcce93eae6401679746dda3cd7c192aba62f4f5

I'll use this bug to land a reftest to prevent this from happening again.

Comment 10

[Darkspirit]

https://data.firefox.com/dashboard/usage-behavior Scroll down without hovering the diagrams.

Comment 11

[Kartikaya Gupta (email:[e-mail])]

Yeah it's pretty much anything with a 2D non-translation transform (i.e. rotation or skew).

Try push with a reftest: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d6e6b8b9a2c8656d32631e3cc2e6b9ab91be0e30

Comment 13

[Kartikaya Gupta (email:[e-mail])]

Attachment: Reftest

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/52029571d897
Add a reftest for async-scrolling content with 2D rotation transforms. r=mstange

Comment 15

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/52029571d897