785 - {sink} Handling frameset's with bodies

Status: VERIFIED FIXED

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

Description

[troy]

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

Comment 1

[leger]

Setting all current Open/Normal to M4.

Comment 2

[Paul MacQuiddy]

per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 3

[gerardok]

QA contact re-assigned according to the product areas we're currently working
on.

Comment 4

[kipp]

I think that this should be handled either in the parser or content-sink, and
thats harishd so I'm reassigning...

Comment 5

[harishd]

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.

Comment 6

[gem]

veriified by developer.