handing this over to serge
Assignee: beppe → serge
Priority: -- → P3
Target Milestone: --- → mozilla1.0.2
hmm, it looks like we are incorrectly calling NPP_StreamAsFile() for NP_NORMAL stream type, according to http://developer.netscape.com/docs/manuals/communicator/plugin/pgfn2.htm#1007302 we have to call it only for NP_ASFILE* type. I doubt this is the regression, that call has been in the code for a long time I've just check rev=1.200 http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/modules/plugin/base/src/nsPluginHostImpl.cpp&rev=1.200&root=/cvsroot#1379 which was created on 13 Feb 2001 13:46 the call is in there. ccing plugins developers av, peterl any idea why we're calling NPP_StreamAsFile() for NP_NORMAL stype?
Serge, but looks like we still set a local cache file in any case. If we are not going to use it for mStreamType >= nsPluginStreamType_AsFile, would not it make sense not to set it for this case?
Please disregard my last comment, I missed other situations when we still need the local file.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: 4xp, patch
the fix for this bug has been checked in by patch for bug 145054 http://bugzilla.mozilla.org/attachment.cgi?id=96929&action=view I'm marking this as fixed.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.