73102 - Change posing of helper app dialog for embedding

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, enhancement, P1)

Description

[Peter Trudelle]

Need to change how unknown content type handler is invoked so it can be 
overridden by embedding clients.  Some details below, more at URL above; danm 
will be happy to help and answer questions.

UI Posed by Gecko (Up Calls)
============================
These are calls to window.open, in one form or another, which are relevant to 
embedding. They're split into calls and callers, where a call is considered to 
be a base-level call to window.open which should be API-ized like we're 
discussing, and a caller is something which currently uses window.open but 
should use one of these new APIs when they're created.

I've tried to categorize these sensibly... Each category implies a UI-posing 
component with methods for each item marked "CALL" in that category. Items 
marked "CALLER" in a given category may imply a new method in that component 
(eg. the print dialog stuff), or may just be a caller of some other extant 
method in a different component (eg. the uri loader opening a new window).

File Loading
------------

Unknown Content Type Handler:
CALL: /xpfe/components/ucth/src/nsUnknownContentTypeHandler.cpp#156:
  chrome://global/content/unknownContent.xul
CALL: /xpfe/components/ucth/src/nsUnknownContentTypeHandler.cpp#220:
  chrome://global/content/helperAppDldProgress.xul
CALL: /xpfe/components/ucth/src/nsUnknownContentTypeHandler.cpp#261:
  chrome://global/content/helperAppLauncher.xul

Comment 1

[Peter Trudelle]

adding mozilla0.9 keyword, dependency.

Comment 2

[Dan Rosen]

Apologies for all the bug spam. For the detailed overview of this task, see
danm's document:

  http://www.mozilla.org/xpfe/embedding-dialogs.html

See my first cut at a component-wise categorization:

  http://bugzilla.mozilla.org/showattachment.cgi?attach_id=27972

And if you want to laugh, see also my brainless incomplete IDL for these components:

  http://bugzilla.mozilla.org/showattachment.cgi?attach_id=28175

dr

Comment 3

[Bill Law]

Nominating, triaging, accepting, yada yada yada...

Comment 4

[Bill Law]

Changing summar; the term "unknown content" is now extremely deprecated.

Comment 5

[Bill Law]

Spam: new helper app dialog not making mozilla0.9, unfortunately.

Comment 6

[Bill Law]

OK, moving back to mozilla0.9 'cause these are "embedding" bugs.

Comment 7

[Bill Law]

damnit, mozilla0.9!

Comment 8

[Bill Law]

See bug 52454

Comment 9

[Peter Trudelle]

Bill, you can move this to 0.9.1, unless you have other need for it in 0.9.

Comment 10

[Bill Law]

We changed our minds; targetting 0.9.1 now.

Comment 11

[Bill Law]

fixed (part of bug 52454)

Comment 12

[sairuh (rarely reading bugmail)]

rubberstamp