remove the event handlers after installing the css so it doesn't get …

…duplicated if a thread is refreshed.
TroZ · Mar 22, 2010 · 9232941 · 9232941
1 parent 700455b
commit 9232941
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/init_events.js b/init_events.js
@@ -8,6 +8,13 @@ Event.prototype.addHandler = function(callback)
     this.eventHandlers.push(callback);
 }
 
+Event.prototype.removeHandler = function(callback)
+{
+    var index = this.eventHandlers.indexOf(callback);
+    if (index >= 0)
+        this.eventHandlers.splice(index, 1);
+}
+
 Event.prototype.raise = function(arg1, arg2, arg3)
 {
     for (var i = 0; i < this.eventHandlers.length; i++)

diff --git a/scripts/highlight_original_poster.js b/scripts/highlight_original_poster.js
@@ -32,6 +32,13 @@ settingsLoadedEvent.addHandler(function()
                     css += " { " + HighlightOriginalPoster.css + " }";
                     insertStyle(css);
                 }
+                HighlightOriginalPoster.uninstall();
+            },
+
+            uninstall: function()
+            {
+                processPostEvent.removeHandler(HighlightOriginalPoster.gatherCss);
+                fullPostsCompletedEvent.removeHandler(HighlightOriginalPoster.installCss);
             }
 
         }