Closed Bug 41110 Opened 25 years ago Closed 1 year ago

remove nsIWebshellWindow/nsIXULWindow usage

Categories

(Core :: DOM: Navigation, task, P3)

x86
Linux
task

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.
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
Keywords: arch
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.
Whiteboard: [nsbeta2-]
M16 has been out for a while now, these bugs target milestones need to be updated.
Keywords: embed
pulling off of embedding radar. this should be cleaned up for internal usage, but we're ok for now on the exposed api front.
Keywords: embed, nsbeta2
Whiteboard: [nsbeta2-]
Keywords: embed
Target Milestone: M16 → mozilla0.8
.
Assignee: valeski → danm
Target Milestone: mozilla0.8 → Future
Is this still needed, or have more recent arch changes made this irrelevant?
Assignee: danm.moz → adamlock
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.
nsWindowDataSource.cpp is the guts behind the window menu, it was split off from the window mediator to remove the RDF dependency.
OK. So sounds like the photon widget impl is all that's left on the way to marking this bug fixed?
Assignee: adamlock → nobody
QA Contact: adamlock → docshell
Severity: normal → S3

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.