Bug 489356: Fennec should store starred bookmarks in unfiledBookmarksFolder - like FF, r=gavin
authorMark Finkle <mfinkle@mozilla.com>
Mon, 01 Jun 2009 15:05:03 -0400
changeset 65239 c7a59ac16918367c50baf5bf98b5045c50debc22
parent 65238 89b4171b3de07638ab8022e25e5407b7d21b4357
child 65240 8f9ca3e673a55f5398e06f0866bb5d355617b138
push idunknown
push userunknown
push dateunknown
reviewersgavin
bugs489356
Bug 489356: Fennec should store starred bookmarks in unfiledBookmarksFolder - like FF, r=gavin
mobile/chrome/content/bindings.xml
mobile/chrome/content/browser-ui.js
--- a/mobile/chrome/content/bindings.xml
+++ b/mobile/chrome/content/bindings.xml
@@ -700,17 +700,17 @@
           ]]>
         </body>
       </method>
 
       <method name="openFolder">
         <parameter name="aRootFolder"/>
         <body>
           <![CDATA[
-            aRootFolder = aRootFolder || PlacesUtils.bookmarks.bookmarksMenuFolder;
+            aRootFolder = aRootFolder || PlacesUtils.bookmarks.unfiledBookmarksFolder;
 
             this._activeItem = null;
 
             let parents = this._parents;
             while (parents.firstChild)
               parents.removeChild(parents.firstChild);
 
             const XULNS = "http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul";
@@ -895,17 +895,17 @@
         </body>
       </method>
 
       <method name="openFolder">
         <parameter name="aRootFolder"/>
         <parameter name="aLevel"/>
         <body>
           <![CDATA[
-            aRootFolder = aRootFolder || PlacesUtils.bookmarks.bookmarksMenuFolder;
+            aRootFolder = aRootFolder || PlacesUtils.bookmarks.unfiledBookmarksFolder;
             aLevel = aLevel || 0;
 
             let items = this._items;
             if (!aLevel) {
               while (items.firstChild)
                 items.removeChild(items.firstChild);
 
               let node = { itemId: aRootFolder, title: PlacesUtils.bookmarks.getItemTitle(aRootFolder), type: Ci.nsINavHistoryResultNode.RESULT_TYPE_FOLDER };
--- a/mobile/chrome/content/browser-ui.js
+++ b/mobile/chrome/content/browser-ui.js
@@ -562,17 +562,17 @@ var BrowserUI = {
       case "cmd_star":
       {
         this.hideControls();
 
         var bookmarkURI = browser.currentURI;
         var bookmarkTitle = browser.contentDocument.title || bookmarkURI.spec;
 
         if (PlacesUtils.getMostRecentBookmarkForURI(bookmarkURI) == -1) {
-          var bookmarkId = PlacesUtils.bookmarks.insertBookmark(PlacesUtils.bookmarks.bookmarksMenuFolder, bookmarkURI, PlacesUtils.bookmarks.DEFAULT_INDEX, bookmarkTitle);
+          var bookmarkId = PlacesUtils.bookmarks.insertBookmark(PlacesUtils.bookmarks.unfiledBookmarksFolder, bookmarkURI, PlacesUtils.bookmarks.DEFAULT_INDEX, bookmarkTitle);
           BrowserUI.updateStar();
 
           var ios = Cc["@mozilla.org/network/io-service;1"].getService(Ci.nsIIOService);
           var favicon = document.getElementById("urlbar-favicon");
           var faviconURI = ios.newURI(favicon.src, null, null);
 
           PlacesUtils.favicons.setAndLoadFaviconForPage(bookmarkURI, faviconURI, true);
         }