Closed Bug 3305 Opened 26 years ago Closed 25 years ago

[CODEBUG] Multiple calls to SetInitialChildList()

Categories

(Core :: Layout: Tables, defect, P2)

Product:
Core
Component:
Layout: Tables
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: troy, Assigned: karnaze)

Details

(Whiteboard: 2/21: 2nd request for verification by reporter or assigned engineer)

QA: Low-level code bug, no need to verify. There are two problems with the frame construction code in ConstructTableFrame(): 1. it makes more than one call to SetInitialChildList() for a given frame and child list name 2. SetInitialChildList() is being called BEFORE processing child frames, which is wrong because Init() is top-down and SetInitialChildList() is bottom-up The code is marked with an XXX. See the nsIFrame header for details
Status: NEW → ASSIGNED
Target Milestone: M8
Moving to M8.
Moving to M10.
Summary: Multiple calls to SetInitialChildList() → [CODEBUG] Multiple calls to SetInitialChildList()
Whiteboard: [NOTESTCASENEEDED]
Marked [CODEBUG] and [NOTESTCASENEEDED].
Target Milestone: M10 → M12
Moving to M12.
mass move to m14.
Changing [NOTESTCASENEEDED] code to new donttest keyword. Sorry for the spam!
Keywords: donttest
Whiteboard: [NOTESTCASENEEDED]
Troy, this should be fixed now. I commented places that could result in multiple calls when the anonymous frame bug (the one where siblings don't share anonymous parent) get fixed.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Troy or ChrisK: Please mark this as verified fixed if you agree.
Whiteboard: 2/4: Requested verification by reporter or assigned engineer
Second request for verification by assigned engineer or reporter, please.
Whiteboard: 2/4: Requested verification by reporter or assigned engineer → 2/21: 2nd request for verification by reporter or assigned engineer
No response to 2nd request for verification by reporter or assigned engineer. Will mark verified. Please reopen if there is a problem.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.