From e022aa16d0bfe80e86b80a6cd6f0b548957abea6 Mon Sep 17 00:00:00 2001
From: "gavin%gavinsharp.com"
 <gavin%gavinsharp.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Wed, 22 Aug 2007 05:05:07 +0000
Subject: [PATCH] Bug 356599: browser.tabs.closeButtons pref observer may cause
 ###!!! ASSERTION: XPConnect is being called on a scope without a 'Components'
 property, patch by Nickolay Ponomarev <asqueella@gmail.com>, r=me

git-svn-id: svn://10.0.0.236/trunk@232801 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/browser/base/content/tabbrowser.xml | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/mozilla/browser/base/content/tabbrowser.xml b/mozilla/browser/base/content/tabbrowser.xml
index ce3e6d1af24..c381d49d5a6 100644
--- a/mozilla/browser/base/content/tabbrowser.xml
+++ b/mozilla/browser/base/content/tabbrowser.xml
@@ -2653,7 +2653,7 @@
           this.adjustTabstrip();
 
           pb2.addObserver("browser.tabs.closeButtons", 
-                          this._prefObserver, true);
+                          this._prefObserver, false);
 
           var self = this;
           function onResize() {
@@ -2680,6 +2680,11 @@
 
       <destructor>
         <![CDATA[
+          var pb2 =
+              Components.classes['@mozilla.org/preferences-service;1'].
+              getService(Components.interfaces.nsIPrefBranch2);
+          pb2.removeObserver("browser.tabs.closeButtons", this._prefObserver);
+
           // Release timer to avoid reference cycles.
           if (this._animateTimer) {
             this._animateTimer.cancel();
@@ -2709,20 +2714,17 @@
           if (topic == "nsPref:changed") {
             switch (data) {
             case "browser.tabs.closeButtons":
-              var pb2 =
-                  Components.classes['@mozilla.org/preferences-service;1'].
-                  getService(Components.interfaces.nsIPrefBranch2);
-              this.tabbox.mCloseButtons = pb2.getIntPref("browser.tabs.closeButtons");
+              subject.QueryInterface(Components.interfaces.nsIPrefBranch);
+              this.tabbox.mCloseButtons = subject.getIntPref("browser.tabs.closeButtons");
               this.tabbox.adjustTabstrip();
               break;
             }
           }
         },
   
-        QueryInterface : function(aIID)
+        QueryInterface: function(aIID)
         {
           if (aIID.equals(Components.interfaces.nsIObserver) ||
-              aIID.equals(Components.interfaces.nsISupportsWeakReference) ||
               aIID.equals(Components.interfaces.nsISupports))
             return this;
           throw Components.results.NS_NOINTERFACE;