diff options
| author | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-06 23:46:55 +0000 |
|---|---|---|
| committer | ananta@chromium.org <ananta@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-06 23:46:55 +0000 |
| commit | a1303589d06a4d4818b5d66eb7e07ba19a95a105 (patch) | |
| tree | 9d27ceeec0411c6a5b8d994d7dad3f7cab6f9dff /webkit/default_plugin | |
| parent | 2c4410df7aaa0ed2c50edb1107c085098124fc99 (diff) | |
| download | chromium_src-a1303589d06a4d4818b5d66eb7e07ba19a95a105.zip chromium_src-a1303589d06a4d4818b5d66eb7e07ba19a95a105.tar.gz chromium_src-a1303589d06a4d4818b5d66eb7e07ba19a95a105.tar.bz2 | |
Don't attempt to destroy the stream in the NPP_URLNotify notification in the default plugin. NPP_URLNotify
is always called after the stream has already been destroyed by the browser. This can lead to crashes especially
when the plugin instance is being torn down, which in turn closes open streams.
Fixes bug http://code.google.com/p/chromium/issues/detail?id=11308
Bug=11308
Review URL: http://codereview.chromium.org/113054
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15476 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/default_plugin')
| -rw-r--r-- | webkit/default_plugin/plugin_impl_win.cc | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/webkit/default_plugin/plugin_impl_win.cc b/webkit/default_plugin/plugin_impl_win.cc index 884f47d..a940927 100644 --- a/webkit/default_plugin/plugin_impl_win.cc +++ b/webkit/default_plugin/plugin_impl_win.cc @@ -281,9 +281,6 @@ void PluginInstallerImpl::URLNotify(const char* url, NPReason reason) { DisplayStatus(IDS_DEFAULT_PLUGIN_NO_PLUGIN_AVAILABLE_MSG); } } - - default_plugin::g_browser->destroystream(instance_, plugin_install_stream_, - NPRES_DONE); } int16 PluginInstallerImpl::NPP_HandleEvent(void* event) { |