Closed
Bug 400109
Opened 17 years ago
Closed 17 years ago
Right click on tag folder in "Recently Used Tags" generates an assert (ASSERT: Node QI Failed) and the context menu is incorrect
Categories
(Firefox :: Bookmarks & History, defect, P2)
Firefox
Bookmarks & History
Tracking
()
VERIFIED
WORKSFORME
Firefox 3
People
(Reporter: deb, Unassigned)
References
Details
Attachments
(3 files)
43.08 KB,
image/png
|
Details | |
8.84 KB,
patch
|
Details | Diff | Splinter Review | |
103.57 KB,
image/png
|
Details |
In the Bookmarks Organizer, go to Bookmarks Toolbar Folder, then Places, then Recently Used Tags. In the right-hand pane (if you have created any tags) there will be a list of tags. Right click on one of those. When I do that, the following error dialog is generated. The context menu shows up after the dialog is dismissed. --- ASSERT: Node QI Failed Stack Trace: 0:QI_node([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)],nsINavHistoryQueryResultNode) 1:asQuery([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 2:PU_getURLsForContainerNode([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 3:PC_buildContextMenu([object XULElement]) 4:buildContextMenu([object XULElement]) 5:onpopupshowing([object MouseEvent]) ---
Flags: blocking-firefox3?
Comment 1•17 years ago
|
||
i think this might be because the tag containers are a generic container object type, but have a result type of a folder: http://mxr.mozilla.org/seamonkey/source/toolkit/components/places/src/nsNavHistory.cpp#4220 if i change the result type to RESULT_TYPE_HOST, for example, the assert goes away and the context menu is properly built. i'd rather not add a new specialized result type for tag containers. instead we should add a generic container type, not specific to any particular content type. (i'd like to move towards a more generic grouping and sorting strategy as well. but that's for another bug ;))
OS: Mac OS X → All
Hardware: Macintosh → All
Comment 2•17 years ago
|
||
the assert won't happen (once we ship, as we won't don't alert / assert), but the context menu will still be messed up. morphing the title. thanks for catching this, deb.
Summary: Right click on tag folder in "Recently Used Tags" generates an error dialog → Right click on tag folder in "Recently Used Tags" generates an assert and the context menu is incorrect
Comment 3•17 years ago
|
||
Comment 4•17 years ago
|
||
note that you will get the same problem if you right click on these menu items on windows. I believe context menus are disabled for menu items on mac
Comment 6•17 years ago
|
||
I'm seeing this in today's build as I was trying to right-click to open the tag contents in all tabs (which you can do for other folder appearing things in bookmarks). If the assert isn't valuable, should it be removed?
Updated•17 years ago
|
Flags: blocking-firefox3? → blocking-firefox3+
Updated•17 years ago
|
Target Milestone: --- → Firefox 3 M10
Comment 7•17 years ago
|
||
Right clicking now causes the whole menu under "Places" in the toolbar to dismiss. Is this expected? Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9a9pre) Gecko/2007110204 Minefield/3.0a9pre
Updated•17 years ago
|
Target Milestone: Firefox 3 M10 → Firefox 3 Mx
Updated•17 years ago
|
Target Milestone: Firefox 3 Mx → Firefox 3 M11
Updated•17 years ago
|
Priority: -- → P2
Updated•17 years ago
|
Priority: P2 → P3
Comment 10•17 years ago
|
||
Same thing happens when middle-clicking on a tag, to open all windows associated with this tag.
Updated•17 years ago
|
Flags: in-litmus?
Comment 13•17 years ago
|
||
See also bug 407499 which I marked as dupe. The same assertion occurs when choosing "Open All in Tabs" from the Bookmarks Toolbar.
Comment 15•17 years ago
|
||
Darnit! I hate logging duplicate bugs. How come a search for the string "Node QI Failed" doesn't return this bug?
Comment 16•17 years ago
|
||
Probably because that string isn't in the summary.
Summary: Right click on tag folder in "Recently Used Tags" generates an assert and the context menu is incorrect → Right click on tag folder in "Recently Used Tags" generates an assert (ASSERT: Node QI Failed) and the context menu is incorrect
Comment 19•17 years ago
|
||
A rightclick on the bookmark bar while the tag is selected also triggers the assert.
Updated•17 years ago
|
Target Milestone: Firefox 3 beta3 → ---
Comment 23•17 years ago
|
||
I'm sure this is new, as I can remember doing this before and not getting this error. But I might be mistaken.... On Firefox 3 Beta 3 I get an error when I go to and click on: Smart Bookmarks > Recent Tags > (random tag) > Open All in Tabs Error comes up: ASSERT: Node QI Failed Stack Trace: 0:QI_node([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)],nsINavHistoryQueryResultNode) 1:asQuery([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 2:PU_getURLsForContainerNode([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 3:PU_openContainerInTabs([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)],[object XULCommandEvent]) 4:oncommand([object XULCommandEvent]) It's different to the error when I go to and right click on: Smart Bookmarks > Recent Tags > (random tag) Which produces: ASSERT: Node QI Failed Stack Trace: 0:QI_node([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)],nsINavHistoryQueryResultNode) 1:asQuery([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 2:PU_nodeIsReadOnly([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 3:PC_isCommandEnabled(placesCmd_sortBy:name) 4:isCommandEnabled(placesCmd_sortBy:name) 5:updatePlacesCommand(placesCmd_sortBy:name) 6:goUpdatePlacesCommands() 7:oncommandupdate([object Event]) 8:focus() 9:buildContextMenu([object XULElement]) 10:onpopupshowing([object MouseEvent]) And again slightly different to when I go to and right click on: Smart Bookmarks > Recent Tags > (random tag) > (random item) Which produces: ASSERT: Node QI Failed Stack Trace: 0:QI_node([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)],nsINavHistoryQueryResultNode) 1:asQuery([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 2:([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 3:PC__hasRemovableSelection(true) 4:PC_isCommandEnabled(placesCmd_moveBookmarks) 5:isCommandEnabled(placesCmd_moveBookmarks) 6:updatePlacesCommand(placesCmd_moveBookmarks) 7:goUpdatePlacesCommands() 8:oncommandupdate([object Event]) 9:focus() 10:buildContextMenu([object XULElement]) 11:onpopupshowing([object MouseEvent])
Comment 25•17 years ago
|
||
Dietrich, how many percent of all work to do is your WIP? Or is it bitrotted meanwhile?
Comment 26•17 years ago
|
||
When using a test build that has possible fixes for numerous Drag & Drop issues, I have found that the asserts no longer occur: http://www.wg9s.com/mozilla/firefox/ Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b4pre) Gecko/2008021613 Minefield/3.0b4pre Firefox/3.0 ID:2008021613
Comment 27•17 years ago
|
||
Firefox hangs now for good when I try to click away the errors. I tried the 23 Feb build of Jim's link but it doesn't start up :(.
Comment 28•17 years ago
|
||
(In reply to comment #27) > Firefox hangs now for good when I try to click away the errors. > confirmed on latest Mac nightly.
Priority: P3 → P1
Target Milestone: --- → Firefox 3 beta4
Comment 29•17 years ago
|
||
I cannot repoduce this with today's nightly on Windows. Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b4pre) Gecko/2008022308 Minefield/3.0b4pre Firefox/3.0 ID:2008022308
Comment 31•17 years ago
|
||
this could dependant on Bug 385245 since that's changing how results are returned, and we could check for Ci.nsINavHistoryQueryOptions.RESULTS_AS_TAG_QUERY
Updated•17 years ago
|
Priority: P1 → P2
Comment 32•17 years ago
|
||
Why has this dropped to P2? I don't see how Firefox can realistically ship with out fixing this, as soon as Journalists, Opera fanboys and Microsofts fanboys find out about this Firefox will become practically a laughing stock. And the bug seems to have got progressively worse, when I used to right click I just got 1 error message, last night on the latest trunk build I got 7 distinct error messages, then when I right clicked again I got about 14 distinct error messages.
Comment 34•17 years ago
|
||
It looks like my patch for bug 385245 is fixing this, at least I cannot confirm the behavior described here. Everything I have tried works fine.
Updated•17 years ago
|
Target Milestone: Firefox 3 beta4 → Firefox 3
Comment 38•17 years ago
|
||
This works for me, at least as of the latest nightly build. So I can confirm something must have changed because it used to hit me badly when I did anything with tag containers.
Comment 39•17 years ago
|
||
It was expected that it will start working after landing of bug 385245. This has happened and was now confirmed. In any case, "Recent Tags" are now built completely differently, so if there are still some problems, a new bug should be filled.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WORKSFORME
Comment 40•17 years ago
|
||
Verified with: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.13pre) Gecko/20080226 BonEcho/2.0.0.13pre ID:2008022603 Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b4pre) Gecko/2008022806 Minefield/3.0b4pre
Status: RESOLVED → VERIFIED
Comment 41•17 years ago
|
||
I can still see this bug with Mozilla Firefox Beta 3 (German) on Windows XP SP2. In order to reproduce it I created a new profile and it still happens. Steps to reproduce: 1. Create new profile and start Firefox Beta 3. 2. I bookmarked the Beta start page http://www.mozilla.com/en-US/firefox/3.0b3/firstrun/ with the tag "mozilla". 3. I left-clicked on the folder "Intelligent Bookmarks" > "Recently used tags" and than right-clicked on the tag "mozilla". 4. Three error massasages/popups came: ASSERT: Node QI Failed Stack Trace: 0:QI_node([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)],nsINavHistoryQueryResultNode) 1:asQuery([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 2:([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 3:PC__hasRemovableSelection(true) 4:PC_isCommandEnabled(placesCmd_moveBookmarks) 5:isCommandEnabled(placesCmd_moveBookmarks) 6:updatePlacesCommand(placesCmd_moveBookmarks) 7:goUpdatePlacesCommands() 8:oncommandupdate([object Event]) 9:focus() 10:buildContextMenu([object XULElement]) 11:onpopupshowing([object MouseEvent]) ASSERT: Node QI Failed Stack Trace: 0:QI_node([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)],nsINavHistoryQueryResultNode) 1:asQuery([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 2:([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 3:PC__hasRemovableSelection(false) 4:PC_isCommandEnabled(cmd_cut) 5:isCommandEnabled(cmd_cut) 6:goUpdateCommand(cmd_cut) 7:goUpdateGlobalEditMenuItems() 8:updateEditUIVisibility([object MouseEvent]) ASSERT: Node QI Failed Stack Trace: 0:QI_node([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)],nsINavHistoryQueryResultNode) 1:asQuery([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 2:([xpconnect wrapped (nsISupports, nsINavHistoryResultNode, nsINavHistoryContainerResultNode)]) 3:PC__hasRemovableSelection(false) 4:PC_isCommandEnabled(cmd_delete) 5:isCommandEnabled(cmd_delete) 6:goUpdateCommand(cmd_delete) 7:goUpdateGlobalEditMenuItems() 8:updateEditUIVisibility([object MouseEvent])
Comment 42•17 years ago
|
||
(In reply to comment #41) > I can still see this bug with Mozilla Firefox Beta 3 (German) on Windows XP It will be fixed with Firefox beta4.
Comment 47•16 years ago
|
||
Test cases were added for 3.x test runs on litmus for regression testing. For 3.0, https://litmus.mozilla.org/show_test.cgi?id=7536 For 3.1, https://litmus.mozilla.org/show_test.cgi?id=7537
Flags: in-litmus? → in-litmus+
Comment 48•15 years ago
|
||
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
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•