scrollbar layout happens after onload

RESOLVED FIXED

Status

()

Core
Layout
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: Robert Sayre, Assigned: roc)

Tracking

Trunk
x86
Linux
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
Bug 363253 made scrollbar layout happen in a callback, and the linux perf box, this resulted in txul times decreasing by a large percentage. On, IRC bz wondered whether making scrollbar munging happen after onload is desirable.
(Reporter)

Updated

11 years ago
Blocks: 363253
So the point is that DoneRemovingReflowRoots() is what fires onload and this comes before reflow callbacks.

Perhaps we should reverse the ordering of those two?
Flags: blocking1.9?
Yeah, I think we should.
Created attachment 257416 [details] [diff] [review]
fix

patch to do just that
Assignee: nobody → roc
Status: NEW → ASSIGNED
Attachment #257416 - Flags: superreview?(bzbarsky)
Attachment #257416 - Flags: review?(bzbarsky)
Comment on attachment 257416 [details] [diff] [review]
fix

Let's try it.
Attachment #257416 - Flags: superreview?(bzbarsky)
Attachment #257416 - Flags: superreview+
Attachment #257416 - Flags: review?(bzbarsky)
Attachment #257416 - Flags: review+
Checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Flags: blocking1.9? → in-testsuite-
Resolution: --- → FIXED
Hmm.  This had no effect on Txul... I wonder why bug 363253 made Txul go down, then...
This is not needed on branch.
Would this be needed once bug 267833 lands?
Flags: blocking1.8.1.8?
No.  Not unless bug 363253 lands on branch.
Thanks. We don't have any plans to backport bug 363253, but we do want bug 267833 on the old branch. Clearing blocking1.8.1.8 nomination.
Flags: blocking1.8.1.8?
You need to log in before you can comment on or make changes to this bug.