The default bug view has changed. See this FAQ.

Visited add-on (preliminary reviewed version 0.093) causes immortal zombie chrome compartments

RESOLVED FIXED

Status

Tech Evangelism
Add-ons
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: TheOne, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [MemShrink:P3], URL)

Attachments

(1 attachment, 1 obsolete attachment)

159.63 KB, application/octet-stream
Details
(Reporter)

Description

5 years ago
This add-on leaks chome compartments.

STR:

1) Install the add-on from the url above
2) Go to any webpage you with already visited links (I used youtube.com).
3) Press ALT+V do activate the add-on
2) Close tab and disable the add-on
4) Go to about:memory?verbose and click <minimize memory usage>
5) The add-on is still listed on about:memory

The author has been informed about this bug.
Summary: KillSpinners add-on causes immortal zombie compartments → KillSpinners add-on causes immortal zombie chrome compartments
(Reporter)

Updated

5 years ago
Summary: KillSpinners add-on causes immortal zombie chrome compartments → Visited add-on causes immortal zombie chrome compartments

Updated

5 years ago
Depends on: 724404
Which version is this?

If this is dependent on bug 724404, does that mean that when that bug is fixed the leak will be gone?  If so, I don't know if we need this bug, it would probably suffice to record which add-ons are affected in bug 724404.
(Reporter)

Updated

5 years ago
Summary: Visited add-on causes immortal zombie chrome compartments → Visited add-on (preliminary reviewed version 0.093) causes immortal zombie chrome compartments
My Visited 0.093 use Addon-kit 1.3 with this modules: 
self, hotkeys, widget, page-mod, tabs, and notifications

I will try to fix this bug, thank you.
(In reply to ZIG_ZIG_ZIG@hotmail.com from comment #2)
> My Visited 0.093 use Addon-kit 1.3 with this modules: 
> self, hotkeys, widget, page-mod, tabs, and notifications

As bug 724404 indicates, it might be the widget module that's causing the leak.
Whiteboard: [MemShrink] → [MemShrink:P3]
Zig, version 1.5 of the SDK is now available. Please rebuild your add-on using this version and submit it to AMO, so we can check if the leaks are fixed.
Today, I use SDK ver 1.5 but it is still listed on about:memory. I will try to fix soon.

I found some bug with widget.
When I install more then one addons that use widget. The addon bar will auto show in new window.

1) Install two addons that use widget. 
2) Press CTRL+/ to close addon bar.
3) Open new window.
2) Addon bar will auto show.

Thank you.
I test SDK 1.5 with "Test Zombie" v.12 but it still be zombie.
https://builder.addons.mozilla.org/addon/1041626/revision/12/

I found cause is module name "widget" and "tabs".

When I comment widget and tabs, this addon isn't listed on about:memory.
https://builder.addons.mozilla.org/addon/1041626/revision/13/
According to bug 730566 comment #5, some SDK leaks are still present.
Component: JavaScript Engine → Add-ons
Product: Core → Tech Evangelism
Version: Trunk → unspecified
ZIG: version 1.6 of the SDK was released and it addresses all known memory leaks. Please rebuild your add-on with the 1.6 SDK and submit an update so we can test it again.
The developer has been contacted again through the editor tools.
I tested it on 1.6 but I see it still be zombie. I'm trying to find couse.
But now my computer that have tested code has a trouble.
I will submit update soon.
Thank you.
Do you have any updates on this, Zig?
(In reply to Jorge Villalobos [:jorgev] from comment #11)
> Do you have any updates on this, Zig?

I will update as soon as possible. Thanks for wait.
Zig: we just need you to repack with the latest version of the SDK, 1.6.1 and upload a new version.  If the leak is still there we'll have a look into it but at the moment we can't discount known leaks in earlier versions of the SDK.

If we don't see an update by the end of next week we'll have to downgrade your listing to reduce the impact on users.
Created attachment 627570 [details]
Visited 0.10b.rev123
Sorry for my late.

I attach Visited 0.10b.rev123
This version use SDK 1.7

I see "jid0-xgzydxpakrowmumfwkinyrxigba-at-jetpack/api-utils/lib/traceback.js" still be zombie.
Alex, I think this is another potential problem in the SDK.
Zig, can you try again with the latest SDK?
Created attachment 656367 [details]
Visited 0.2b

I test with SDK 1.9 and after I disable it and call GC, I see 

compartment([System Principal], resource://jid0-xgzydxpakrowmumfwkinyrxigba-at-jetpack/addon-kit/lib/notifications.js, 0x9ee6000)

compartment([System Principal], jar:file:///C:/Users/XXXXX/AppData/Roaming/Mozilla/Firefox/Profiles/XXXXX.default/extensions/jid0-xGZYdxpAkROWMUMfWKINyrXigBA@jetpack.xpi!/bootstrap.js, 0x9bf9000)

still be zombies.
Attachment #627570 - Attachment is obsolete: true
If I enable it after start Firefox then disable and call GC, I see 

compartment([System Principal], resource://jid0-xgzydxpakrowmumfwkinyrxigba-at-jetpack/api-utils/lib/traceback.js, 0x92ca000)

compartment([System Principal], jar:file:///C:/Users/XXXXX/AppData/Roaming/Mozilla/Firefox/Profiles/XXXXX.default/extensions/jid0-xGZYdxpAkROWMUMfWKINyrXigBA@jetpack.xpi!/bootstrap.js, 0x886e000)

still be zombies.
I don't see zombie compartments with the latest release on Firefox 16. Calling this fixed. You should nonetheless repack with the latest Jetpack release.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.