From a0abf1d3416217cdc2a4ecd409e8a72ad4605c0b Mon Sep 17 00:00:00 2001
From: "dbaron%fas.harvard.edu" <dbaron%fas.harvard.edu>
Date: Thu, 27 Jul 2006 14:56:41 +0000
Subject: [PATCH] Fix shutdown leaks of observer service via cycles through JS
 caused by closures and global variables.  b=170022  r=morse  sr=jag

---
 .../common/permissions/cookieTasksOverlay.xul | 47 +++++++++----------
 1 file changed, 23 insertions(+), 24 deletions(-)

diff --git a/suite/common/permissions/cookieTasksOverlay.xul b/suite/common/permissions/cookieTasksOverlay.xul
index cf932eb804ec..bcc437a4a568 100644
--- a/suite/common/permissions/cookieTasksOverlay.xul
+++ b/suite/common/permissions/cookieTasksOverlay.xul
@@ -34,8 +34,6 @@
 
     /******* THE FOLLOWING IS FOR THE STATUSBAR OVERLAY *******/
 
-    var gObserverService;
-    var gPref;
     var gButtonPressed;
 
     var cookieIconObserver = {
@@ -68,10 +66,8 @@
     };
 
     function viewP3PDialog() {
-      if (!gObserverService) {
-        gObserverService = Components.classes["@mozilla.org/observer-service;1"].getService(Components.interfaces.nsIObserverService);
-      }
-      gObserverService.notifyObservers(null, "cookieIcon", "off");
+      var observerService = Components.classes["@mozilla.org/observer-service;1"].getService(Components.interfaces.nsIObserverService);
+      observerService.notifyObservers(null, "cookieIcon", "off");
 
       window.openDialog
         ("chrome://cookie/content/p3pDialog.xul","_blank","modal=yes,chrome,resizable=no", this);
@@ -82,28 +78,31 @@
       }
     }
 
-    // determine if p3p pref is set
-    gPref = Components.classes['@mozilla.org/preferences-service;1'];
-    gPref = gPref.getService();
-    gPref = gPref.QueryInterface(Components.interfaces.nsIPrefBranch);
-    if (gPref.getIntPref("network.cookie.cookieBehavior") == 3) {
+    function CookieTasksStartup() {
+      // determine if p3p pref is set
+      var pref = Components.classes['@mozilla.org/preferences-service;1'].
+                     getService(Components.interfaces.nsIPrefBranch);
+      if (pref.getIntPref("network.cookie.cookieBehavior") == 3) {
 
-      // make sure p3p dll exists, else we can't keep pref set
-      if (!("@mozilla.org/cookie-consent;1" in Components.classes)) {
-        gPref.setIntPref("network.cookie.cookieBehavior", 0);
+        // make sure p3p dll exists, else we can't keep pref set
+        if (!("@mozilla.org/cookie-consent;1" in Components.classes)) {
+          pref.setIntPref("network.cookie.cookieBehavior", 0);
+        }
+      }
+
+      if ("@mozilla.org/cookie-consent;1" in Components.classes) {
+
+        // p3p dll exists so create an observer for changes in visibility of cookie icon
+        var observerService = Components.classes["@mozilla.org/observer-service;1"].getService(Components.interfaces.nsIObserverService);
+        observerService.addObserver(cookieIconObserver, "cookieIcon", false);
+
+        // determine whether or not cookie icon in this new window should be displayed
+        // we need to wait until window is created before displaying icon
+        addEventListener("load", listener, false);
       }
     }
 
-    if ("@mozilla.org/cookie-consent;1" in Components.classes) {
-
-      // p3p dll exists so create an observer for changes in visibility of cookie icon
-      var observerService = Components.classes["@mozilla.org/observer-service;1"].getService(Components.interfaces.nsIObserverService);
-      observerService.addObserver(cookieIconObserver, "cookieIcon", false);
-
-      // determine whether or not cookie icon in this new window should be displayed
-      // we need to wait until window is created before displaying icon
-      addEventListener("load", listener, false);
-    }
+    CookieTasksStartup();
 
   ]]>
   </script>