102769 - Shutdown and Destroy not called properly on legacy style plug-in

Status: VERIFIED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[Alec Lazarescu]

I have two Netscape 4.x legacy plug-ins that I tested it on (one in C++ and one 
in Delphi).  I am loading only one on the page (unlike bug #65661 and bug 
#72017), but the Destroy/Shutdown functions are still never being called.

I'm attaching my npshell.cpp.  You can download the plug-in DLL from 
http://demomml.intellipro.com/npdebug.zip. This version has debug info and the 
message boxes popping up.  You can test at the URL I put in this bug.

The Delphi example can be downloaded here (it's not mine, it's a general 
example):
http://ourworld.compuserve.com/homepages/mikes/NPDemo.zip
I'm just mentioning this one to give you an idea that it's not just me...

Just add MessageBox(0,'Destroy','Destroy',MB_OK); to the destroy in 
NPPlugin.pas to see it in action (or should I say not in action) there.  

Download the compiled DLL with the messageboxes already added from:
http://demomml.intellipro.com/npdemodebug.zip

Tested in latest nightly build.

Comment 1

[Alec Lazarescu]

Attachment: My npshell API implementation with debugging messages added

Comment 2

[Alec Lazarescu]

Fix confirmed in 0.9.6 Build.

Comment 3

[shrirang khanzode]

verif based on reporter's comment.