Closed
Bug 21250
Opened 25 years ago
Closed 24 years ago
Need to implement stream-as-file service
Categories
(Core :: Networking: Cache, defect, P3)
Core
Networking: Cache
Tracking
()
VERIFIED
FIXED
M16
People
(Reporter: fur, Assigned: davidm)
References
Details
(Whiteboard: 1d)
Attachments
(3 files)
3.60 KB,
text/plain
|
Details | |
4.74 KB,
text/plain
|
Details | |
877 bytes,
patch
|
Details | Diff | Splinter Review |
I've defined an interface to be used by those components that require stream-as-file semantics, i.e. the 'jar:' protocol handler and the classic browser plug-in API. See: http://lxr.mozilla.org/seamonkey/source/netwerk/cache/public/nsIStreamAsFile.idl The cache manager already implements this interface (with some caveats - see below). In an ideal world, I think each protocol handler would be able to handle the nsIChannel::CACHE_AS_FILE flag and, via some Open, hand back an instance of nsIStreamAsFile. This would limit knowledge of whether or not a URI refers to a local file within each protocol handler. However, I don't think this will happen any time soon, so I'm proposing a subroutine that reads from a URL and stores the result in the cache. (See attachment.) I think this method should live in nsIIOService since it will be needed by multiple callers.
Reporter | ||
Comment 1•25 years ago
|
||
Reporter | ||
Comment 2•25 years ago
|
||
Reporter | ||
Comment 3•25 years ago
|
||
I said there were some caveats about the cache manager implementing nsIStreamAsFile. First, I've never tested that portion of the cache manager and it may need some debugging. Second, looking over the code, I found one problem. (Patch attached). I'll need you to check this in for me once the tree opens.
Reporter | ||
Comment 4•25 years ago
|
||
Bulk move of all Cache (to be deleted component) bugs to new Networking: Cache component.
Updated•25 years ago
|
Target Milestone: M14
Comment 6•24 years ago
|
||
Depends on implementing the disk cache. Davidm says that there's code there for stream-as-file, but it's not hooked up to the protocols yet, and hasn't been tested.
Depends on: 21100
Comment 7•24 years ago
|
||
This bug blocks bug 27763, which is beta1 PDT+, so I'm nominating this bug for beta1 review as well.
Keywords: beta1
Comment 8•24 years ago
|
||
Note: if PDT team assigns a fix cut-off date to this bug, for consistency, it shouldn't be earlier than the fix cut-off date for bug 27763, which is beta1 PDT+.
Comment 9•24 years ago
|
||
However, as I said in bug 27763, Stanley is working on an alternative solution that doesn't involve stream-as-file.
Comment 10•24 years ago
|
||
Putting on PDT-. We really want java and hope that stanleys work if fruitful
Whiteboard: [PDT-]
Comment 13•24 years ago
|
||
David, can you please post a status update here? Thanks,
Comment 16•24 years ago
|
||
Need to know the latest input on this...is there more needed here than protocol handler and the classic browser plug-in support? What is still the blocker for beta2?
Whiteboard: 1d → [NEED INFO]1d
Comment 17•24 years ago
|
||
Not needed for nsbeta2 since this is only for downloaded jar files.
Keywords: nsbeta2
Whiteboard: [NEED INFO]1d → 1d
Comment 18•24 years ago
|
||
This does not only affect jar file support - it's needed by legacy plugins also. See http://bugzilla.mozilla.org/show_bug.cgi?id=27763 and http://bugzilla.mozilla.org/show_bug.cgi?id=23667
Assignee | ||
Comment 20•24 years ago
|
||
checked in code. I tested it using a version of testprotocols. If anyone knows of any code to exercise stream as file using mozilla let me know.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 21•24 years ago
|
||
Don't the audio plugins do this (e.g. RealAudio)? Ask amusil or av.
Comment 22•24 years ago
|
||
The plugin glue code at http://lxr.mozilla.org/seamonkey/source/modules/plugin/nglsrc/nsPluginHostImpl.c pp doesn't use the cache yet, so you won't be able to exercise stream as file using a plugin yet.
You need to log in
before you can comment on or make changes to this bug.
Description
•