735766 - (LA-seoquake-seo-exte) SeoQuake add-on creates zombie compartments

Status: RESOLVED FIXED

(WebExtensions :: General, defect, P3)

Description

[Kris Maglione [:kmag]]

The SeoQuake add-on creates a single zombie compartment for every page it analyzes. Viewing a new page causes the previous zombie compartment to disappear. No special steps to reproduce are required. Simply installing the extension is enough.

Author CCed.

Comment 1

[Andrew Williamson [:eviljeff]]

SeoQuake Developer(s): is there any update you can give us on your progress in fixing this memory leak?

Comment 2

[Nick Pershin]

Sure, Andrew, the fix will be included into our next scheduled release.

Comment 3

[Andrew Williamson [:eviljeff]]

(In reply to Nick Pershin from comment #2)
> Sure, Andrew, the fix will be included into our next scheduled release.

Excellent; do you know when this will be (so I know when to check back on this bug)?

Comment 4

[Nick Pershin]

Approximately you can check this bug again in two and a half months, Andrew.

Comment 5

[Andrew Williamson [:eviljeff]]

(In reply to Nick Pershin from comment #4)
> Approximately you can check this bug again in two and a half months, Andrew.

Hi Nick, while I appreciate you have a release schedule and need to time to QA, etc, 2 and a half months is a rather long time.  Would you be able to issue an interim release to just fix this issue in the next 2 weeks?  If its not possible we will downgrade the add-on in the mean time to preliminary reviewed to reduce the number of new users it impacts.

Comment 6

[Nick Pershin]

(In reply to Andrew Williamson [:eviljeff] from comment #5)
> Hi Nick, while I appreciate you have a release schedule and need to time to
> QA, etc, 2 and a half months is a rather long time.  Would you be able to
> issue an interim release to just fix this issue in the next 2 weeks?  If its
> not possible we will downgrade the add-on in the mean time to preliminary
> reviewed to reduce the number of new users it impacts.

OK, we'll try to release this fix asap.

Comment 7

[Nick Pershin]

Hello,
we've added the SEOquake 2.8.2 release (this bugfix incl.) to the Mozilla repository.
Apparently all the zombie compartments from our side are prevented.
Could you approve it ASAP to close this bug?

Comment 8

[Andrew Williamson [:eviljeff]]

Memory leak doesn't exist in 2.8.2 so marking as fixed.