404987 - nsFingerChannel seems to be confused

Status: RESOLVED WONTFIX

(Core :: Networking, defect)

Description

[timeless]

http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/extensions/finger/nsFingerChannel.h&rev=1.17&mark=63-66,69-75#63
  63 class nsFingerChannel : public nsIChannel
  69     NS_DECL_ISUPPORTS
  70     NS_DECL_NSIREQUEST
  71     NS_DECL_NSICHANNEL
  72     NS_DECL_NSISTREAMLISTENER
  73     NS_DECL_NSIREQUESTOBSERVER
  74     NS_DECL_NSITRANSPORTEVENTSINK
  75     NS_DECL_NSIPROXIEDCHANNEL

http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/extensions/finger/nsFingerChannel.cpp&rev=1.61&mark=74-79#63
  74 NS_IMPL_ISUPPORTS5(nsFingerChannel, 
  75                    nsIChannel, 
  76                    nsIRequest,
  77                    nsIStreamListener, 
  78                    nsIRequestObserver,
  79                    nsIProxiedChannel)

Unless I'm missing something, the QI impl does not match the contract impl, that seems to be bad

Comment 1

[Matthias Versen [:Matti]]

The finger protocol is gone