Closed
Bug 1188912
Opened 8 years ago
Closed 7 years ago
[e10s] calling .focus() on an input field via cpows causes firefox to hang.
Categories
(Firefox :: Extension Compatibility, defect, P2)
Tracking
()
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
e10s | + | --- |
People
(Reporter: jack, Unassigned)
References
Details
(Keywords: hang, Whiteboard: triaged)
Attachments
(1 file)
10.42 KB,
text/plain
|
Details |
With e10s and multiple content processes, recently Nightlies have started locking up when using Lastpass. I've seen this on both chase.com and allstate.com. The whole browser locks up completely and must be force killed at the command line. Steps to reproduce: 1. Visit a site where a lastpass login is stored 2. Use the toolbar button menu to select "Show Matching Sites" and find the site and select "Autofill". 3. Cry. Other menu items seem to be fine, but this one seems to reliably cause a lockup.
Reporter | ||
Updated•8 years ago
|
Blocks: e10s
Summary: lastpass causes e10s lockup when selecting autofill → [e10s] lastpass causes e10s lockup when selecting autofill
Comment 1•8 years ago
|
||
Could be related to bug 1008768, but I'm not sure.
Comment 2•8 years ago
|
||
I can reproduce, but only with e10s enabled. Did something change with CPOWs recently? In general, if you have e10s turned on, I'd recommend using our experimental e10s build: https://rodan.lastpass.com/dev/lp_e10s.xpi
Reporter | ||
Comment 4•8 years ago
|
||
I was not aware there was such a thing. Indeed it seems to fix the problem.
Flags: needinfo?(jack)
Updated•8 years ago
|
Comment 6•8 years ago
|
||
FYI, I traced this freezing to calling focus() on an input field. It definitely seems like a CPOW bug in Firefox.
Comment 7•8 years ago
|
||
This isn't lastpass specific. To reproduce, in Nightly open google.com and execute the following in a browser toolbox console:
> gBrowser.selectedBrowser.contentDocumentAsCPOW.getElementById("lst-ib").focus()
(note "lst-ib" is the id of the search field on google). The browser hangs and doesn't come back. Andrew tells me his most recent e10s compatible version of the LastPass addon no longer calls .focus() due to this - which is why the problem seems to go away with that latest installed, but the underlying problem remains.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Summary: [e10s] lastpass causes e10s lockup when selecting autofill → [e10s] calling .focus() on an input field via cpows causes firefox to hang.
![]() |
||
Updated•8 years ago
|
tracking-e10s:
--- → ?
Comment 8•8 years ago
|
||
(In reply to Mark Hammond [:markh] from comment #7) > Andrew tells me his most recent e10s compatible > version of the LastPass addon no longer calls .focus() due to this - which > is why the problem seems to go away with that latest installed, but the > underlying problem remains. Just to clarify, the e10s build of LastPass (based on the add-on SDK) never relied on CPOWs in the first place. I've updated our current production build of LastPass (which does rely on CPOWs) to not call focus() in the autofill case for our next release. If this bug is fixed, I'll revert that change.
Comment 9•8 years ago
|
||
NI to tracy to confirm that this is multicontent process only
Flags: needinfo?(twalker)
Comment 10•8 years ago
|
||
(In reply to Brad Lassey [:blassey] (use needinfo?) from comment #9) > NI to tracy to confirm that this is multicontent process only I can confirm that this is only with e10s enabled - comment 7 works as expected with e10s disabled.
Flags: needinfo?(twalker)
Comment 11•8 years ago
|
||
Thanks Mark. I was NI'd to determine if dom.ipc.processCount is in play. Do you have it set to the default of 1 or something greater than that? If the latter, please reset it to 1, then see if this is reproducible. Thanks again.
Flags: needinfo?(markh)
Flags: needinfo?(wmccloskey)
Comment 12•8 years ago
|
||
(In reply to [:tracy] Tracy Walker - QA Mentor from comment #11) > Thanks Mark. I was NI'd to determine if dom.ipc.processCount is in play. Do > you have it set to the default of 1 or something greater than that? If the > latter, please reset it to 1, then see if this is reproducible. Thanks > again. This happens when that pref is set to the default of 1.
Flags: needinfo?(markh)
![]() |
||
Comment 13•8 years ago
|
||
![]() |
||
Updated•8 years ago
|
OS: Unspecified → Windows
Hardware: Unspecified → x86
Updated•8 years ago
|
Flags: needinfo?(wmccloskey)
Comment 15•7 years ago
|
||
based on comment 7 - sounds like not a huge deal anymore and we haven't seen more reports of this impacting add-ons. please untriage if you disagree
Whiteboard: triaged
Updated•7 years ago
|
Priority: -- → P2
I'm going to close this WORKSFORME. Mark's STR doesn't cause a hang for me. A lot of CPOW bugs got fixed six months ago or so. Hopefully this one did too.
Status: REOPENED → RESOLVED
Closed: 8 years ago → 7 years ago
Flags: needinfo?(wmccloskey)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•