diff --git a/chrome/content/zotero/browser.js b/chrome/content/zotero/browser.js index d6d53ef9e5..a226de9b96 100644 --- a/chrome/content/zotero/browser.js +++ b/chrome/content/zotero/browser.js @@ -45,7 +45,6 @@ var Zotero_Browser = new function() { this.toggleMode = toggleMode; this.toggleCollapsed = toggleCollapsed; this.chromeLoad = chromeLoad; - this.chromeUnload = chromeUnload; this.contentLoad = contentLoad; this.contentHide = contentHide; this.tabClose = tabClose; @@ -110,8 +109,6 @@ var Zotero_Browser = new function() { window.addEventListener("load", function(e) { Zotero_Browser.chromeLoad(e) }, false); - window.addEventListener("unload", - function(e) { Zotero_Browser.chromeUnload(e) }, false); ZoteroPane_Local.addReloadListener(reload); reload(); @@ -321,27 +318,18 @@ var Zotero_Browser = new function() { function(e) { Zotero_Browser.resize(e) }, false); } - /* - * Called when chrome is unloaded - */ - function chromeUnload() { - } - /* * An event handler called when a new document is loaded. Creates a new document * object, and updates the status of the capture icon */ function contentLoad(event) { - var isHTML = event.originalTarget instanceof HTMLDocument; var doc = event.originalTarget; - var rootDoc = doc; + var isHTML = doc instanceof HTMLDocument; + var rootDoc = (doc instanceof HTMLDocument ? doc.defaultView.top.document : doc); + var browser = Zotero_Browser.tabbrowser.getBrowserForDocument(rootDoc); + if(!browser) return; if(isHTML) { - // get the appropriate root document to check which browser we're on - while(rootDoc.defaultView.frameElement) { - rootDoc = rootDoc.defaultView.frameElement.ownerDocument; - } - // ignore blacklisted domains try { if(doc.domain) { @@ -368,18 +356,6 @@ var Zotero_Browser = new function() { } catch (e) {} - // Figure out what browser this contentDocument is associated with - var browser; - var browsers = Zotero_Browser.tabbrowser.browsers; - var nBrowsers = Zotero_Browser.tabbrowser.browsers.length; - for(var i=0; i