From 5d1b563de8f504c4175f4c203534955b97496281 Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Sun, 18 May 2014 12:47:50 +0100
Subject: Strip top-level chrome folders.

---
 background_scripts/completion.coffee | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

(limited to 'background_scripts/completion.coffee')

diff --git a/background_scripts/completion.coffee b/background_scripts/completion.coffee
index afaec4ea..fb3356da 100644
--- a/background_scripts/completion.coffee
+++ b/background_scripts/completion.coffee
@@ -140,7 +140,17 @@ class BookmarkCompleter
   # Traverses the bookmark hierarchy, and returns a flattened list of all bookmarks.
   traverseBookmarks: (bookmarks) ->
     results = []
-    bookmarks.forEach (folder) => @traverseBookmarksRecursive folder, results
+    bookmarks.forEach (folder) =>
+      # folder is a chrome virtual root folder.
+      folder.children.forEach((folder) =>
+        # folder is one of the chrome, built-in folders ("Other Bookmarks", "Mobile Bookmarks", ...).
+        folder.children.forEach((bookmark) =>
+          # bookmark is a user-defined folder, or itself a bookmark.
+          @traverseBookmarksRecursive bookmark, results))
+          #
+          # We do not add the (possible) bookmark itself.  The user has used the "/" key to begin searching
+          # within bookmark folders.  It's not clear what that even means for top-level bookmarks.
+          # results.push bookmark
     results
 
   # Recursive helper for `traverseBookmarks`.
-- 
cgit v1.2.3