[Pkg-mozext-commits] [adblock-plus-element-hiding-helper] 344/483: Topic 8918 - Don't keep a persistent selection element around, it delays compartment GC
David Prévot
taffit at moszumanska.debian.org
Thu Jan 22 21:41:55 UTC 2015
This is an automated email from the git hooks/post-receive script.
taffit pushed a commit to branch master
in repository adblock-plus-element-hiding-helper.
commit a5c09779f8572d6c78074e8671600cb11501dfe4
Author: Wladimir Palant <trev at adblockplus.org>
Date: Thu Dec 8 09:07:56 2011 +0100
Topic 8918 - Don't keep a persistent selection element around, it delays compartment GC
---
modules/Aardvark.jsm | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/modules/Aardvark.jsm b/modules/Aardvark.jsm
index 5f417f9..bb91597 100644
--- a/modules/Aardvark.jsm
+++ b/modules/Aardvark.jsm
@@ -79,8 +79,7 @@ var Aardvark =
this.browser.contentWindow.focus();
let doc = this.browser.contentDocument;
- if (!this.boxElem)
- this.boxElem = E("ehh-elementmarker").firstChild;
+ this.boxElem = doc.importNode(E("ehh-elementmarker").firstChild.cloneNode(true));
this.initHelpBox();
@@ -388,8 +387,6 @@ var Aardvark =
else
label.className = "label onTop";
- if (this.boxElem.ownerDocument != doc)
- this.boxElem = doc.importNode(this.boxElem, true);
doc.documentElement.appendChild(this.boxElem);
this.paintNode = doc.defaultView;
@@ -575,13 +572,14 @@ var Aardvark =
this.browser.removeEventListener("keypress", this.onKeyPress, true);
this.browser.removeEventListener("mousemove", this.onMouseMove, true);
this.browser.contentWindow.removeEventListener("pagehide", this.onPageHide, true);
-
+
this.anchorElem = null;
this.selectedElem = null;
this.window = null;
this.browser = null;
this.commentElem = null;
this.lockedAnchor = null;
+ this.boxElem = null;
E = function(id) null;
return false;
},
--
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/adblock-plus-element-hiding-helper.git
More information about the Pkg-mozext-commits
mailing list