Closed Bug 360422 Opened 15 years ago Closed 13 years ago

Remove nsTableOuterFrame::IsNested once reflow branch lands

Categories

(Core :: Layout: Tables, defect)

Product:
Core
Component:
Layout: Tables
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: bernd_mozilla)

References

Details

(Whiteboard: [reflow-refactor])

Attachments

(1 file)

patch
2.19 KB, patch
bzbarsky
: review+
bzbarsky
: superreview+
Details | Diff | Splinter Review 
The only place IsNested() is used is:

1258   // If mComputedWidth > availWidth and availWidth >= minWidth for a nested percent table 
1259   // then adjust mComputedWidth based on availableWidth if this isn't the initial reflow   

and since reflow branch handles this differently, we can remove the function once it lands.
Whiteboard: [reflow-refactor]
Assignee: nobody → bernd_mozilla
Attached patch patch — — Splinter Review
Attachment #343809 - Flags: superreview?(bzbarsky)
Attachment #343809 - Flags: review?(bzbarsky)
Comment on attachment 343809 [details] [diff] [review]
patch

Yay code-be-gone.
Attachment #343809 - Flags: superreview?(bzbarsky)
Attachment #343809 - Flags: superreview+
Attachment #343809 - Flags: review?(bzbarsky)
Attachment #343809 - Flags: review+
Depends on: reflow-refactor
http://hg.mozilla.org/mozilla-central/rev/b7385c2a17eb
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.