41632 - plugin leaked if page has a js error

Status: RESOLVED WORKSFORME

(Core :: JavaScript Engine, defect, P3)

Description

[sean echevarria]

If 
1) you load a page that has an <embed> plugin AND 
2) the page uses 'navigator.plugin' in a js block AND 
3) the page experiences a js exception in any js block
then the embedded plugin does not get freed.

Not sure if this belongs to DOM or JS Engine.
Will attach test case.

Comment 1

[sean echevarria]

Attachment: testcase - baseline, no plugin leak

Comment 2

[sean echevarria]

Attachment: testcase - viewing this page will cause a leak

Comment 3

[sean echevarria]

Attachment: testcase - source for sample plugin with additional trace statements

Comment 4

[sean echevarria]

The third attachment is a modified version of the npsimple plugin.  You can use 
the makefile in /mozilla/modules/plugin/test to build the plugin.  It has 
printfs for the constructor and destructor for SimplePlugin and 
SimplePluginInstance classes.

View the first 2 testcase html pages after building and installing the sample 
plugin.  When you close the browser after viewing the first test case, you will 
see that the destructor for both the Plugin and the PlugInstance are reached.  
When you close the browser after viewing the second testcase, both objects are 
leaked.

Comment 5

[sean echevarria]

added mlk keyword

Comment 6

[sean echevarria]

It's working ok now - don't know when it got fixed.  Resolving as WORKSFORME.