Closed
Bug 1479201
Opened 6 years ago
Closed 5 years ago
HTML page full of emojis scrolls poorly
Categories
(Core :: Graphics: WebRender, defect, P3)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mayankleoboy1, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: perf)
Attachments
(1 file)
(deleted),
text/html
|
Details |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:63.0) Gecko/20100101 Firefox/63.0
Build ID: 20180728101501
Steps to reproduce:
create new profile. Enable WR
Open https://bugzilla.mozilla.org/attachment.cgi?id=8995714
scroll with the mouse
Actual results:
poor janky scrolling
https://perfht.ml/2NSsPvP
Expected results:
not so
non-WR is pretty smooth
Reporter | ||
Updated•6 years ago
|
Component: Untriaged → Graphics: WebRender
Product: Firefox → Core
Reporter | ||
Updated•6 years ago
|
Comment 2•6 years ago
|
||
confirmed
Comment 3•6 years ago
|
||
From the profile it looks like the renderer thread is swamped. So bug 1480317 would help.
Depends on: 1480317
Updated•6 years ago
|
Priority: -- → P3
Reporter | ||
Comment 5•6 years ago
|
||
the testcase has started to act a bit weird. Now it takes 2GB memory to display with tons of CPU
Reporter | ||
Comment 6•6 years ago
|
||
It now scrolls better than what I remember it used to : https://perfht.ml/2L7Knnb
Comment 7•6 years ago
|
||
The new profile in comment 6 shows better behaviour than before, but still shows long composites (upwards of 60ms). 3% of time on the renderer thread is in NtQueryLicenseValue much like your other profiles. But this time 7% is in draw_elements_instanced so there might be something we can improve here.
Reporter | ||
Comment 8•6 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #7)
>3% of time on the renderer thread is in NtQueryLicenseValue much like your other profiles. But this time >7% is in draw_elements_instanced so there might be something we can improve here.
Anecdotally, the last round of reducing mem_cpy work on the WR side reduced the time spent in NtQueryLicenseValue in general on all pages.
Reporter | ||
Comment 9•6 years ago
|
||
This is a UTF encoded emoji page, that will show actual emojis in proper rows and columns.
This still scrolls with jank and checkerboard with the latest patch from bug 1493353 applied. Bug 1493353 did fix the original testcase from #comment 0
Reporter | ||
Comment 11•6 years ago
|
||
Reporter | ||
Comment 12•6 years ago
|
||
of course, feel free to WONTFIX this, as its terribly artificail.
Flags: needinfo?(mayankleoboy1)
Reporter | ||
Comment 13•6 years ago
|
||
heres a profile I took on a slow network. Which means the page took over 20 seconds to load on the network.
Reporter | ||
Comment 14•5 years ago
|
||
Marking this as resolved fixed. as the testcase doesnt jank at all while scrolling. It does turn white, but that is the general "make painting fast" bug.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•