Closed
Bug 786387
Opened 14 years ago
Closed 14 years ago
Add comments to document the subtleties of nsFrame::Reflow and better document the main dirty frame bits
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla18
People
(Reporter: jwatt, Assigned: jwatt)
Details
Attachments
(1 file)
|
5.23 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
Spinning this out from bug 734751. I found bz and dbaron's clarifications about how reflow and dirty bits work (both in that bug and on IRC) to be very helpful. This bug is for adding those insights to the code in the form of documentation comments.
|
Assignee | |
Comment 1•14 years ago
|
||
Attachment #656123 -
Flags: review?(bzbarsky)
|
|
Assignee | |
Comment 2•14 years ago
|
||
Probably I should s/even if no dirty bits are set/even if no dirty bits are set on the children/
|
|
||
Comment 3•14 years ago
|
||
Comment on attachment 656123 [details] [diff] [review]
patch
r=me
Attachment #656123 -
Flags: review?(bzbarsky) → review+
|
|
Assignee | |
Comment 4•14 years ago
|
||
Flags: in-testsuite-
Target Milestone: --- → mozilla18
|
|
Assignee | |
Comment 5•14 years ago
|
||
(In reply to Jonathan Watt [:jwatt] from comment #2)
> Probably I should s/even if no dirty bits are set/even if no dirty bits are
> set on the children/
Note I tweaked this part of the text for clarification, but not quite as simply as suggested in comment 2.
|
||
Comment 6•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•