Closed
Bug 1451168
Opened 6 years ago
Closed 6 years ago
No scrollbar thumb is displayed
Categories
(Core :: Graphics: WebRender, defect, P1)
Tracking
()
VERIFIED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox-esr60 | --- | unaffected |
firefox59 | --- | unaffected |
firefox60 | --- | unaffected |
firefox61 | --- | verified |
firefox62 | --- | verified |
People
(Reporter: alice0775, Assigned: kats)
References
(Blocks 1 open bug, )
Details
(Keywords: regression)
Attachments
(5 files)
Reproducible: always Steps To Reproduce: 1. Make sure that WebRender is enabled 2. Open https://www.mozilla.org/en-US/privacy/firefox/ 3. Open 3 dots menu at the top-left of the page 4. Expand sub menu so that the left sidebar will overflow or reduce height of browser Actual Results: No scrollbar thumb is displayed
Reporter | ||
Comment 1•6 years ago
|
||
Reporter | ||
Comment 2•6 years ago
|
||
Regression window: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=7a6d3d03bb20c881d25ae7e6fdb2fbb981e335b7&tochange=cf1fdb432d46e52f795b3e253e851fbf775b2af1 Regressed by:cf1fdb432d46 Kartikaya Gupta — Bug 1423370 - Create fewer WebRenderLayerScrollData items for transformed items. r=jrmuizel
Blocks: 1423370
Keywords: regression
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → bugmail
Updated•6 years ago
|
Assignee | ||
Comment 3•6 years ago
|
||
Why did you mark 59 and 60 affected?
Flags: needinfo?(stefan.georgiev)
Reporter | ||
Comment 4•6 years ago
|
||
59,60,61 are not affected by default. The bug is reproduced if WebRender is enabled.
Comment 5•6 years ago
|
||
I thought I can see the same behavior there as in the attached screenshot. However on more closer look I can actually see that scrollbar thumb is displayed. Sorry about that.
Flags: needinfo?(stefan.georgiev)
Updated•6 years ago
|
Blocks: stage-wr-nightly
Priority: -- → P1
Updated•6 years ago
|
Blocks: webrender-site-issues
OS: Windows 10 → All
Assignee | ||
Comment 7•6 years ago
|
||
Assignee | ||
Comment 8•6 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9565f1fbc0c6fedb2a73e455f0d93596b3318bfd The patch in this try push fixes it for me locally, but it seems to fail other reftests, so it's probably not right. It also doesn't fix bug 1450871 so I'll look at that one in more detail to see if I can figure out what the right fix is.
Assignee | ||
Comment 9•6 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=46838eece07a7776f3b9485eedef1ca69dc3181e This one should be better. Also fixes bug 1450871 for me locally.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 16•6 years ago
|
||
Sorry for the churn, I uploaded an old version (without comments) first and then I forgot to fix the commit message...
Comment 17•6 years ago
|
||
mozreview-review |
Comment on attachment 8972415 [details] Bug 1451168 - Add a reftest. https://reviewboard.mozilla.org/r/241026/#review246810 ::: gfx/tests/reftest/1451168-ref.html:9 (Diff revision 3) > + <style type="text/css"> > + > +nav{display:block} > +body,html{margin:0;padding:0} > +html{font-size:112.5%;background:#fff} > +.moz-global-nav-drawer{font-family:"Open Sans",X-LocaleSpecific,sans-serif;background:#000;bottom:0;color:#fff;padding:20px;position:absolute;top:0;width:280px;z-index:1000} Do we need font-family:"Open Sans"?
Assignee | ||
Comment 18•6 years ago
|
||
No. I didn't bother reducing the CSS to the absolute minimum possible although I guess I probably should.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 21•6 years ago
|
||
mozreview-review |
Comment on attachment 8972414 [details] Bug 1451168 - Ensure the transforms from ancestor stacking contexts are combined properly. https://reviewboard.mozilla.org/r/241024/#review246820
Attachment #8972414 -
Flags: review?(jmuizelaar) → review+
Comment 22•6 years ago
|
||
mozreview-review |
Comment on attachment 8972415 [details] Bug 1451168 - Add a reftest. https://reviewboard.mozilla.org/r/241026/#review246966
Attachment #8972415 -
Flags: review?(jmuizelaar) → review+
Comment 23•6 years ago
|
||
Pushed by kgupta@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d9b0f9b3bc39 Ensure the transforms from ancestor stacking contexts are combined properly. r=jrmuizel https://hg.mozilla.org/integration/autoland/rev/4b8c19afd14e Add a reftest. r=jrmuizel
Comment 24•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d9b0f9b3bc39 https://hg.mozilla.org/mozilla-central/rev/4b8c19afd14e
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Updated•6 years ago
|
status-firefox59:
--- → unaffected
status-firefox60:
--- → unaffected
status-firefox-esr52:
--- → unaffected
status-firefox-esr60:
--- → unaffected
Updated•6 years ago
|
Flags: qe-verify+
Comment 25•6 years ago
|
||
I’ve reproduced this issue on Firefox 61.0a1 (2018-04-03) under Windows 10 x86. The issue is fixed on Firefox 62.0a1 (2018-05-21) and Firefox 61.0b7 (20180521110957) under Windows 10 x86, Ubuntu 16.04 x64 and macOS 10.13.
You need to log in
before you can comment on or make changes to this bug.
Description
•