70223 - nsIURIContentListener GetProtocolHandler needs to be removed.

Status: VERIFIED FIXED

(Core Graveyard :: Embedding: APIs, defect)

Description

[Judson Valeski]

No-one's using it, and this is a public interface. It's adding to confusion.

Comment 1

[Judson Valeski]

nom 0.9

Comment 2

[Michael Dunn]

Correction: Changing QA contact for the Embed API bugs to David Epstein.

Comment 3

[David Epstein]

reassign qa contact to Ashish.

Comment 4

[rpotts (gone)]

hey scott,

i'll take this one - cause i really want to get nsIURIContentListener frozen ;-)

-- rick

Comment 5

[rpotts (gone)]

Attachment: Patch which removes all references to nsIURIContentListener::GetProtocolHandler(...)

Comment 6

[Judson Valeski]

Comment on attachment 48904 [details] [diff] [review]
Patch which removes all references to nsIURIContentListener::GetProtocolHandler(...)

hit the commerical tree too (I saw a couple impls there)

Comment 7

[Scott MacGregor]

Comment on attachment 48904 [details] [diff] [review]
Patch which removes all references to nsIURIContentListener::GetProtocolHandler(...)

sr=mscott. Are you sure one of our embedding customers wasn't using this? I seem to remember a certain engineer asking me questions about it  and how to use get protocol handler.

Comment 8

[rpotts (gone)]

I don't think that anyone is using GetProtocol(...) because it is
*unimplemented* and uncalled within the mozilla tree.

If external embeddors need this functionality, they can get it via the
nsIIOService and necko ;-)

-- rick
 

Comment 9

[rpotts (gone)]

just checked changes into the tip on both mozilla and the commercial tree...

-- rick

Comment 10

[Ashish Bhatt]

marking as verified