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

RESOLVED FIXED in mozilla1.8beta2

Status

()

P1
normal
RESOLVED FIXED
14 years ago
27 days ago

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Tracking

Trunk
mozilla1.8beta2
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
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.
(Assignee)

Comment 1

14 years ago
Created attachment 179107 [details] [diff] [review]
Proposed patch

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).
(Assignee)

Updated

14 years ago
Attachment #179107 - Flags: superreview?(roc)
Attachment #179107 - Flags: review?(roc)
(Assignee)

Updated

14 years ago
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+
(Assignee)

Comment 2

14 years ago
Fixed.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Component: Layout: View Rendering → Layout: Web Painting
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.