[tor-commits] [torbutton/master] Bug 5729: Remove resize calls during JS hook update

mikeperry at torproject.org mikeperry at torproject.org
Thu May 10 23:40:13 UTC 2012


commit efcd3a01b163967b8d6c82c5aea1bbcabbd066c1
Author: Mike Perry <mikeperry-git at fscked.org>
Date:   Thu May 10 15:29:58 2012 -0700

    Bug 5729: Remove resize calls during JS hook update
    
    Resizing on new window creation is good enough.
---
 src/chrome/content/torbutton.js |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/src/chrome/content/torbutton.js b/src/chrome/content/torbutton.js
index b52bf9f..4a46cac 100644
--- a/src/chrome/content/torbutton.js
+++ b/src/chrome/content/torbutton.js
@@ -1520,6 +1520,7 @@ function torbutton_do_new_identity() {
   obsSvc.notifyObservers(this, "net:prune-all-connections", null);
 
   // XXX: This may not clear zoom site-specific
+  // browser.content.full-zoom
   var cps = Cc["@mozilla.org/content-pref/service;1"].
       createInstance(Ci.nsIContentPrefService);
   cps.removeGroupedPrefs();
@@ -4382,7 +4383,6 @@ function torbutton_update_tags(win, new_loc) {
             // NoScript
             torbutton_log(3, "Javascript changed from "+browser.docShell.allowJavascript+" to: "+js_enabled);
             browser.docShell.allowJavascript = js_enabled;
-            torbutton_set_window_size(browser.contentWindow);
 
             // Tag this specially, so that the next onLocationChange
             // doesn't redo these hooks.
@@ -4391,10 +4391,6 @@ function torbutton_update_tags(win, new_loc) {
             // JS was not fully enabled for some page elements. 
             // Need to reload
             browser.reload(); 
-        } else {
-            // We need to do the resize here as well in case the window
-            // was minimized during toggle...
-            torbutton_set_window_size(browser.contentWindow);
         }
     }
 



More information about the tor-commits mailing list