230370 - remove bookmark separator properties dialog p=mconnor

246431 - Add "New Bookmark" to the bookmarks context menu r=mconnor

browser/components/bookmarks/content/bookmarks.js

@@ -276,21 +276,21 @@ var BookmarksCommand = {
     // bm_properties
     switch (type) {
     case "BookmarkSeparator":
-      commands = ["bm_newfolder", "bm_separator", 
+      commands = ["bm_newbookmark", "bm_newfolder", "bm_newseparator", "bm_separator",
                   "cut", "copy", "paste", "bm_separator",
                   "delete", "bm_separator",
                   "bm_properties"];
       break;
     case "Bookmark":
       commands = ["bm_open", "bm_openinnewwindow", "bm_openinnewtab", "bm_separator",
-                  "bm_newfolder", "bm_separator",
+                  "bm_newbookmark", "bm_newfolder", "bm_newseparator", "bm_separator",
                   "cut", "copy", "paste", "bm_separator",
                   "delete", "bm_separator",
                   "bm_properties"];
       break;
     case "Folder":
       commands = ["bm_expandfolder", "bm_openfolder", "bm_managefolder", "bm_separator", 
-                  "bm_newfolder", "bm_separator",
+                  "bm_newbookmark", "bm_newfolder", "bm_newseparator", "bm_separator",
                   "cut", "copy", "paste", "bm_separator",
                   "delete", "bm_separator",
                   "bm_properties"];
@@ -656,7 +656,7 @@ var BookmarksCommand = {
   {
     var selection = BookmarksUtils.getSelectionFromResource(aResource, aTarget.parent);
     var ok        = BookmarksUtils.insertAndCheckSelection(aTxnType, selection, aTarget);
-    if (ok) {
+    if (ok && aTxnType != "newseparator") {
       ok = this.openBookmarkProperties(selection);
       if (!ok)
         BookmarksCommand.deleteBookmark(selection);