Bug 1095406 - Remove synchronous getBookmarksForURI and getMostRecentBookmarkForURI from PlacesUtils. r=standard8 draft
authorMarco Bonardo <mbonardo@mozilla.com>
Tue, 23 Jan 2018 10:58:31 +0100
changeset 723441 9006b4b5dc0832e432c5839e9819e5bdb3e89bdb
parent 723064 3d23e6d98a09a3395bf2b0d9cb03dd4be358c715
child 746860 61a3eeb05292c577445b8f17be0d623ac8305029
push id96430
push usermak77@bonardo.net
push dateTue, 23 Jan 2018 10:02:39 +0000
reviewersstandard8
bugs1095406
milestone60.0a1
Bug 1095406 - Remove synchronous getBookmarksForURI and getMostRecentBookmarkForURI from PlacesUtils. r=standard8 MozReview-Commit-ID: 8DRDjnNqwPF
toolkit/components/places/PlacesUtils.jsm
--- a/toolkit/components/places/PlacesUtils.jsm
+++ b/toolkit/components/places/PlacesUtils.jsm
@@ -1318,38 +1318,16 @@ this.PlacesUtils = {
         return null;
       return stmt.row.post_data;
     } finally {
       stmt.finalize();
     }
   },
 
   /**
-   * Get all bookmarks for a URL, excluding items under tags.
-   */
-  getBookmarksForURI:
-  function PU_getBookmarksForURI(aURI) {
-    return this.bookmarks.getBookmarkIdsForURI(aURI);
-  },
-
-  /**
-   * Get the most recently added/modified bookmark for a URL, excluding items
-   * under tags.
-   *
-   * @param aURI
-   *        nsIURI of the page we will look for.
-   * @returns itemId of the found bookmark, or -1 if nothing is found.
-   */
-  getMostRecentBookmarkForURI:
-  function PU_getMostRecentBookmarkForURI(aURI) {
-    let bmkIds = this.bookmarks.getBookmarkIdsForURI(aURI);
-    return bmkIds.length ? bmkIds[0] : -1;
-  },
-
-  /**
    * Returns a nsNavHistoryContainerResultNode with forced excludeItems and
    * expandQueries.
    * @param   aNode
    *          The node to convert
    * @param   [optional] excludeItems
    *          True to hide all items (individual bookmarks). This is used on
    *          the left places pane so you just get a folder hierarchy.
    * @param   [optional] expandQueries