OnFTPControlLog still needed ?

RESOLVED INVALID

Status

()

RESOLVED INVALID
a year ago
a year ago

People

(Reporter: enrico.weigelt, Unassigned)

Tracking

52 Branch
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

a year ago
It seems that OnFTPControlLog isn't used anymore.

nsDirectoryViewer docks in via nsIFTPChannel and seem to forward to JS.
But I can't find any consumer for that.

This makes nsDirectoryViewer depending on nsIFTPChannel.
searchfox shows that this does still get used by ftp channels (gets called a bit down the line in oninputstreamready on the control channel).
Status: UNCONFIRMED → RESOLVED
Last Resolved: a year ago
Resolution: --- → INVALID
Also, bugzilla is not for questions, regardless of whether or not you've been told to stop flooding the mailing lists.
(Reporter)

Comment 3

a year ago
(In reply to Nicholas Hurley [:nwgh][:hurley] (also hurley@todesschaf.org) from comment #1)
> searchfox shows that this does still get used by ftp channels (gets called a
> bit down the line in oninputstreamready on the control channel).

Wait a second ...

It is correct that the OnFTPControlLog callback is called:

https://searchfox.org/mozilla-central/source/xpfe/components/directory/nsDirectoryViewer.cpp#150

BUT: this in turn just calls an jscript function OnFTPControlLog(), which I couldn't find anywhere.
You need to log in before you can comment on or make changes to this bug.