Closed
Bug 41110
Opened 25 years ago
Closed 1 year ago
remove nsIWebshellWindow/nsIXULWindow usage
Categories
(Core :: DOM: Navigation, task, P3)
Tracking
()
RESOLVED
FIXED
Future
People
(Reporter: jud, Unassigned)
Details
(Keywords: arch, embed)
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.
Reporter | ||
Comment 1•25 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.
Keywords: nsbeta2
Target Milestone: --- → M16
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.
M16 has been out for a while now, these bugs target milestones need to be
updated.
Reporter | ||
Comment 4•25 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.
Reporter | ||
Updated•25 years ago
|
Target Milestone: M16 → mozilla0.8
Comment 6•21 years ago
|
||
Is this still needed, or have more recent arch changes made this irrelevant?
Assignee: danm.moz → adamlock
![]() |
||
Comment 7•21 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.
Comment 8•21 years ago
|
||
nsWindowDataSource.cpp is the guts behind the window menu, it was split off from
the window mediator to remove the RDF dependency.
![]() |
||
Comment 9•21 years ago
|
||
OK. So sounds like the photon widget impl is all that's left on the way to
marking this bug fixed?
Updated•16 years ago
|
Assignee: adamlock → nobody
QA Contact: adamlock → docshell
Updated•3 years ago
|
Severity: normal → S3
Comment 10•1 year ago
|
||
nsIWebshellWindow/nsIXULWindow are no longer in the m-c code base.
Status: NEW → RESOLVED
Type: defect → task
Closed: 1 year ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•