327508 - Assertions on Startup NS_ASSERTION(xuldoc, "expected a XUL document");

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Scott MacGregor]

Fall out from Bug 285631.

When I start up Thunderbird I hit this new assertion several times in nsXULContentBuilder::HasGeneratedContent:

NS_DebugBreak_P(unsigned int 1, const char * 0x02565aa4, const char * 0x02565a9c, const char * 0x02565a48, int 1655) line 351 + 12 bytes
nsXULContentBuilder::HasGeneratedContent(nsXULContentBuilder * const 0x032ed558, nsIRDFResource * 0x02ef0018, nsIAtom * 0x00000000, int * 0x0012e46c) line 1655 + 43 bytes
nsContentTestNode::Constrain(InstantiationSet & {...}) line 110 + 37 bytes
TestNode::Constrain(InstantiationSet & {...}) line 403 + 21 bytes
nsXULTemplateQueryProcessorRDF::Propagate(nsIRDFResource * 0x02ef0018, nsIRDFResource * 0x019f0d68, nsIRDFNode * 0x036ce308) line 757 + 24 bytes
nsXULTemplateQueryProcessorRDF::OnAssert(nsXULTemplateQueryProcessorRDF * const 0x02efeea4, nsIRDFDataSource * 0x032edc30, nsIRDFResource * 0x02ef0018, nsIRDFResource * 0x019f0d68, nsIRDFNode * 0x036ce308) line 574
CompositeDataSourceImpl::OnAssert(CompositeDataSourceImpl * const 0x032edc34, nsIRDFDataSource * 0x032edef0, nsIRDFResource * 0x02ef0018, nsIRDFResource * 0x019f0d68, nsIRDFNode * 0x036ce308) line 1463
nsMsgRDFDataSource::assertEnumFunc(nsISupports * 0x032edc34, void * 0x0012e6d0) line 410
nsSupportsArray::EnumerateForwards(nsSupportsArray * const 0x032edf78, int (nsISupports *, void *)* 0x026b9870 nsMsgRDFDataSource::assertEnumFunc(nsISupports *, void *), void * 0x0012e6d0) line 627 + 20 bytes
nsMsgRDFDataSource::NotifyObservers(nsIRDFResource * 0x02ef0018, nsIRDFResource * 0x019f0d68, nsIRDFNode * 0x036ce308, nsIRDFNode * 0x00000000, int 1, int 0) line 394
nsMsgFolderDataSource::OnItemAddedOrRemoved(nsIRDFResource * 0x02ef0018, nsISupports * 0x036ce330, int 1) line 926
nsMsgFolderDataSource::OnItemAdded(nsMsgFolderDataSource * const 0x032edf20, nsIRDFResource * 0x02ef0018, nsISupports * 0x036ce330) line 911
nsMsgMailSession::OnItemAdded(nsMsgMailSession * const 0x02efec14, nsIRDFResource * 0x02ef0018, nsISupports * 0x036ce330) line 225
nsMsgDBFolder::NotifyItemAdded(nsMsgDBFolder * const 0x02ef0040, nsISupports * 0x036ce330) line 4620
nsMsgAccountManager::LoadVirtualFolders(nsMsgAccountManager * const 0x02f8b8a8) line 2926
X

Comment 1

[Neil Deakin]

I don't see any assertion on startup but I do see it when moving a saved search folder. Sounds like the root node for some template is being removed from the document. Might be related to bug 323988 which is still an issue with the new template builder but in different places.

Comment 2

[Neil Deakin]

I think this may be caused by the template for the Get Messages button, which is generated first on the toolbarpalette, and then the entire palette gets removed by the toolbar code leaving the template data hanging around.

Comment 3

[Neil Deakin]

This should be fixable just by removing the assertion and returning NS_OK, along with the other one in GetElementsForResult.