From 438c8f83998c9115d1bdebd59917a2985ff40076 Mon Sep 17 00:00:00 2001
From: "rjc%netscape.com" <rjc%netscape.com>
Date: Wed, 15 Mar 2000 09:59:04 +0000
Subject: [PATCH] Fix bug # 22521: context menus should work on background of
 bookmarks window and background of bookmarks sidebar panel.

---
 .../bookmarks/resources/bookmarks.js          | 34 +++++++++++++------
 .../bookmarks/src/nsBookmarksService.cpp      |  6 +++-
 2 files changed, 29 insertions(+), 11 deletions(-)

diff --git a/xpfe/components/bookmarks/resources/bookmarks.js b/xpfe/components/bookmarks/resources/bookmarks.js
index 532c0200314..378a381af6f 100644
--- a/xpfe/components/bookmarks/resources/bookmarks.js
+++ b/xpfe/components/bookmarks/resources/bookmarks.js
@@ -205,7 +205,7 @@ function doPaste()
 
 			if (RDFC.IndexOf(IDRes) > 0)
 			{
-				dump("Unable to add ID:'" + theID + "' as its already in this folder.\n");
+				debug("Unable to add ID:'" + theID + "' as its already in this folder.\n");
 				continue;
 			}
 
@@ -506,17 +506,29 @@ function fillContextMenu(name)
 
     var select_list = treeNode.selectedItems;
 
-    debug("# of Nodes selected: " + treeNode.selectedItems.length);
+    debug("# of Nodes selected: " + treeNode.selectedItems.length + "\n");
 
     // perform intersection of commands over selected nodes
     var cmdArray = new Array();
-
-    for (var nodeIndex=0; nodeIndex<select_list.length; nodeIndex++)
+    var selectLength = select_list.length;
+    if (selectLength == 0)	selectLength = 1;
+    for (var nodeIndex=0; nodeIndex < selectLength; nodeIndex++)
     {
-        var node = select_list[nodeIndex];
-        if (!node)    break;
-        var id = node.getAttribute("id");
-        if (!id)    break;
+    	var id = null;
+
+	// if nothing is selected, get commands for the "ref" of the bookmarks root
+    	if (select_list.length == 0)
+    	{
+		id = treeNode.getAttribute("ref");
+	        if (!id)    break;
+    	}
+    	else
+    	{
+	        var node = select_list[nodeIndex];
+	        if (!node)    break;
+	        id = node.getAttribute("id");
+	        if (!id)    break;
+	}
         var rdfNode = rdf.GetResource(id);
         if (!rdfNode)    break;
         var cmdEnum = db.GetAllCmds(rdfNode);
@@ -562,8 +574,10 @@ function fillContextMenu(name)
     var rdfNameResource = rdf.GetResource("http://home.netscape.com/NC-rdf#Name");
     if (!rdfNameResource)        return(false);
 
+/*
     // build up menu items
     if (cmdArray.length < 1)    return(false);
+*/
 
     for (var cmdIndex = 0; cmdIndex < cmdArray.length; cmdIndex++)
     {
@@ -604,10 +618,10 @@ function fillContextMenu(name)
 			}
 
 			// and then add a "Properties" menu items
-			var propMenuName = "Properties...";
+			var propMenuName = "Properties...";			// XXX localize
 			
 			var menuItem = document.createElement("menuitem");
-			menuItem.setAttribute("value", "Properties...");		// XXX localize
+			menuItem.setAttribute("value", propMenuName);
 			menuItem.setAttribute("oncommand", "return BookmarkProperties();");
 			popupNode.appendChild(menuItem);
 		}
diff --git a/xpfe/components/bookmarks/src/nsBookmarksService.cpp b/xpfe/components/bookmarks/src/nsBookmarksService.cpp
index ad17a461539..9b00778a5ad 100644
--- a/xpfe/components/bookmarks/src/nsBookmarksService.cpp
+++ b/xpfe/components/bookmarks/src/nsBookmarksService.cpp
@@ -3239,7 +3239,7 @@ nsBookmarksService::GetAllCmds(nsIRDFResource* source,
 	{
 		cmdArray->AppendElement(kNC_BookmarkCommand_DeleteBookmark);
 	}
-	if (isBookmarkFolder)
+	if (isBookmarkFolder && (source != kNC_BookmarksRoot))
 	{
 		cmdArray->AppendElement(kNC_BookmarkCommand_DeleteBookmarkFolder);
 	}
@@ -3640,6 +3640,10 @@ nsBookmarksService::ReadBookmarks()
 	NS_ASSERTION(NS_SUCCEEDED(rv), "Unable to make NC:BookmarksRoot a sequence");
 	if (NS_FAILED(rv)) return rv;
 
+	// Make sure bookmark's root has the correct type
+	rv = mInner->Assert(kNC_BookmarksRoot, kRDF_type, kNC_Folder, PR_TRUE);
+	if (NS_FAILED(rv)) return rv;
+
 	PRBool	foundIERoot = PR_FALSE;
 
 #ifdef	DEBUG