[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[or-cvs] [torbutton/master 10/47] Bug 1283: Disable the fix. Too much collateral damage..



Author: Mike Perry <mikeperry-git@xxxxxxxxxx>
Date: Mon, 22 Mar 2010 16:42:34 -0700
Subject: Bug 1283: Disable the fix. Too much collateral damage..
Commit: 32f85b922dd67eb24f04afbe51ecb05a2bc8ec44

We need to work on better UI to explain to users why their browser
appears dead. And we need to make it a little less dead, too.
---
 src/chrome/content/torbutton.js |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/src/chrome/content/torbutton.js b/src/chrome/content/torbutton.js
index 3410f07..429e0bd 100644
--- a/src/chrome/content/torbutton.js
+++ b/src/chrome/content/torbutton.js
@@ -1988,6 +1988,9 @@ function torbutton_check_js_tag(browser, tor_enabled, js_enabled) {
     }
 
     /* Solution from: https://bugzilla.mozilla.org/show_bug.cgi?id=409737 */
+    // XXX: This is disabled. It kills the entire window. We need to redirect
+    // focus and inform the user via a lightbox.
+    // See https://bugs.torproject.org/flyspray/index.php?do=details&id=1283
     try {
         if (!browser.contentWindow)
             torbutton_log(3, "No content window to disable JS events.");
@@ -1995,6 +1998,7 @@ function torbutton_check_js_tag(browser, tor_enabled, js_enabled) {
             eventSuppressor = browser.contentWindow.
                 QueryInterface(Components.interfaces.nsIInterfaceRequestor).
                        getInterface(Ci.nsIDOMWindowUtils);
+        eventSuppressor = null;
     } catch(e) {
         torbutton_log(4, "Failed to disable JS events: "+e)
     }
-- 
1.7.1