To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=59437
                  Issue #:|59437
                  Summary:|Mozilla plugin shutdown code is error prone.
                Component:|framework
                  Version:|680m142
                 Platform:|All
                      URL:|
               OS/Version:|All
                   Status:|NEW
        Status whiteboard:|
                 Keywords:|
               Resolution:|
               Issue type:|DEFECT
                 Priority:|P3
             Subcomponent:|code
              Assigned to:|mav
              Reported by:|mav





------- Additional comments from [EMAIL PROTECTED] Fri Dec 16 02:53:49 -0800 
2005 -------
The mozilla plugin shutdown code should be reviewed. The problem is that the
scenario when NPP_SetWindow() gets an empty pointer as second argument is not
handled completely. And this is a legal call according to Netscape Plugin API,
this call usually means that the window is going to be closed. 

Additionaly, dispatch API is used to close the frame. This does not allow to
control the frame closing. The code should switch to using of the frame API when
it is possible.

If the previous two problems are handled correctly, the plugin shutdown should
be blocked till the frame is successfuly closed. It will allow to avoid crashes
because of destroyed parent window and because of dialogs that can not be
completely modal in this case.

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to