Judson Valeski Reporter
	Description • 24 years ago

nsIWebshellWindow/nsIXULWindow usage needs to be removed (except for code using
the appshell service directly). This will allow all window users to utilize
embedded windows (which do not support nsIWebshellWindow/nsIXULWindow.

	Judson Valeski Reporter
	Comment 1 • 24 years ago

nominating nsbeta2. This is fairly ongoing work, but I need to progressively 
check changes in, and I want to abide by the rules.

	leger
	Comment 2 • 24 years ago

Putting on [nsbeta2-] radar. Not needed for beta2, but come see PDT when you 
have stuff to check in and we'll work something out.

	Mike
	Comment 3 • 24 years ago

M16 has been out for a while now, these bugs target milestones need to be 
updated.

	Judson Valeski Reporter
	Comment 4 • 24 years ago

pulling off of embedding radar. this should be cleaned up for internal usage,
but we're ok for now on the exposed api front.

	Dan M
	Comment 5 • 24 years ago

.

	Hixie (not reading bugmail)
	Comment 6 • 20 years ago

Is this still needed, or have more recent arch changes made this irrelevant?

	Boris Zbarsky [:bzbarsky]
	Comment 7 • 20 years ago

These interfaces are still used by the cached compose window stuff in mailnews
as well as nsMsgPrintEngine::ShowWindow (also mailnews).  Leaving this is ok, I
think...

The photon widget impl uses nsIXULWindow (to get docshells).  This is pretty bogus.

The rest of the usage seems to be under xpfe.  The appshell, bootstrap, and XUL
users are all fine; I dunno about the
xpfe/components/windowds/nsWindowDataSource.cpp user, since I have no clue what
it does.

	neil@parkwaycc.co.uk
	Comment 8 • 20 years ago

nsWindowDataSource.cpp is the guts behind the window menu, it was split off from
the window mediator to remove the RDF dependency.

	Boris Zbarsky [:bzbarsky]
	Comment 9 • 20 years ago

OK.  So sounds like the photon widget impl is all that's left on the way to
marking this bug fixed?