Closed Bug 1489833 Opened Last year Closed 9 months ago

Remove nsIWindowMediator::getZOrderDOMWindowEnumerator

Categories

(Core :: XUL, enhancement, P5)

enhancement

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: dao, Assigned: dao)

References

(Blocks 4 open bugs)

Details

Attachments

(1 file)

Bug 1489552 removes getZOrderDOMWindowEnumerator's only caller in mozilla-central.
Priority: -- → P5
Summary: Remove nsIWindowMediator::getZOrderDOMWindowEnumerator → Remove nsIWindowMediator::getZOrderDOMWindowEnumerator / getZOrderXULWindowEnumerator
Summary: Remove nsIWindowMediator::getZOrderDOMWindowEnumerator / getZOrderXULWindowEnumerator → Remove nsIWindowMediator::getZOrderDOMWindowEnumerator
Blocks: 1490250
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
There's a bunch of uses in comm-central.  What should they use instead?
Flags: needinfo?(dao+bmo)
(In reply to Boris Zbarsky [:bzbarsky, bz on IRC] from comment #2)
> There's a bunch of uses in comm-central.  What should they use instead?

They should adopt something like BrowserWindowTracker.getTopWindow which needs no xpfe black magic. This seems better for everyone as getZOrderDOMWindowEnumerator has been broken forever.
Flags: needinfo?(dao+bmo)
Thanks for the heads-up, I'll file a bug. Check this comment:

mail/base/content/msgMail3PaneWindow.js
670	// XXX We'd like to use getZOrderDOMWindowEnumerator here, but it doesn't work
Depends on: 1509060
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/82dc18fb40a3
Remove nsIWindowMediator::getZOrderDOMWindowEnumerator. r=bzbarsky
https://hg.mozilla.org/mozilla-central/rev/82dc18fb40a3
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.