Comment #2 on issue 23706 by [email protected]: NPN_Invoke asserts if called within NPP_Destroy http://code.google.com/p/chromium/issues/detail?id=23706
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
-~----------~----~----~----~------~----~------~--~---
