Closed Bug 73188 Opened 24 years ago Closed 15 years ago

We may crash during printing because nsHTMLContainerFrame::ReparentViewTo does not cycle through all of the frame's childlists

Categories

(Core :: Printing: Output, defect)

Product:
Core
Component:
Printing: Output
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WORKSFORME
Milestone:
Future

People

(Reporter: kmcclusk, Unassigned)

References

Details

(Keywords: topembed-, Whiteboard: [T2])

nsHTMLContainerFrame::ReparentView to only looks at the normal frame list, overflow list and floater list when views are reparented. There may be frames other childlists with views that need to be reparented. I don't think we ever want to move a frame in the normal frame lists or any of the additional frame lists without moving their views. If we don't move the views then the frame tree and the view tree will be inconsistent. However, when I applied patch 28544 in bug 71032 which cycles through all of the frame lists, we crash while printing www.icq.com. (Possibly caused by a combobox? - since they do not put their views in the normal place in the view tree).
Target Milestone: --- → Future
Status: NEW → ASSIGNED
Blocks: 64841
Keywords: topembed
topembed-. There aren't any known crashers related to this issue.
Keywords: topembed → topembed-
Corfirming topembed- [T2] per EDT triage.
Whiteboard: [T2]
Assignee: kmcclusk → nobody
Status: ASSIGNED → NEW
QA Contact: sujay → printing
Since nsHTMLContainerFrame::ReparentView no longer exists, I'm guessing this isn't valid any more.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.