415390 - refreshing a livebookmark in a bookmark menu's submenu throws 'Container view not found'

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, defect, P3)

Description

[Hussam Al-Tayeb]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b3pre) Gecko/2008020300 Firefox/3.0b3pre
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b3pre) Gecko/2008020300 Firefox/3.0b3pre

Reloading a a live bookmark doesn't do anything and it spits out a lot of these errors in error console when I try to reload a live bookmark.

Error: [Exception... "'Container view not found' when calling method: [nsINavHistoryResultViewer::itemRemoved]"  nsresult: "0x8057001e (NS_ERROR_XPC_JS_THREW_STRING)"  location: "JS frame :: file:///usr/lib/firefox/components/nsPlacesTransactionsService.js :: PSSFBN_doTransaction :: line 856"  data: no]
Source File: file:///usr/lib/firefox/components/nsPlacesTransactionsService.js
Line: 856

Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Hussam Al-Tayeb]

This was working in my previous cvs build on 20080201.

Comment 2

[Hussam Al-Tayeb]

Another note is that this happens only in live bookmarks in a subfolder in the 'bookmarks menu' and not in the 'bookmarks folder'

Comment 3

[Hussam Al-Tayeb]

Sorry I mean 'bookmarks toolbar' instead of 'bookmarks folder'.
Sorry for the spam and typing errors.

Last comment should be:
Another note is that this happens only in live bookmarks in a subfolder in the
'bookmarks menu' and not in the 'bookmarks toolbar'

Comment 4

[Aleksei Perepelkin [:chaos]]

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b3pre) Gecko/2008020419 Minefield/3.0b3pre

Yep, it happens to me. Got same error

Comment 5

[Marco Bonardo [:mak]]

confirmed
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b3pre) Gecko/2008020404 Minefield/3.0b3pre

Comment 6

[Marco Bonardo [:mak]]

STR:

- have a livemark into a subfolder of bookmark menu
- go to livemark into the menu, right click, Reload Live Bookmark

Comment 7

[Hussam Al-Tayeb]

Another problem: Sorting by name from bookmarks menu (not organiser) doesn't work and spits out the same error.

There may be further breakage. If I find anything else broken with same error message, I will make sure to post it here.

Comment 8

[Marco Bonardo [:mak]]

could someone check if this regressed after 29/01?

Comment 9

[Mike Beltzner [:beltzner, not reading bugmail]]

Mano, the fact that it's talking about missing a container makes me wonder if this is a places thing ...

Comment 10

[Hussam Al-Tayeb]

(In reply to comment #8)
> could someone check if this regressed after 29/01?
> 

yes, it is after 29/01 because I remember it was working for me on 20080201.

Comment 11

[Marco Bonardo [:mak]]

after checking that this was working in 28/01 build i suppose another regression from Bug 412148

Comment 12

[Marco Bonardo [:mak]]

Attachment: move back _containerNodesMap

this fixes the problem, however i'm not sure about the cause, Mano you probably want to fix this better or investigate on what happens here, since i don't know this code interactions so well

Comment 13

[Marco Bonardo [:mak]]

from what i saw, _containerNodesMap was empty when looking for the node, so it was probably resetted after being setup

Comment 14

[Hussam Al-Tayeb]

(In reply to comment #12)
> Created an attachment (id=302794) [details]
> move back _containerNodesMap
> 
> this fixes the problem, however i'm not sure about the cause, Mano you probably
> want to fix this better or investigate on what happens here, since i don't know
> this code interactions so well
> 

you patch also fixes the issue I mentioned in Comment #7   ;)

Comment 15

[Dietrich Ayala (:dietrich)]

maybe causing bug 407192.

Comment 16

[Hussam Al-Tayeb]

Yes same bug because Marco Bonardo's patch also fixes bug 407192 for me.

Comment 17

[Marco Bonardo [:mak]]

while working at drop marker on the bookmarks menu i found that containerNodesMap in "set result()" is cleared when right clicking on the menu item (the livemark)...

Comment 18

[Marco Bonardo [:mak]]

more generally, set result is called every time i right click on a Container, so the nodesMap is cleared while it should not

Comment 19

[Marco Bonardo [:mak]]

here is a stack printed with NS_ASSERT when that happens:

ASSERT: undefined
Stack Trace: 
0:([xpconnect wrapped nsINavHistoryResult])
1:viewer([xpconnect wrapped nsINavHistoryResultViewer])
2:PU_getURLsForContainerNode([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode, nsINavHistoryQueryResultNode)])
3:PC_buildContextMenu([object XULElement])
4:buildContextMenu([object XULElement])
5:onpopupshowing([object MouseEvent])

Comment 20

[Asaf Romano (gone)]

Attachment: patch

Comment 21

[Asaf Romano (gone)]

mozilla/browser/components/places/content/menu.xml 1.100
mozilla/browser/components/places/content/toolbar.xml 1.122
mozilla/browser/components/places/content/treeView.js 1.39

Comment 23

[Gervase Markham [:gerv]]

Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv