Closed Bug 462222 Opened 11 years ago Closed 6 years ago
ZOrder DOMWindow Enumerator broken on Linux and Mac
getZOrderDOMWindowEnumerator seems to be working only on Windows. We need to adjust the #ifdefs at the following places accordingly: http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.js#4267 http://mxr.mozilla.org/mozilla-central/source/browser/components/nsBrowserContentHandler.js#253
Bug 456439 will be adding another copy of this code to nsBrowserGlue.js as well.
As per comment 2, dunno why he didn't flip the flag himself ;)
Flags: blocking-firefox3.1? → blocking-firefox3.1+
Target Milestone: --- → Firefox 3.1
Looks like this isn't going to get fixed on Mac anytime soon. Ehsan, do you have a good understanding of the changes we need to handle this?
Without the dependencies being fixed, there is nothing to do here. This bug is mostly about removing the conditional code inside of BROKEN_WM_Z_ORDER blocks, once the dependencies are fixed. Given that bug 450576 is blocking1.9.1-, and bug 156333 is wanted1.9.1+, I'm not sure how this one ended up being blocking1.9.1+. Turning the flag to ? for reconsideration.
Flags: blocking-firefox3.1+ → blocking-firefox3.1?
bug 450576 was previously blocking but josh has concluded that it is too risky to take for 1.9.1, so we can't block here either.
Assignee: dtownsend → nobody
Flags: blocking-firefox3.1? → blocking-firefox3.1-
Pretty sure we still have to fix the ifdefs so that we don't assume mac will work...
Flags: blocking-firefox3.1- → blocking-firefox3.1+
this is what nsPrivateBrowsingService.js already does
The patch in bug 462478 would make this one simpler.
Whiteboard: [has patch][needs review mconnor]
Comment on attachment 356633 [details] [diff] [review] updated (checked in) thanks!
Whiteboard: [has patch][needs review mconnor] → [needs landing]
Comment on attachment 356633 [details] [diff] [review] updated (checked in) http://hg.mozilla.org/mozilla-central/rev/d765319f547d http://hg.mozilla.org/releases/mozilla-1.9.1/rev/a05e3e37e030
Attachment #356633 - Attachment description: updated → updated (checked in)
Assignee: dao → nobody
Target Milestone: Firefox 3.1 → ---
This is "fine" for now, but we should stay on top of this for .next
Connor, this is blocking, but there's a checked in patch? I think we want to: - close this bug - open a new bug with a description of what has to change - make that bug blocking, if you believe it needs to be - make sure that bug has an assignee Assigning this to mconnor for now to handle.
Assignee: nobody → mconnor
After further discussion with mconnor - this doesn't block. mconnor does still believe we should fix it, instead of just assuming broken z order everywhere except windows. It may well belong in its own bug instead of here, but either way, marking it blocking-.
Flags: blocking-firefox3.6+ → blocking-firefox3.6-
while on Mac it is working it still fails on Linux. Is there anything we can do here to get this enumerator fixed?
Looks like this is working now across platforms. Ehsan, can you verify that?
The code in question doesn't exist any more.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
It still exists. (See the URL field.)
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Oh, right, it was moved... by me! ;-)
What I wanted to say in comment 17 is that we have a test for Mozmill which makes use of the getZOrderDOMWindowEnumerator and works well across platforms. So I wonder if those remaining ifdef declarations are still necessary.
According to bug 874566, the ZOrder enumerator now works on Mac, so updating this accordingly.
Summary: getZOrderDOMWindowEnumerator broken on both Linux and Mac → getZOrderDOMWindowEnumerator broken on Linux
(In reply to Mark Hammond (:markh) from comment #22) > According to bug 874566, the ZOrder enumerator now works on Mac, so updating > this accordingly. Please continue that discussion in bug 450576. Once resolved, we should update this bug, and file a bug on updating our BROKEN_WM_Z_ORDER logic.
Summary: getZOrderDOMWindowEnumerator broken on Linux → getZOrderDOMWindowEnumerator broken on Linux and Mac
This bug doesn't seem useful anymore.
Status: REOPENED → RESOLVED
Closed: 7 years ago → 6 years ago
Resolution: --- → FIXED
I morphed bug 891194 into being about fixing BROKEN_WM_Z_ORDER
Assignee: nobody → dao
Resolution: INCOMPLETE → FIXED
Target Milestone: --- → Firefox 3.6
Dao, so this issue got fixed 4 years ago, right? I ask because MDN still refers to this bug and says that it is broken on Linux and OS X. https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XPCOM/Reference/Interface/nsIWindowMediator#getZOrderDOMWindowEnumerator()
You're misunderstanding what this bug is about. It's not about whether getZOrderDOMWindowEnumerator is broken but where it's broken.
Somebody should update MDN to say that it's broken on Linux and point to bug 156333 rather than this bug. Unfortunately editing MDN requires a github account these days and I don't have one.
(In reply to Dão Gottwald [::dao] from comment #28) > Somebody should update MDN to say that it's broken on Linux and point to bug > 156333 rather than this bug. Unfortunately editing MDN requires a github > account these days and I don't have one. Thanks Dao. Sheppy, can you or someone else please update the above referenced page? Thanks.
You need to log in before you can comment on or make changes to this bug.