Tweak the fix for bug 1366250 so that it works with background windows too

RESOLVED FIXED in Firefox 55

Status

()

Core
DOM
RESOLVED FIXED
7 months ago
7 months ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

The patch will be on top of the patch in bug 1366517
Created attachment 8869838 [details] [diff] [review]
background_focus.diff

https://treeherder.mozilla.org/#/jobs?repo=try&revision=26f88b4da533bf63d0b57681168a7b62e0798947


This is a bit annoying since it copies part of the SetFocusInner logic.
But still rather conservative approach.
Attachment #8869838 - Flags: review?(ehsan)

Updated

7 months ago
Attachment #8869838 - Flags: review?(ehsan) → review+

Comment 2

7 months ago
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d614e6f2b9ad
try to avoid layout flushes also when calling .focus() in non-focused windows, r=ehsan

Comment 3

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d614e6f2b9ad
Status: NEW → RESOLVED
Last Resolved: 7 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.