If call from Javascript: window.createPopup(document.getElementById('urlbar'), document.getElementById('popup'), 0, 0, 'popup', 'topleft','topleft'); Mozilla crashes. I think the problem is that nsWebShellWindow::CreatePopup is not returning the [nsIDOMWindow** outPopup] parameter.

->hyatt

This function has not been updated to reflect the new popup code. Low priority. Moving to M17.

Could I get a hint what needs to be done here? I'd like to have a go at implementing URL autocomplete and this is a necessary function for this I think.

The window IDL file needs to be fixed to remove the DOMWindow as a return value. Popups aren't DOM windows anymore. Then the C++ implementation has to be fixed so that it no longer returns a DOM window.

So since popups are no longer DOM windows, I assume that there will be no way of closing a popup from Javascript once it is open? Something like: popup = window.createPopup(document.getElementById('urlbar'), document.getElementById('popup'), 0, 0, 'popup', 'topleft','topleft'); . . . popup.close(); This is something I would need if I'm to emulate the IE style of URL completion.

Yes, there will be. I have a bug on that already marked for M14.

Will be an interface/API on the popup element itself.

I have fixed this bug in my tree. Will create & upload patches as soon as patches from bug 22979 have been applied/cleared out of my tree.

createPopup needs to move onto the popup element eventually. I was thinking of renaming it to "open" when I did this. Pulling this bug in to M14. This should be done at the same time i add a close() method to the popup element.

Taking menu/popup bugs.

you get this back, per our conversation.

BULK MOVE: Changing component from XUL to XP Toolkit/Widgets: XUL. XUL component will be deleted.

Moving to M15. Lowe, I'd love to have your patch to stop the crash. If I get it earlier, I'll take it.

Hyatt: hope you don't mind if I take this bug. I'll attach patch and ask you to review it before I checkin.

Go for it.

Hyatt: have attached patch file for review. Please take a look when you get a chance. Thanks. Michael

Reassign to hyatt for code review. Send back to me if you want me to checkin this.

putting on beta1 radar

Thanks. Will try to get to this tomorrow.

I went ahead and removed createPopup, replacing it with openPopup. Fix checked in.

please ignore, massive spam giving jrgm@netscape.com backlog of XPToolkits resolved fixed bugs to verify

Adding crash keyword