Layer invalidation issue with will-change: transform when used with menupopup
Categories
(Core :: Graphics, defect, P2)
Tracking
()
People
(Reporter: rowbot, Unassigned)
References
Details
(Keywords: regression)
Attachments
(3 files)
Using Windows 10 with the latest Nightly.
STR:
- Add the Bookmarks Menu icon to the url bar.
- Have enough bookmarks in the Bookmark Menu so that it is scrollable.
- Scroll up and down.
AR:
The Bookmarks Menu sometimes overflows its box, by ~20px, while scrolling. Moving the mouse will fix it as it presumably causes the browser to redraw the menu.
ER:
The Bookmarks Menu should not overflow.
I think this is a gfx issue since moving the mouse fixes it, but I suppose it could be a widget or css issue as well. I have tried turning off OMTP, HW acceleration, Webrender, Advanced Layers, and the GPU process. Kind of out of ideas now. Ran mozregression real quick and I can reproduce this issue all the way back to January 2018, so it doesn't seem new, though I don't really recall noticing it before. I'll see about going back further in mozregression.
Reporter | ||
Comment 1•6 years ago
|
||
Unfortunately, mozregression couldn't narrow it down any further.
Last good: 2018-01-17
First bad: 2018-01-18
Push Log: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=b7a651281314d6369658eeb58e3bb181cf95016f&tochange=d4d53f4069f43eab1aae8bcf4def08bdb5a84d8d
Reporter | ||
Comment 2•6 years ago
|
||
After poking at this a little bit, I've found that removing the will-change: transform, opacity;
property from the #BMB_bookmarksPopup
element fixes this glitch. Narrowing it down more, only having will-change: transform;
present causes the glitch to reappear.
Bug 1425213 was in the regression range and appears to be the only change related to transforms. :hiro, are you able to tell if this is caused by Bug 1425213?
Comment 3•6 years ago
|
||
I am not sure. The fact that removing will-change: transform
fixes the issue sounds to me that bug 1425213 is unrelated. The will-change
style there refers to bug 1414033 it's fixed very recently so mattwoodrow knows something?
Updated•5 years ago
|
Reporter | ||
Comment 4•5 years ago
|
||
In the video, if you watch the right side of the bookmarks menu, you can see it keeps overflowing when scrolling.
Comment 5•5 years ago
|
||
I'm not entirely sure, it appears we have a bug somewhere with will-change:transform.
Bug 1543508 is probably going to remove this usage of it, so that might be a good fix in the short term.
Updated•5 years ago
|
Updated•5 years ago
|
Comment 6•5 years ago
|
||
Hi Gijs,
Wondering when you think you'll be able to fix Bug 1543508? (seems this one depends on it).
[regression triage for 68]
Thanks,
-Patricia
Comment 7•5 years ago
|
||
I've added a patch on the other bug. Is there any reason not to just mark this as a dupe of 1543508? It looks like exactly the same issue to me.
Reporter | ||
Comment 8•5 years ago
|
||
I will turn this bug into a bug for investigating the underlying issue with will-change: transform
since Matt pointed out that Bug 1543508 is only papering over the issue.
Reporter | ||
Comment 9•5 years ago
|
||
In this image, I have removed the -moz-appearance: none
css attribute from #BMB_bookmarksPopup
, which causes the popup to have the -moz-appearance: menupopup
css attribute. We can see that the "overflow" is filling up the space between the edge of the bookmarks menu to the edge of the menupopup element. Not really sure if I should poke someone from gfx, css, or dom about this.
Updated•5 years ago
|
Updated•2 years ago
|
Description
•