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

RESOLVED FIXED in mozilla25

Status

()

defect
RESOLVED FIXED
6 years ago
4 months ago

People

(Reporter: andrei, Assigned: enndeakin)

Tracking

unspecified
mozilla25
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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
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
Closed: 6 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)
Depends on: 893896
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. :(
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.