Closed Bug 288365 Opened 19 years ago Closed 19 years ago

[FIX]No more need to defer the view update batch in ApplyRenderingChangeToTree

Categories

(Core :: Web Painting, defect, P1)

x86
Linux
defect

Tracking

()

RESOLVED FIXED
mozilla1.8beta2

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Details

Attachments

(1 file)

Since finishing up a view update batch will now generally process reflows if
there were invalidates during the batch (with bug 244366 fixed), there is no
need to do a deferred view batch in ApplyRenderingChangeToTree.  Instead we can
just wrap a view update batch around all the restyle processing.
Attached patch Proposed patchSplinter Review
I left the batch in ApplyRenderingChangeToTree because that's also called by
nsITheme changes and by some content modifications (ones that affect the canvas
background, basically).
Attachment #179107 - Flags: superreview?(roc)
Attachment #179107 - Flags: review?(roc)
Priority: -- → P1
Summary: No more need to defer the view update batch in ApplyRenderingChangeToTree → [FIX]No more need to defer the view update batch in ApplyRenderingChangeToTree
Target Milestone: --- → mozilla1.8beta2
Attachment #179107 - Flags: superreview?(roc)
Attachment #179107 - Flags: superreview+
Attachment #179107 - Flags: review?(roc)
Attachment #179107 - Flags: review+
Fixed.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Component: Layout: View Rendering → Layout: Web Painting
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: