Closed Bug 739393 Opened 8 years ago Closed 8 years ago
New tab opened in background is not set to inactive
Attachment #609471 - Flags: review?(mark.finkle)
Attachment #609471 - Flags: review?(mark.finkle) → review+
Comment on attachment 609471 [details] [diff] [review] patch This patch does not work correctly. Setting browser.docShell.isActive too soon after the browser is created has no effect, and also seems to prevent later assignements from having any effect. It also seems to cause an intermittent exception during startup that leaves the browser in a broken state. We might need to call setActive after a timeout, or something.
Attachment #609471 - Flags: review-
Desktop Firefox sets docShellIsActive=false on all newly-created <browser> elements without a timeout... I haven't figured out yet why this isn't working in Fennec.
I found out that docShellIsActive will only work after the <browser> element has been added to the DOM.
Comment on attachment 616419 [details] [diff] [review] patch v2 In hindsight it makes perfect sense. Nice catch.
Attachment #616419 - Flags: review?(mark.finkle) → review+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
Verified fixed on: -build: Firefox for Android 21.0a1 (2013-02-11) -device: Samsung Galaxy Nexus -OS: Android 4.1.2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.