Closed Bug 785 Opened 27 years ago Closed 26 years ago

{sink} Handling frameset's with bodies

Categories

(Core :: DOM: Core & HTML, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: troy, Assigned: harishd)

Details

When dealing with a framset document mBody will be NULL, but DidBuildModel() is called anyway. Down at the bottom of the function there's some code that informs the document that content has been appended: mDocument->ContentAppended(mBody, mBodyChildCount); I added a NULL pointer check around the call (and an additional call that I added, which had been crashing), but we probably shouldn't be calling this function at all for framesets
Status: NEW → ASSIGNED
Summary: Crash in DidBuildModel() → Handling frameset's with bodies
Setting all current Open/Normal to M4.
per leger, assigning QA contacts to all open bugs without QA contacts according to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
QA Contact: 4015 → 4141
QA contact re-assigned according to the product areas we're currently working on.
Summary: Handling frameset's with bodies → {sink} Handling frameset's with bodies
Assignee: kipp → harishd
Status: ASSIGNED → NEW
I think that this should be handled either in the parser or content-sink, and thats harishd so I'm reassigning...
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
The call to the DidBuildModel() is made by the parser. But, there is no reason why the parser should ignore that call for frameset documents. I think that this issue is totally related to the content sink. Since the sink, now, handles such calls, after troy's changes, I'm marking the bug fixed.
Status: RESOLVED → VERIFIED
Target Milestone: M15 → M7
veriified by developer.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.