-
Notifications
You must be signed in to change notification settings - Fork 123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
zombie compartment of DragNgoModoki_Fx3.7.uc.js #8
Comments
Umm, I can not reproduce.
|
You're right. If A drag a link in new Tab, say, B (About Mozilla in your procedure), B will be cleaned, but not A. "about us" compartment will be gone after a while, but "www.mozilla.org" compartments will be there no matter how long you wait. |
Because you did not close tab(www.mozilla.org)... |
No, it was closed. If A drag a link open tab B. Then A compartment stay when it's closed, in fact, even after all tab closed. |
autoContextmenu.uc.xulがFireGestures 1.6.6、All-in-One Gestures 0.22.1を導入していると機能しません |
I had found out what is the problem: self.sourcenode is never delete, even after tab are close. switch (obj) {
case 'link':
data = sourceNode;
if (data instanceof HTMLAnchorElement) {
var url = data.href;// self.getDroppedURL_Fixup(data.href);
var baseURI = self.ioService.newURI(data.ownerDocument.documentURI, null, null);
url = self.ioService.newURI(url, null, baseURI).spec;
if (url) {
info.urls.push(url);
info.texts.push(gatherTextUnder(data));
info.nodes.push(data);
info.files.push(null);
self.sourcenode = data; // Here store an object reference to the page
}
}
break; |
Drag a link, open it in new tab will create a zombie compartment in about:memory.
Test under Fx nightly and only installed userchrome.js the extension and two script of yours: DragNgoModoki_Fx3.7.uc.js userchrome.js
The text was updated successfully, but these errors were encountered: