73953 - Using async RDF in XUL template doesn't work properly

Status: VERIFIED FIXED

(Core Graveyard :: RDF, defect, P3)

Description

[[e-mail]]

Using Mozilla 0.8.1 Build ID: 2001032604
Steps to reproduce problem:
1. Add the following lines to mailWindowOverlay.xul:
2. <?xul-overlay href="chrome://messenger/content/msgFolderPickerOverlay.xul"?>
3. <box autostretch="never"><menulist id="searchableFolders"/></box>

The effect is different depending on where you put the menulist; in the menu bar
the menulist won't drop at all; in the toolbar all the items are duplicated.

The problem does not occur in the Modern theme.

Comment 1

[[e-mail]]

Sorry, original report was wrong, closing and opening Mail "fixed" the problem.

So now the actual problem is this:
If you add searchableFolders to the mail window they don't work on the first open.
They do work on subsequent opens.

I'm not sure which component to file this against so I've picked RDF.

This all worked fine in Mozilla 0.8

Comment 2

[Chris Waterson]

-> mail/news

Comment 3

[basic]

reporter is this bug still valid?

Comment 4

[basic]

spam, cc'ing self

Comment 5

[[e-mail]]

An RDF template defined directly in XUL using the datasources and ref attributes
for async data sources doesn't load properly.

If the datasources are added in script after the document is created then the
template loads properly.

If the datasource has been fully loaded before the document is created then the
template loads properly.

If the XUL is for a menu then the template loads properly.

Comment 6

[Chris Waterson]

Attachment: possible fix

Comment 7

[Chris Waterson]

neil: I think that attachment 51945 [details] [diff] [review] may fix this bug (it fixes bug 100952);
however, I'm not sure what I need to do to test the patch. Could you provide a
diff for mailWindowOverlay.xul that I can apply?

Comment 8

[[e-mail]]

Attachment: test case

Comment 9

[Chris Waterson]

neil: cool, thanks. I tried your patch, and am happy to report that attachment
51945 [details] [diff] [review] appears to fix this bug. Hopefully I'll check this in soon under the guise
of bug 100952.

Comment 10

[Chris Waterson]

Fixed with checking for bug 100952.

Comment 11

[[e-mail]]

I hate to say this but my testcase is still failing on build 2001110603 :-(

Comment 12

[Chris Waterson]

Try this with 2001-11-07; my fix for bug 100952 was backed out at the end of
0.9.5, and I sucked and didn't get around to fixing it again until the evening
of 2001-11-06.

Comment 13

[[e-mail]]

Yep, works with 2001110703.