Closed
Bug 396099
Opened 17 years ago
Closed 17 years ago
[FIX]Simplify reflow/restyle processing a bit
Categories
(Core :: Layout, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla1.9beta1
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
(Depends on 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
Details | Diff | Splinter Review |
Now that we always flush content in nsPresShell::DoFlushPendingNotifications, we can remove the content flushing in ProcessReflowCommands (which is only called from DoFlushPendingNotifications) and can use FlushPendingNotifications() to handle restyle events in the frame constructor and remove the custom code that lives there now.
This depends on bug 387669 because without that flushing content doesn't always work. It will also need merging once bug 394014 lands.
Attachment #280809 -
Flags: superreview?(roc)
Attachment #280809 -
Flags: review?(roc)
Attachment #280809 -
Flags: approval1.9?
|
Assignee | |
Updated•17 years ago
|
Assignee: nobody → bzbarsky
Priority: -- → P2
Summary: Simplify reflow/restyle processing a bit → [FIX]Simplify reflow/restyle processing a bit
Target Milestone: --- → mozilla1.9 M9
Attachment #280809 -
Flags: superreview?(roc)
Attachment #280809 -
Flags: superreview+
Attachment #280809 -
Flags: review?(roc)
Attachment #280809 -
Flags: review+
Attachment #280809 -
Flags: approval1.9?
Attachment #280809 -
Flags: approval1.9+
|
|
Assignee | |
Comment 1•17 years ago
|
||
Attachment #280809 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 2•17 years ago
|
||
Checked in. There should be no behavior change, so no test.
Status: NEW → RESOLVED
Closed: 17 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•