the users of this should be doing a createinstance rather than a get service. we also need to move this over into the URI loader dir.

Setting milestone and nominating. Wishful thinking, maybe.

Clarification: this needs to be moved over to the new embedding/components/ui/helperApps dir, not the URI loader, correct?

I'm not sure where' dan's putting all this stuff. Also, shouldn't this be a blocker for your dialog bug(s) dan?

This is fixed as part of my fix for bug 52454.

I wasn't imagining moving all these UI-controlling components into the embedding directory. I look at them as still belonging more to the code whose UI they define than to the embedded world. I was thinking of just leaving them with the component whose UI they were controlling. I've only moved the nsIPrompt stuff into embedding because it was part of appshell, and I have this dream that appshell shouldn't need to be part of an embedded distribution. So sometimes I deram small. Yeah, I'll add it to my UI meta bug so it'll look bigger.

I second that. the UI belongs to the component needing/throwing it (usually).

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

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

Putting date in status whiteboard to match blocking bug 52454 (watch that bug for any real news).

damnit, mozilla0.9!

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

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

We changed our minds; targetting 0.9.1 now.

fixed (part of bug 52454)

nsIHelperAppLauncherDialog object gets created with a do_CreateInstance(). verified in testembed (Verify Bugs > 70228). http://lxr.mozilla.org/seamonkey/source/embedding/qa/testembed/