Bug 565953 - Intermittent failure in test_pluginstream_err.html because we're sometimes delivering NPP_StreamAsFile even after NPP_Write has failed, which is probably a violation of NPAPI expectations, although this behavior isn't actually specced out explicitly anywhere. r=josh

This commit is contained in:
Benjamin Smedberg 2011-11-21 13:40:19 -05:00
parent 42c0f8d076
commit 02bc18ef09

View File

@ -249,8 +249,9 @@ BrowserStreamChild::Deliver()
// asynchronously). Doing this resolves bug 687610, bug 670036 and possibly
// also other bugs.
if (mStreamAsFilePending) {
mInstance->mPluginIface->asfile(&mInstance->mData, &mStream,
mStreamAsFileName.get());
if (mStreamStatus == kStreamOpen)
mInstance->mPluginIface->asfile(&mInstance->mData, &mStream,
mStreamAsFileName.get());
mStreamAsFilePending = false;
}