519590 - Investigate not interrupting reflow until enough time has elapsed

Status: RESOLVED FIXED

(Core :: Layout, defect, P1)

Description

[Boris Zbarsky [:bzbarsky]]

Attachment: wip

Work-in-progress patch attached.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: With joel's help, a fix that actually works.

This does depend on bug 519590.  Only tested on Mac so far; will test elsewhere too.

Disabling interrupts in chrome was just something I thought we should do; please let me know if you think we shouldn't....

Comment 2

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 404111 [details] [diff] [review]
With joel's help, a fix that actually works.

Add "using mozilla::TimeStamp;", "using mozilla::TimeDuration;" to the top of nsPresContext.cpp instead of repeating the mozilla:: prefix?

Comment 3

[Boris Zbarsky [:bzbarsky]]

Pushed http://hg.mozilla.org/mozilla-central/rev/e9d59855daf9 with that change.

Comment 4

[Boris Zbarsky [:bzbarsky]]

I think this should be a beta blocker.  It's a change that we really need beta
testing on (and conversely, beta testing without this change is much less
useful).

Comment 5

[Boris Zbarsky [:bzbarsky]]

Pushed http://hg.mozilla.org/releases/mozilla-1.9.2/rev/33bedc4f97c0