532336 - Bookmarks toolbar corrupted after upgrade, ASSERT notification

Status: RESOLVED DUPLICATE of bug 528006

(Firefox :: Bookmarks & History, defect)

Description

[fawer]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a1pre) Gecko/20091129 Minefield/3.7a1pre (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a1pre) Gecko/20091129 Minefield/3.7a1pre (.NET CLR 3.5.30729)

I don't know if the summary is accurate but people have to search for something.

Background: I had put my ArtofProblemSolving (AoPS) bookmark in my bookmarks toolbar (it was the only one there). Then I had moved the AoPS icon off of the bookmarks toolbar onto the "command strip" and minimized the toolbar. The "command strip" I am talking about were had icons in the order of [back] [forward] [refresh] [stop] [separator] [AoPS] [New Tab] [New Window] [Undo Delete] [address bar] etc.

I upgraded from 3.5.5 to this version of Minefield. Then, the size of the AoPS icon remained the same, but the icon picture was not there. In its place was a drop down menu containing AoPS. Removing it from the bookmarks toolbar resulted in one of the ASSERT messages down there, and then the empty icon was gone. Adding it back in resulted in the other ASSERT message, and the empty icon was back, only now it had increased to a longer length (it is dependent on the length of the bookmark name). Adding other bookmarks to my toolbar resulted in the same behavior.

If n icons are on the toolbar, n-1 of them show up. The last one added appears in the drop down menu.

I can reproduce this behavior consistently; I will not turn off my computer for a while (maybe a week or two) so I will not have to restart my Minefield.

Reproducible: Always

Steps to Reproduce:
The assert messages happen every time, not the upgrade problems. I have not tried to reupgrade.



ASSERT: parent node must have _DOMElement set
Stack Trace: 
0:PMV_nodeRemoved([object ResultNodeClassInfo],[object ResultNodeClassInfo],0)
1:removeItem(232)
2:PRIT_doTransaction()
3:doTransaction()
4:PAT_commit(false)
5:PAT_doTransaction()
6:doTransaction([xpconnect wrapped nsITransaction])
7:placesTxn_doTransaction([xpconnect wrapped nsITransaction])
8:doTransaction([xpconnect wrapped nsITransaction])
9:PC__removeRowsFromBookmarks(Remove Selection)
10:PC_remove(Remove Selection)
11:PC_doCommand(placesCmd_delete)
12:doCommand(placesCmd_delete)
13:goDoPlacesCommand(placesCmd_delete)
14:oncommand([object XULCommandEvent])

ASSERT: parent node must have _DOMElement set
Stack Trace: 
0:PMV_nodeInserted([object ResultNodeClassInfo],[object ResultNodeClassInfo],0)
1:insertBookmark(3,[xpconnect wrapped nsIURI],-1,Art of Problem Solving)
2:PCIT_doTransaction()
3:doTransaction()
4:PAT_commit(false)
5:PAT_doTransaction()
6:doTransaction([xpconnect wrapped nsITransaction])
7:placesTxn_doTransaction([xpconnect wrapped nsITransaction])
8:doTransaction([xpconnect wrapped nsITransaction])
9:PC_paste()
10:PC_doCommand(cmd_paste)
11:doCommand(cmd_paste)
12:goDoCommand(cmd_paste)
13:oncommand([object XULCommandEvent])

Comment 1

[Tim (fmdeveloper)]

This looks to be a duplicate of bug 528006. Let me know if you think that it isn't.

Comment 2

[fawer]

(In reply to comment #1)
> This looks to be a duplicate of bug 528006. Let me know if you think that it
> isn't.

You are right. Maybe if I started out with two items in my bookmarks toolbar before the upgrade, perhaps the chevron would hold 2 items instead of 1. But I don't know, and will not test.