From 1ef56374211d3f8a47a9dd14b52706e20c514819 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Paul=20O=E2=80=99Shannessy?= <paul@oshannessy.com>
Date: Thu, 11 Feb 2010 15:08:28 -0800
Subject: [PATCH] Bug 545487 - Cleanup TabTracker_observe [r=Mardak] Remove the
 "load" event listener and don't QI since we don't need to.

---
 services/sync/modules/engines/tabs.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/services/sync/modules/engines/tabs.js b/services/sync/modules/engines/tabs.js
index 8c1d088ffca4..24030ae72126 100644
--- a/services/sync/modules/engines/tabs.js
+++ b/services/sync/modules/engines/tabs.js
@@ -253,12 +253,12 @@ TabTracker.prototype = {
 
   observe: function TabTracker_observe(aSubject, aTopic, aData) {
     // Add tab listeners now that a window has opened
-    let window = aSubject.QueryInterface(Ci.nsIDOMWindow);
     if (aTopic == "domwindowopened") {
       let self = this;
-      window.addEventListener("load", function() {
+      aSubject.addEventListener("load", function onLoad(event) {
+        aSubject.removeEventListener("load", onLoad, false);
         // Only register after the window is done loading to avoid unloads
-        self._registerListenersForWindow(window);
+        self._registerListenersForWindow(aSubject);
       }, false);
     }
   },