1648282 - button with opacity has jumpy scrolling behavior

Status: RESOLVED FIXED

(Core :: Web Painting, defect, P3)

Description

[Mark Bokil]

Attachment: scroll-opacity-bug.png

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0) Gecko/20100101 Firefox/78.0

Steps to reproduce:

Create a data grid with rows using display:flex. Place a button inside the grid with an opacity of 0.6. Scroll the DIV so it overflows. I was able to fix the problem by adding position:relative to my buttons. I prepared a test case for analysis. Another aspect to this bug is opening the inspector console appears to fix the jumpy scrolling. So test the bug first with all consoles closed. http://markbokil.com/private/cs-vue-data-grid-bug/

Actual results:

The scrolling is choppy and can cause elements in rows to visually overlap. The scrolling row elements appear to lag while scrolling.

Expected results:

Scrolling of the elements in the row should have been smooth

Comment 1

[Alice0775 White]

I can reproduce the issue on Nihjtly79.0a1(20200624215010) Windows10 if WebRender is disabled.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=358ec67834f6960938e60ced3018626f0d0cfd43&tochange=5417b73364be93e83abf43ec1993d4da78396a83

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:mattwoodrow, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 3

[Miko Mynttinen]

Attachment: 1648282.html

Comment 4

[Miko Mynttinen]

Attachment: Bug 1648282 - Create active layers for container items that have child items with different ASRs, similar to AGRs r=mstange

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7de711320e1e
Create active layers for container items that have child items with different ASRs, similar to AGRs r=mstange

Comment 6

[Bogdan Tara[:bogdan_tara | bogdant]]

Backed out changeset 7de711320e1e (bug 1648282) for reftests/bugs/1648282-1 failures.

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&selectedTaskRun=ZymNcvf5Q166p8KiFpxx9A.0&searchStr=linux%2C18.04%2Cx64%2Casan%2Copt%2Creftests%2Ctest-linux1804-64-asan%2Fopt-reftest-no-accel-e10s%2Cru8&tochange=c1469386234d8a60423e757624276c2bc1be97d1&fromchange=4b99cd59c105d2ef7947a7654f104d30994eab75

Backout link: https://hg.mozilla.org/integration/autoland/rev/c1469386234d8a60423e757624276c2bc1be97d1

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=318128906&repo=autoland&lineNumber=11880

[task 2020-10-09T03:05:10.111Z] 03:05:10     INFO - REFTEST TEST-START | layout/reftests/bugs/1648282-1a.html == layout/reftests/bugs/1648282-1-ref.html
[task 2020-10-09T03:05:10.111Z] 03:05:10     INFO - REFTEST TEST-LOAD | file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/bugs/1648282-1a.html | 2054 / 2056 (99%)
[task 2020-10-09T03:05:10.183Z] 03:05:10     INFO - REFTEST TEST-LOAD | file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/bugs/1648282-1-ref.html | 2054 / 2056 (99%)
[task 2020-10-09T03:05:10.261Z] 03:05:10     INFO - REFTEST INFO | REFTEST fuzzy test (0, 0) <= (1, 74664) <= (1, 70331)
[task 2020-10-09T03:05:10.504Z] 03:05:10     INFO - REFTEST TEST-UNEXPECTED-FAIL | layout/reftests/bugs/1648282-1a.html == layout/reftests/bugs/1648282-1-ref.html | image comparison, max difference: 1, number of differing pixels: 74664
[task 2020-10-09T03:05:10.504Z] 03:05:10     INFO - REFTEST   IMAGE 1 (TEST): data:image/png;base64,iVBORw0KGgoAAAANSUhEU....
[task 2020-10-09T03:05:10.504Z] 03:05:10     INFO - REFTEST   IMAGE 2 (REFERENCE): data:image/png;base64,iVBORw0KGgoAAAANSUhEU....
[task 2020-10-09T03:05:10.506Z] 03:05:10     INFO - REFTEST TEST-END | layout/reftests/bugs/1648282-1a.html == layout/reftests/bugs/1648282-1-ref.html

Comment 7

[Miko Mynttinen]

The failure was due to reftest requiring more fuzziness with basic compositor.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/65a85eec3a56
Create active layers for container items that have child items with different ASRs, similar to AGRs r=mstange

Comment 9

[Razvan Maries]

Backed out for perma failures.

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception&searchStr=windows%2C7%2Copt%2Creftests%2Ctest-windows7-32%2Fopt-reftest-e10s%2Cr2&revision=7c5864d62b1356bb4271e7bc3f8424ced44c6732&selectedTaskRun=EgI7pRyATkyEWxVQ2zgssQ.0

Log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=318191543&repo=autoland&lineNumber=22164

Backout: https://hg.mozilla.org/integration/autoland/rev/4dbac61081d24d30cb046e7a200774ccc161e611

Comment 10

[Miko Mynttinen]

Seems that Windows 7 wants its own fuzziness too. I should have ran reftests on all platforms from the beginning.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/466c14dd254e
Create active layers for container items that have child items with different ASRs, similar to AGRs r=mstange

Comment 12

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/466c14dd254e