With TestMode enabled Firefox changes the order of windows when a background window is focused

RESOLVED FIXED in mozilla25

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: Andrei Eftimie, Assigned: Neil Deakin)

Tracking

unspecified
mozilla25
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
This behaviour is tested via Mozmill (see bug 887315).

With testMode enabled Firefox still changes the order the windows when a background window gets focused (it brings the focused window upfront).

This gets fixed by Neil's attachement 765942 from bug 803492
With that patch applied the window order remains consistent (as returned by getZOrderDOMWindowEnumerator)

Neil, I am assigning this directly to you.
I hope I'm not breaking protocol too much by doing this directly, but I don't want this issue to get lost. Thanks
(Assignee)

Comment 1

5 years ago
Created attachment 770202 [details] [diff] [review]
Don't call SetVisibility in test mode

Not sure what side effects this will have, but its said to fix the issue in test mode.
Attachment #770202 - Flags: review?(bugs)
Comment on attachment 770202 [details] [diff] [review]
Don't call SetVisibility in test mode

Well, if testing mode users need this, fine.
Attachment #770202 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/d6ca24ba3673
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Neil, do you think we can get this backported to aurora? It would be important for us given that 24 becomes the next ESR release and will be around for a while. It would save us a lot of headaches. Thanks.
Flags: needinfo?(enndeakin)

Updated

5 years ago
Depends on: 893896
(Assignee)

Comment 5

4 years ago
Do you still need this? Just set the flags as necessary on the patch/bug.
Flags: needinfo?(enndeakin)
(In reply to Neil Deakin from comment #5)
> Do you still need this? Just set the flags as necessary on the patch/bug.

Well, now its too late to modify the product for a testing change. :(
You need to log in before you can comment on or make changes to this bug.