Closed
Bug 1775
Opened 26 years ago
Closed 7 years ago
stream converters need to implement the new StreamConverter interface.
Categories
(Core :: DOM: Serializers, defect, P2)
Tracking
()
RESOLVED
INVALID
Future
People
(Reporter: rhp, Unassigned)
References
Details
(Keywords: helpwanted, Whiteboard: [needed to cleanup nsMessenger::SaveAs])
I've made a couple of temporary changes for netlib. I have a
version of libmime running in a DLL on my machine that works
with Raptor and will render Berkeley mail folder messages.
To do this (and keep things pretty modular, I need to
register my stream handling function for RFC822 messages
from outside of netlib. Can we add NET_RegisterContentTypeConverter
to the \mozilla\network\module\net.defmodule definition
file? When we get the XP-COM interface for this function,
we can remove this export.
Also, I've added the lines:
"exts=\"eml\" type=message/rfc822 \
desc=\"RFC822 Messages\" icon=resource:/res/network/gopher-text.gif",
to the \mozilla\network\mimetypes\ngtypes.h. This is the
system association for mail files on most Win32 machines.
We need an XP-COM way of registering both the stream converters and
mime associations.
- rhp
Updated•26 years ago
|
QA Contact: 3819
Comment 2•26 years ago
|
||
setting paulmac as QA contact for all gagan's bugs (sorry for the spam)
Reporter | ||
Comment 4•26 years ago
|
||
I do want to be involved since I will be the first customer of this
functionality, but I'm not sure I can take it on my plate to own. Are we out of
luck if I can't do it because this is a must have for us with the new netlib.
- rhp
Reporter | ||
Comment 5•25 years ago
|
||
This really needs to assigned to someone in the netlib group...phil, any
thoughts?
- rhp
Updated•25 years ago
|
Assignee: rhp → warren
Comment 6•25 years ago
|
||
Reassigning to warren@netscape.com. Rich, are you blocked on this, or is it just
something we have to do before B1?
Reporter | ||
Comment 7•25 years ago
|
||
No, nothing blocked here, in fact I have this all working today with an XP-COM
interface for netlib stream converters. This is just a heads up in the event
that netlib is rewritten since I could see this slipping by the wayside for a
while. Just want to stay on the radar screen.
- rhp
Updated•25 years ago
|
Status: NEW → ASSIGNED
Comment 8•25 years ago
|
||
is this m6 or is it now moved into the necko sched.
Reporter | ||
Comment 9•25 years ago
|
||
This is a Necko thing and should tie in with that schedule. Actually,
it will probably change from a stream converter to a new DTD to handle
message/rfc822 data.
- rhp
Updated•25 years ago
|
Target Milestone: M6 → M7
Comment 10•25 years ago
|
||
moving to m7. warren can you set the right target milestone if thats not
right?
Reporter | ||
Comment 11•25 years ago
|
||
Warren,
In light of our meeting last week, will there still be an interface like this,
or will the DTD approach be the way this stuff is handled. Also, was there a
follow up on this discussion?
- rhp
Updated•25 years ago
|
Assignee: warren → rickg
Status: ASSIGNED → NEW
Comment 12•25 years ago
|
||
I think this probably falls on rickg's plate now. (Rick, let me know if not.) I
wouldn't expect to see this before M9 at the earliest.
Comment 13•25 years ago
|
||
Warren -- I'm giving this back to you as a means of closing the loop in the
issues. (I may own some of this). Rich is asking for 2 things:
1. Registering stream converters
2. mime associations
I won't be the owner of #1 (I think you are), and I *may* own #2 as soon as we
can all agree on the plan we're following.
Comment 14•25 years ago
|
||
Adding myself to the cc list.
Updated•25 years ago
|
Target Milestone: M7 → M9
Comment 15•25 years ago
|
||
m9
Updated•25 years ago
|
Target Milestone: M9 → M8
Comment 16•25 years ago
|
||
Actually I'm going to bump this back to M8. It's part of our mailnews necko
integration stuff. We need it in order to integrate mailnews which is an M8
item. We're talking to gagan and rickg about this stuff.
Updated•25 years ago
|
Assignee: warren → gagan
Comment 17•25 years ago
|
||
Off to gagan...
Comment 18•25 years ago
|
||
Necko landing...
Comment 19•25 years ago
|
||
Pushed past necko landing...
Comment 20•25 years ago
|
||
*red flag* *red flag*
we know we can't land necko without a pluggable stream converter service right?
I'm going to jump up and down and do acrobatic stunts unless we're sure necko
isn't going online until after M8 (which is probably reasonable). In which case
I buy that this could be an M9 bug. But to me this is a requirement for necko
landing if we think we're going to be turning necko on.
Comment 21•25 years ago
|
||
Changing all Networking Library/Browser bugs to Networking-Core component for
Browser.
Occasionally, Bugzilla will burp and cause Verified bugs to reopen when I do
this in a bulk change. If this happens, I will fix. ;-)
Comment 22•25 years ago
|
||
Over to Jud for the latest on this...
Updated•25 years ago
|
Status: NEW → ASSIGNED
Comment 23•25 years ago
|
||
latest interfaces for this are in mozilla/netwerk/streamconv/public. I am
actively working on this.
who's doing the 822<->html converter, mscott or rhp?
Reporter | ||
Comment 24•25 years ago
|
||
Scott is more in tune with what needs to go on to plug in libmime, but I can
surely help if pointed in the right direction...scott?
- rhp
Comment 25•25 years ago
|
||
That's probably me Jud. Since we have that hack working in the file channel this
isn't such a pressing need for us right now. We can probably move this off the
M9 radar unless you are planning on being done with M9.
Mime is currently using the nsIStreamConverter that I have in
netwerk\base\public. We should compare it to the one you are working on and see
how much more (or less) stuff needs to be done.
Comment 26•25 years ago
|
||
*** Bug 11310 has been marked as a duplicate of this bug. ***
Updated•25 years ago
|
Summary: Need to specify an libnet XP-COM inteface for stream converters → stream converters need to implement the new StreamConverter interface.
Comment 27•25 years ago
|
||
This interface has been specified. I'm changing the summary to the real issue.
I'm not sure this is really M9.
Updated•25 years ago
|
Target Milestone: M9 → M10
Comment 28•25 years ago
|
||
m10
Comment 29•25 years ago
|
||
hey Jud, is the stream converter service hooked into layout yet? i.e. if we do
use the new stream converter, will the doc loader know to use the converter
service to convert to something it can handle?
Comment 30•25 years ago
|
||
mscott, yes. The stream converter sits in the nsChannelListener (which sits
inbetween necko and the doc loader). All you have to do is use the new stream
conv api, and add you mime-type to the nsChannelListener::OnStartRequest().
Comment 31•25 years ago
|
||
Excellent! Thanks Jud. I'll start converting mime over this week. It should be
pretty straightforward because mailnews\mime was already converted to use
nsIStreamConverter2(??) which is pretty similar to the one you eventually ended
up using I think.
Updated•25 years ago
|
Assignee: valeski → akkana
Status: ASSIGNED → NEW
Target Milestone: M10 → M14
Comment 32•25 years ago
|
||
mscott has mail/news using the new api. miltipart-mixed-replace, and FTP
directory listings are using it as well. Gessner/pnunn/ and I are looking at it
for stream scanning -> mime-type guessing. I'm assigning this bug to akkana as
she has some stream converter logic to convert to the new api. This is not a
priority as she's already got her stuff going using her api. If she decides do
kill this bug that woudl be fine w/ me, but it would be nice to be unified on a
stream converter api.
Updated•25 years ago
|
Status: NEW → ASSIGNED
Whiteboard: HELP WANTED
Target Milestone: M14 → M20
Comment 33•25 years ago
|
||
I'd like to do this, and plugging in the code that actually does the conversion
is no problem; the problem is that I don't have time to figure out how the
stream converter APIs work. I'd love help with this.
Updated•25 years ago
|
Component: Networking-Core → Output
Comment 34•25 years ago
|
||
Changing Component to Output.
Comment 35•25 years ago
|
||
Hi Akkana, Any progress on this one? Is this really M20 -- it shows up on the
beta criterion list for necko (bug#12833).
(And what's the Output component?)
Comment 36•25 years ago
|
||
Warren,
akk asked me to take a look on this. I did, but it is fairly complex for me. I
would need a month or so without success garanty.
If this needs to be in beta, don't count on me. If not, *maybe* I'll fix it.
Comment 37•25 years ago
|
||
Clarification on what I'm looking for here:
I made a start on this, in htmlparser/src/nsTextConverter.{h,cpp} based on the
sample stream converter APIs, but I kept hitting questions that I didn't
understand about the interface -- for instance, what the several stream
arguments mean and how they all relate to each other, and whether asynchronous
mode really needs to be supported.
It would be a great help if someone who understands the stream converter API and
has a need for this (and therefore knows how it's going to be used) would get
these files to where they compile, then I can plug in the code that does the
actual conversion.
Reporter | ||
Comment 38•25 years ago
|
||
Hi Akkana,
This is a situation where I need to go from RFC822 to Plain Text, but without
the HTML - text stream converter interface in place, I'm kind of stuck. Is
there any change of this getting moved up from M20?
- rhp
No longer blocks: 23418
Updated•25 years ago
|
Keywords: helpwanted
Whiteboard: HELP WANTED
Comment 39•25 years ago
|
||
Bulk move of all "Output" component bugs to new "DOM to Test Conversion"
component. Output will be deleted as a component.
Comment 43•24 years ago
|
||
Anthonyd is taking over Output. Not sure whether this is still an issue, though.
Comment 44•24 years ago
|
||
Really assigning to anthony this time. Not clear whether anyone still wants
this, though.
Assignee: akkana → anthonyd
Status: ASSIGNED → NEW
Comment 45•24 years ago
|
||
> Not clear whether anyone still wants this, though.
I still think, it's a good idea, generally, theoretically. But I won't do the
work ;-P.
Comment 47•22 years ago
|
||
I think this bug is obsolete since the conversion moved to content. If nobody
comments I will INVALID it.
Comment 48•22 years ago
|
||
actually, this is still needed.
for why, see nsMessenger::SaveAs()
but we've worked around it for this long, I don't expect it to happen any time
soon.
Whiteboard: [needed to cleanup nsMessenger::SaveAs]
Comment 49•21 years ago
|
||
So what is this bug about at this point, Seth? Is it about making our document
encoders and decoders implement nsIStreamConverter so that you can, say, pump in
HTML and get out plaintext or something along those lines?
nsMessenger::SaveAs seems to just need a RFC822 to text/plain converter; right
now you use this saveListener object to handle that?
Akkana, I'd be willing to help out with stream converter stuff; I think I have a
pretty good understanding of how they work....
Comment 50•21 years ago
|
||
I'd be happy to help on how to call the serializers, if you need it. You might
want to start with the automated test in htmlparser/tests/outsinks, since that's
a standalone converter from html to whatever (plaintext in the case of the
existing tests).
Comment 51•21 years ago
|
||
Oh, Boris, there's also some serializer doc at
http://www.mozilla.org/editor/serializers.html
If there are omissions there that you think should be mentioned, let me know and
I'll update the document (or feel free to make your own additions, if you feel
so inclined).
Comment 52•21 years ago
|
||
Well, I'm still trying to figure out what the goal of this bug is... once this
bug is fixed, what do we want to be able to do?
Comment 53•21 years ago
|
||
bz, sorry for the delay.
here's what this bug is about.
in mozilla mail, we allow the user to do File | Save As | File...
They can save as:
.eml (rfc/822), .txt (plain text), or .html (html)
when saving to .txt, we need these two converters: "rfc/822 -> html" | "html ->
plain text"
There's no stream converter (at least, there wasn't) that can go from html ->
plain text
the way the current code works is if you save a message as plain text, we still
convert
rfc/822 to html, and we buffer it up.
then, in nsSaveMsgListener::OnStopRequest(), we call a static function
ConvertBufToPlainText()
(in nsMessenger.cpp, which uses the parser and a nsIHTMLToTextSink to convert.
so to clean up the code in nsMessenger.cpp, we'd a stream converter that could
take html and convert to plain text.
if you have more questions, let me know.
I need to re-read more of the bug, because the summary doesn't match what
mailnews needs anymore. (has this bug morphed? or is the original issue still
open?)
Comment 54•21 years ago
|
||
Seth, thanks for the clarification. Sounds like it would be good if content
registered a converter component that basically wraps one of the serializers....
I'll have to look at how the serializers work, exactly. ;)
Updated•16 years ago
|
Assignee: kinmoz → nobody
QA Contact: sujay → dom-to-text
Comment 55•7 years ago
|
||
This seems no longer relevant. If it still is, a new focused bug would be better.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•