Closed
Bug 89446
Opened 24 years ago
Closed 23 years ago
rdfcat doesn't work with ``http:'' URLs
Categories
(Core Graveyard :: RDF, defect, P4)
Core Graveyard
RDF
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.0.1
People
(Reporter: waterson, Assigned: waterson)
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
4.62 KB,
patch
|
mozilla
:
review+
waterson
:
superreview+
|
Details | Diff | Splinter Review |
The ``rdfcat'' test program appears to work correctly with ``file:'' URLs, but
not with ``http:'' URLs. It's possible that it's not driving Necko correctly
anymore. Needs investigation...
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Keywords: qawanted,
regression
Priority: -- → P4
Target Milestone: --- → mozilla1.0
Comment 1•24 years ago
|
||
It seems that rdfcat is broken for remote URLs because nsHttpChannel::Open()
is unimplemented -- only AsyncOpen() is supported. Sincerdfcat doesn't work
with ``http:'' URLs
RDFXMLDataSourceImpl::BlockingParse() returns NS_OK when it can't open the
channel, the failure is silent...
http://lxr.mozilla.org/seamonkey/source/rdf/base/src/nsRDFXMLDataSource.cpp#546
http://lxr.mozilla.org/seamonkey/source/netwerk/protocol/http/src/nsHttpChannel.cpp#1816
I think this broke when the HTTP branch landed in May.
This should probably be sent to Networking, but since it's assigned I'll leave
that for waterson.
Keywords: qawanted
Comment 2•24 years ago
|
||
That comment got messed up. The first paragraph should have read:
It seems that rdfcat is broken for remote URLs because nsHttpChannel::Open()
is unimplemented -- only AsyncOpen() is supported. Since
RDFXMLDataSourceImpl::BlockingParse() returns NS_OK when it can't open the
channel, the failure is silent...
Comment 3•24 years ago
|
||
darin: What happened to that wrapper class for Open using asyncopen?
threadsafety isn't a problem here.
Comment 4•24 years ago
|
||
i don't want to include a broken non-threadsafe nsHttpChannel::Open
implementation. however, that being said.. we are pretty close to being able to
implement Open correctly.. there just hasn't been any need (until now).
bbaetz: i still have that wrapper implementation in my mailbox.
Comment 5•24 years ago
|
||
Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1
(you can query for this string to delete spam or retrieve the list of bugs I've
moved)
Target Milestone: mozilla1.0 → mozilla1.0.1
Comment 6•24 years ago
|
||
Rewritten to use non-blocking Refresh() and pump an event queue until the
datasource loads. I also corrected the bogus usage and macroized the failure
messages.
The event pumping will generate lots of thread-safety assertions (it looks like
1 per call to ProcessPendingEvents()) for the nsNativeComponentManager, but
since these also occur with the network TestFileTransport app, I wasn't going
to worry about them.
Updated•24 years ago
|
Comment 7•24 years ago
|
||
clean diff after dbaron's NS_SetupRegistry changes.
Attachment #61811 -
Attachment is obsolete: true
Assignee | ||
Comment 8•24 years ago
|
||
Comment on attachment 62362 [details] [diff] [review]
v2
sr=waterson
Attachment #62362 -
Flags: superreview+
Comment 9•24 years ago
|
||
rjc, can you review? thanks.
Comment 10•23 years ago
|
||
Comment on attachment 62362 [details] [diff] [review]
v2
Sure, r=rjc
Attachment #62362 -
Flags: review+
Comment 11•23 years ago
|
||
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•