Consider using idle queue for RefreshDriver ticks after first contentful paint but before load

RESOLVED FIXED in Firefox 67

Status

()

enhancement
P3
normal
RESOLVED FIXED
6 months ago
2 months ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

(Blocks 1 bug)

unspecified
mozilla67
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

6 months ago
Some work was happening in 
https://bugzilla.mozilla.org/show_bug.cgi?id=1503339
(Assignee)

Comment 1

6 months ago
And if we do this, perhaps we could remove nglayout.initialpaint.delay
Priority: -- → P3
(Assignee)

Updated

6 months ago
(Assignee)

Updated

4 months ago
Summary: Consider using idle queue for RefreshDriver ticks after first non-blank-paint but before load → Consider using idle queue for RefreshDriver ticks after first contentful paint but before load
(Assignee)

Comment 2

4 months ago
Posted patch slower_load_vsync_3.diff (obsolete) — Splinter Review
(Assignee)

Comment 3

4 months ago

This is quite similar to bug 1339135.

This is also related to bug 1517850, which tries to ensure we get first contentful paint sooner, but after that we should slow down painting rate.

I don't have yet too good numbers, but at least in performance profiles we end up spending less time reflowing/painting the page, which should lead to faster page loads.

Attachment #9037285 - Attachment is obsolete: true
Attachment #9037810 - Flags: review?(afarre)
Comment on attachment 9037810 [details] [diff] [review]
slower_load_vsync_4.diff

Review of attachment 9037810 [details] [diff] [review]:
-----------------------------------------------------------------

Discussed a bit about if prefs would be helpful or not on irc. I'm ok without.
Attachment #9037810 - Flags: review?(afarre) → review+
(Assignee)

Comment 6

4 months ago

This might even build.

Comment 7

4 months ago
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/840dbcc8d159
use idle queue for RefreshDriver tick betweek fcp and load, r=farre

Comment 8

4 months ago
bugherder
Status: NEW → RESOLVED
Last Resolved: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Depends on: 1524207
You need to log in before you can comment on or make changes to this bug.