Closed
Bug 1124408
Opened 10 years ago
Closed 10 years ago
[e10s] Crash in [ChildView keyDown:] when using password manager
Categories
(Core :: Widget: Cocoa, defect)
Tracking
()
RESOLVED
FIXED
mozilla39
People
(Reporter: handyman, Assigned: handyman)
References
Details
(Keywords: crash, topcrash-mac)
Crash Data
Attachments
(1 file)
(deleted),
patch
|
masayuki
:
review+
smichaud
:
review+
|
Details | Diff | Splinter Review |
This bug is a spinoff of bug 1051842. It was reported in bug 1051842 comment 103 by Julian Viereck. The STR from there:
---------------
Using the Firefox Nightly from 20-Jan-2015 I still get crashes: https://crash-stats.mozilla.com/report/index/471bbb0d-9b41-483e-b02d-af4fa2150120
Steps to reproduce:
1. Restart Firefox
2. Go to a page that has a stored password
3. As I have locked my saved passwords via a master password, Firefox prompts me to enter the password
4. In roughly 50% of the cases, after entering the first few characters of my password, the browser crashes
---------------
My sense of this is that the password manager creates a (chrome) popup window and that switching focus to the popup windows fails for IPC reasons that are similar to those in bug 1051842, which were related to OSX window events (raising/lowering/activating/deactivating). I have not confirmed that.
I do not see this issue in non-e10s windows.
Updated•10 years ago
|
Updated•10 years ago
|
Severity: normal → critical
Assignee | ||
Comment 3•10 years ago
|
||
Just to pile on, I've (not surprisingly) seen this with the Bugzilla addon's password popup as well.
Assignee | ||
Comment 4•10 years ago
|
||
Renoming since its the #1 mac crash, according to smichaud in bug 1133103 comment 1.
Comment 5•10 years ago
|
||
Filling in the signature field to make this bug known to https://crash-stats.mozilla.com/
Crash Signature: [@ -[ChildView keyDown:]]
Comment 6•10 years ago
|
||
Mats, that won't help. See bug 1133103 comment #3.
Updated•10 years ago
|
Assignee | ||
Comment 7•10 years ago
|
||
More reliable STR:
* Activate password manager for bugzilla.mozilla.org
* Open an e10s window
* Navigate to a bug — for example: https://bugzilla.mozilla.org/show_bug.cgi?id=1124408
* Cancel the password manager popup (do not log in)
* Select the search box in the bugzilla page (not the browser’s search box). Dont type anything.
* Command-Tab away from the browser, then command-tab back
* Press Command-r to reload the page. The password manager popup will appear again.
* Type a letter.
Expected result:
The letter appears in the password field
Actual result:
CRASH!
Assignee | ||
Comment 8•10 years ago
|
||
Narrowing it down... another STR:
* Activate password manager for bugzilla.mozilla.org
* Open an e10s window
* Navigate to a bug — for example: https://bugzilla.mozilla.org/show_bug.cgi?id=1124408
* Cancel the password manager popup (do not log in)
* Click a link to another bug (say, one of the duplicates). When the browser loads the new page, the password manager popup will appear again.
* Type a letter.
Expected result:
The letter appears in the password field
Actual result:
CRASH!
Updated•10 years ago
|
Keywords: topcrash-mac
Assignee | ||
Comment 10•10 years ago
|
||
A view should not claim focus if its window has key focus but is also showing a sheet... in that case, the sheet window has focus. Here, the sheet window is the password popup.
Tests:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=4ac6b0fac9e9
Attachment #8568644 -
Flags: review?(masayuki)
Comment 11•10 years ago
|
||
Comment on attachment 8568644 [details] [diff] [review]
Report cocoa view focus correctly when sheets are exposed
Looks good to me. But I want Steven to check this.
Attachment #8568644 -
Flags: review?(smichaud)
Attachment #8568644 -
Flags: review?(masayuki)
Attachment #8568644 -
Flags: review+
Comment 12•10 years ago
|
||
Comment on attachment 8568644 [details] [diff] [review]
Report cocoa view focus correctly when sheets are exposed
Looks good to me. I haven't tested this, but let's see how it plays out.
Attachment #8568644 -
Flags: review?(smichaud) → review+
Assignee | ||
Comment 13•10 years ago
|
||
Thanks -- I've got high hopes that this is at least why its a top crash.
Tests are here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4ac6b0fac9e9
Keywords: checkin-needed
Keywords: checkin-needed
Comment 15•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Comment 16•10 years ago
|
||
I still get crashes with the latest nightly: https://crash-stats.mozilla.com/report/index/5f846d1b-cff1-455d-86dc-6dd4e2150301. This is when entering the password in the master-password prompt.
Should this bug be reopened?
Comment 17•10 years ago
|
||
I just saw this crash with build (20150302) of Nightly on Mac. But I can't reliably reproduce it. I'll keep an eye on it and try to sort out current STR's. Note: This was only happening to me, prior to the fix, when the browser was on a external (to laptop) monitor.
Assignee | ||
Comment 18•10 years ago
|
||
The bug here has been fixed (the STRs no longer produce a crash). However, the crash signature appears for at least one other case. I'm opening a new bug (bug 1138678) for a new STR that I found while trying to break things. It may or may not be related to the crashes seen by Julian and Tracy... but its probably the best we can do without STR.
You need to log in
before you can comment on or make changes to this bug.
Description
•