fix for bug #381750: with lots of bookmarks, opening up the bookmark manager dialog is slow

r=dietrich
2007-05-28 03:57:20 +00:00 · 2007-05-28 03:57:20 +00:00 · a062dc65a7
--- a/browser/components/places/content/controller.js
+++ b/browser/components/places/content/controller.js
@ -41,7 +41,7 @@ const NHRVO = Ci.nsINavHistoryResultViewObserver;

 // XXXmano: we should move most/all of these constants to PlacesUtils
 const ORGANIZER_ROOT_HISTORY_UNSORTED = "place:beginTime=-2592000000000&beginTimeRef=1&endTime=7200000000&endTimeRef=2&type=1";
-const ORGANIZER_ROOT_BOOKMARKS = "place:folder=2&group=3&excludeItems=1";
+const ORGANIZER_ROOT_BOOKMARKS = "place:folder=2&group=3&excludeItems=1&queryType=1";
 const ORGANIZER_SUBSCRIPTIONS_QUERY = "place:annotation=livemark%2FfeedURI";

 // No change to the view, preserve current selection
--- a/browser/components/places/content/tree.xml
+++ b/browser/components/places/content/tree.xml
@ -170,6 +170,9 @@
        <parameter name="placeURI"/>
        <body><![CDATA[
          function findNode(container, placeURI) {
+            if (container.uri == placeURI)
+              return container;
+              
            container.containerOpen = true;
            for (var i = 0; i < container.childCount; ++i) {
              var child = container.getChild(i);