just seeing if this fixes leaks; will backout soon.

xpfe/components/bookmarks/resources/bookmarks.xml

@@ -97,6 +97,7 @@
       ]]></constructor>
       <destructor><![CDATA[
         this.outlinerBuilder.removeObserver(this.builderObserver);
+        this.outliner.controllers.removeController(this.controller);
       ]]></destructor>
 
       <property name="outlinerBoxObject">
@@ -260,7 +261,7 @@
       <!-- observer -->
       <field name="DNDObserver" readonly="true"><![CDATA[
       ({
-        mOuter: this,
+        mOuter: null,
         onDragStart: function (aEvent, aXferData, aDragAction)
         {
           if (this.mOuter.outlinerBody.getAttribute("sortActive") == "true")
@@ -312,7 +313,7 @@
       <!-- nsIController -->
       <field name="controller" readonly="true"><![CDATA[
       ({
-        mOuter: this,
+        mOuter: null,
         
         supportsCommand: function BMOLController_supportsCommand(aCommand) 
         {
@@ -531,7 +532,7 @@
       <!-- nsIXULOutlinerBuilderObserver -->
       <field name="builderObserver"><![CDATA[
       ({
-        mOuter: this,
+        mOuter: null,
         inDropBefore: 1,
         inDropOn: 2,
         inDropAfter: 3,