13815 - tree does not respond to ContentInserted() notifications

Status: VERIFIED FIXED

(Core :: XUL, defect, P1)

Description

[Chris Waterson]

To reproduce:

1. Save attached test case as "test.xul"
2. Run viewer.
3. Open test.xul in viewer.
4. Click "append" a couple of times; you should see new tree items appended.
5. Click "insert". Nothing will happen.

I've traced this through the debugger to verify that ContentInserted() is
actually being called. (Beware: if you set a breakpoint in
nsRDFElement::InsertBefore, you'll get several "bogus" calls from the test
routine's MakeTreeItem() function before the "real" call comes through.)

Comment 1

[Chris Waterson]

Attachment: test case; save as "test.xul"

Comment 2

[Peter Trudelle]

targetting m11, p3

Comment 3

[Phil Peterson]

Upgrading severity to blocker, assuming that all the bugs marked depending on
this really are.

Comment 4

[Peter Trudelle]

bumping priority to p1

Comment 5

[David Hyatt]

My hands have deteriorated to the point where I can no longer type.  I need
help.  If you think you can fix this bug on your own, please take it away from
me.  If you'd like to volunteer to be my hands for a specific bug, then I'll be
happy to come up to your cube and sit with you and fix the bug (assuming you
have the patience for that).

Comment 6

[scottputterman]

I will look into this one.  I will probably be by for some pointers.

Comment 7

[David Hyatt]

know how to fix. let's get together.

Comment 8

[scottputterman]

I have hyatt's fix in my tree.  I will check it in when I get approval.

Comment 9

[Chris Waterson]

YOU ROCK!

Comment 10

[scottputterman]

Actually Hyatt rocks, but luckily my fingers are good at typing :)

Comment 11

[scottputterman]

I checked in the fix.

Comment 12

[ckritzer (gone)]

waterson, could you please verify? I will close this Monday under the assumtion
it has been verified unless I hear otherwise from you...Thanks!

Comment 13

[ckritzer (gone)]

Mr. Waterson, my apologies.  I've got this one.

Marking VERIFIED FIXED on MacOS86 & Linux6 102708 apprunner & Win98 102708
apprunner.

Comment 14

[leger]

BULK MOVE: Changing component from XUL to XP Toolkit/Widgets: XUL.  XUL 
component will be deleted.