[tor-commits] [flashproxy/master] Add wrapper function to attach DOM events.

[dcf at torproject.org]

commit e2d353475109bfa587a685c006818980572974cb
Author: Alexandre Allaire <alexandre.allaire at mail.mcgill.ca>
Date:   Thu Dec 6 19:03:59 2012 -0500

    Add wrapper function to attach DOM events.
    
    The current code uses the standard addEventListener
    which is not supported in IE <= 8. Add a small
    wrapper function which uses the IE attachEvent method
    if it is available.
---
 proxy/options.js |   17 +++++++++++++----
 1 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/proxy/options.js b/proxy/options.js
index 326043e..f39f358 100644
--- a/proxy/options.js
+++ b/proxy/options.js
@@ -6,7 +6,16 @@ var COOKIE_NAME = "flashproxy";
 /* max-age is not supported in IE. */
 var COOKIE_LIFETIME = "Thu, 01 Jan 2020 00:00:00 GMT";
 
-window.addEventListener("load", function () {
+/* This wrapper will attach events correctly in older
+   versions of IE. */
+function add_event(elem, evt, handler) {
+    if (elem.attachEvent)
+        elem.attachEvent("on" + evt, handler);
+    else
+        elem.addEventListener(evt, handler);
+}
+
+add_event(window, "load", function () {
 
     function cookie_present() {
         var cookies = document.cookie.split(";");
@@ -46,9 +55,9 @@ window.addEventListener("load", function () {
 
     if (navigator.cookieEnabled) {
         var buttons = document.getElementById("buttons");
-        buttons.addEventListener("click", update_setting_text);
-        document.getElementById("yes").addEventListener("click", set_cookie);
-        document.getElementById("no").addEventListener("click", del_cookie);
+        add_event(buttons, "click", update_setting_text);
+        add_event(document.getElementById("yes"), "click", set_cookie);
+        add_event(document.getElementById("no"), "click", del_cookie);
         buttons.style.display = "block";
         update_setting_text();
     } else {