HTML page full of emojis scrolls poorly

NEW
Unassigned

Status

()

defect
P3
normal
10 months ago
4 months ago

People

(Reporter: mayankleoboy1, Unassigned)

Tracking

(Blocks 1 bug, {perf})

63 Branch
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Reporter

Description

10 months ago
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

10 months ago
Component: Untriaged → Graphics: WebRender
Product: Firefox → Core
Reporter

Comment 1

10 months ago
see also: bug 1479196
Has STR: --- → yes
Reporter

Updated

10 months ago
See Also: → 1479196

Comment 2

10 months ago
confirmed
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: perf
From the profile it looks like the renderer thread is swamped. So bug 1480317 would help.
Depends on: 1480317
Priority: -- → P3
I don't think this needs to block release
Reporter

Comment 5

8 months 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 months ago
It now scrolls better than what I remember it used to : https://perfht.ml/2L7Knnb
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

5 months 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

Updated

5 months ago
Depends on: 1493353
Reporter

Comment 9

5 months ago
Posted file emoji_working.html
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
Mayank, can you post of a profile?
Flags: needinfo?(mayankleoboy1)
Reporter

Comment 12

5 months ago
of course, feel free to WONTFIX this, as its terribly artificail.
Flags: needinfo?(mayankleoboy1)
Reporter

Comment 13

4 months ago

heres a profile I took on a slow network. Which means the page took over 20 seconds to load on the network.

https://perfht.ml/2FueSTP

You need to log in before you can comment on or make changes to this bug.