From acfdf93b873fc8a4d6e8fa483a5d756723e5fa57 Mon Sep 17 00:00:00 2001 From: "waterson%netscape.com" Date: Fri, 11 Jun 1999 07:32:29 +0000 Subject: [PATCH] Whoops. Forgot to update URL shortcuts to use the XPIDL-ized bookmarks service. Fixed now. git-svn-id: svn://10.0.0.236/trunk@34777 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/xpfe/browser/src/navigator.js | 47 +++++++++------------------ 1 file changed, 15 insertions(+), 32 deletions(-) diff --git a/mozilla/xpfe/browser/src/navigator.js b/mozilla/xpfe/browser/src/navigator.js index 3c6377fcc41..93b00dafedb 100644 --- a/mozilla/xpfe/browser/src/navigator.js +++ b/mozilla/xpfe/browser/src/navigator.js @@ -391,9 +391,9 @@ function BrowserAddBookmark(url,title) { - var svc = Components.classes["component://netscape/browser/bookmarks-service"].getService(); - svc = svc.QueryInterface(Components.interfaces.nsIBookmarksService); - svc.AddBookmark(url, title); + var bmks = Components.classes["component://netscape/browser/bookmarks-service"].getService(); + bmks = bmks.QueryInterface(Components.interfaces.nsIBookmarksService); + bmks.AddBookmark(url, title); } function BrowserEditBookmarks() @@ -495,39 +495,22 @@ return; } - // rjc: added support for URL shortcuts (3/30/1999) + // rjc: added support for URL shortcuts (3/30/1999) + try { + var bmks = Components.classes["component://netscape/browser/bookmarks-service"].getService(); + bmks = bmks.QueryInterface(Components.interfaces.nsIBookmarksService); - // get RDF Core service - var rdfCoreFound = false; - var rdfCore = XPAppCoresManager.Find("RDFCore"); - if (rdfCore) - { - rdfCoreFound = true; - } - else - { - rdfCore = new RDFCore(); - if (rdfCore) - { - rdfCore.Init("RDFCore"); -// We do not need this add() call. It fails and doesn't execute the loadURL -// call that is following. The job of add has already been done by the Init() -// call. - // XPAppCoresManager.Add(rdfCore); - rdfCoreFound = true; - } - } - if (rdfCoreFound == true) - { - var shortcutURL = rdfCore.findBookmarkShortcut(document.getElementById('urlbar').value); + var shortcutURL = bmks.FindShortcut(document.getElementById('urlbar').value); - dump("FindBookmarkShortcut: in='" + document.getElementById('urlbar').value + "' out='" + shortcutURL + "'\n"); + dump("FindShortcut: in='" + document.getElementById('urlbar').value + "' out='" + shortcutURL + "'\n"); - if ((shortcutURL != null) && (shortcutURL != "")) - { - document.getElementById('urlbar').value = shortcutURL; - } + if ((shortcutURL != null) && (shortcutURL != "")) { + document.getElementById('urlbar').value = shortcutURL; + } } + catch (ex) { + // stifle any exceptions so we're sure to load the URL. + } appCore.loadUrl(document.getElementById('urlbar').value);