Comment #2 on issue 23713 by [email protected]: Asynchronous  
destruction of plugin instance if it's the last one
http://code.google.com/p/chromium/issues/detail?id=23713

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=28141

------------------------------------------------------------------------
r28141 | [email protected] | 2009-10-06 12:11:22 -0700 (Tue, 06 Oct 2009) |  
5 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/npobject_stub.cc?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/npobject_stub.h?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/plugin_channel.cc?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/webplugin_delegate_stub.cc?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/webplugin_delegate_stub.h?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/webplugin_delegate_proxy.cc?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/webplugin_delegate_proxy.h?r1=28141&r2=28140
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/npapi/ensure_scripting_works_in_destroy.html
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/npapi/ensure_scripting_works_in_destroy_iframe.html
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/ui/npapi_uitest.cc?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_client.cc?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_npobject_lifetime_test.cc?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_test.cc?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_test.h?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_windowed_test.cc?r1=28141&r2=28140
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_windowed_test.h?r1=28141&r2=28140

Fix scripting during NPP_Destroy.  Note that if the plugin is making a call  
to the renderer so this instance is in the callstack, destruction will have  
to be asynchronous and so scripting still won't work.  This change also  
fixes use of PluginChannel after it's deleted (if this was the last  
instance).

BUG=23713, 23706
TEST=added ui test
Review URL: http://codereview.chromium.org/258026
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to