99622 - Freeze nsIHelperAppLauncherDialog

Status: VERIFIED FIXED

(Core Graveyard :: Embedding: APIs, defect)

Description

[Chak Nanga]

Freeze nsIHelperAppLauncherDialog

Please refer to
http://www.mozilla.org/projects/embedding/EmbedInterfaceFreeze.html for the
issues to be addressed, if any, for this interface.

Please follow the guidelines outlined in "How to mark an interface as frozen?"
section of the document above.

Comment 1

[timeless]

hrm last i checked this dialog wasn't in a good UI state so i think i object.

Comment 2

[Matthew T (active 1999-2002)]

The bug for fixing the dialog is bug 86640. I don't see anything particularly
dangerous in the API, except that it needs to allow for the `View as Text'
option. (It would probably be a good idea to allow for a `View as HTML' option
too, in case we decide to include that as well.)

Comment 3

[timeless]

for now at least s/obsolescent/obsolete/

i might want to be able to perhaps use this dialog to select a plugin, an 
internal renderer or to view inline using OLE.

mpt's comment is enough for me to feel this interface isn't ready for freezing.

Comment 4

[Boris Zbarsky [:bzbarsky]]

hmm... the interface changes to allow view as text and such are in nsIMIMEInfo.

This interface just allows one to open a dialog off a nsIHelperAppLauncher
object.

The interfaces that really need improvement are nsIMIMEInfo and
nsIHelperAppLauncher

Comment 5

[Chak Nanga]

->0.9.6

Hi,

Could we get this one into 0.9.6?

Thanks
Chak

Comment 6

[Bill Law]

This interface is "frozen," so far as I can tell.  There are no issues mentioned
in the referenced document and there are no planned changes to this interface.

Please let me know what else needs to be done.  Do I need to go add some magic
keyword to the .idl file?  Do I do that?

Comment 7

[Bill Law]

->mozilla0.9.7 since the meta bug it blocks was moved to mozilla1.0.

I don't think there is any work remaining on this; just need clarification.

Comment 8

[Bill Law]

Give this a try...

Comment 9

[David Epstein]

What are the finalized API methods for the iface? In the idl file, there's
show() and showProgressDialog(), plus contractID. If this is correct, I'll mark
this verified. Also, you should indicate in the idl file that the status is
Frozen. Thanks.

Comment 10

[Bill Law]

Yes, as I indicated previously, there are no plans to change this interface. 
I'll mark the .idl "frozen" ASAP.

Comment 11

[David Epstein]

Thanks. Marking verified.