Closed Bug 1215950 Opened 4 years ago Closed 4 years ago
Crash on Google 2-step verification
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:44.0) Gecko/20100101 Firefox/44.0 Build ID: 20151016030228 Steps to reproduce: Go to google.com Click sign in Enter email address Enter password Get to 2-step verification page Actual results: Crash
tracking-fennec: --- → ?
Please post some URLs of these crashlog reports from about:crashes
Here are a few: https://crash-stats.mozilla.com/report/index/bp-7fc685eb-46cb-4d32-8368-459e62151020 https://crash-stats.mozilla.com/report/index/bp-d675c173-14a3-47b0-bb1c-21c6f2151018 https://crash-stats.mozilla.com/report/index/bp-8f4afbca-ccfd-4951-b905-8769f2151018
4 years ago
I can reproduce the crash everytime with the steps provided on latest Nightly (2015-10-21) using Prestigio PSP5508 (Android 4.4.2)
4 years ago
Severity: normal → critical
Status: UNCONFIRMED → NEW
Ever confirmed: true
Version: Trunk → Firefox 44
Assignee: nobody → s.kaspari
tracking-fennec: ? → 42+
Duplicate of this bug: 1216563
Background: This is a regression from the Android 6.0 keyboard workaround (bug 1211848). The crash reports are from Android 6.0 too - that's good. I'm surprised that Teodora can reproduce the crash on an Android 4.4.2 device. I wonder if without the crash the keyboard would not show up too (That's what the workaround fixed). As mentioned in the duplicate (bug 1216563) I'm going to push the re-focus workaround code to the UI thread.
Attachment #8677592 - Flags: review?(nchen) → review+
Comment on attachment 8677592 [details] [diff] [review] 1215950-refocus-on-ui-thread.patch Approval Request Comment [Feature/regressing bug #]: Regression introduced by bug bug 1216563. Patch in bug 1216563 fixed an issue with the keyboard not showing up on Android 6.0 devices. This fix can lead to a crash. The patch has been uplifted to Aurora (43) and Beta (42). [User impact if declined]: Potential crash on Android 6 if keyboard should come up. [Describe test coverage new/current, TreeHerder]: Tested locally on different devices (Android 6, Android 5, Android 2.3). [Risks and why]: Code logic has not been changed. Instead workaround code is now running on the UI thread (preventing crash on non-UI thread). [String/UUID change made/needed]: -
Sebastian, you haven't really answered to the risk question. Could you explain if it is a risky change or not. Threading changes in beta 10 scare me.
By the way, the bug in "[Feature/regressing bug #]" is incorrect. This is the duplicate.
(In reply to Sylvestre Ledru [:sylvestre] from comment #9) > Sebastian, you haven't really answered to the risk question. > Could you explain if it is a risky change or not. Threading changes in beta > 10 scare me. I consider it low risk. The normal code flow remains the same. The workaround code is now pushed to the UI thread. This crash is happening because it is not running on the UI thread (it's touching views). (In reply to Sylvestre Ledru [:sylvestre] from comment #10) > By the way, the bug in "[Feature/regressing bug #]" is incorrect. This is > the duplicate. Oh, sorry. The bug is: bug 1211848 - That's where the regression / this crash has been introduced.
Flags: needinfo?(s.kaspari) → needinfo?(sledru)
Comment on attachment 8677592 [details] [diff] [review] 1215950-refocus-on-ui-thread.patch OK, Android 6.0 is going more and more popular during the next 6 weeks. Taking it avoid having do to a dot release. Should be in 42 beta 10.
I can't reproduce the crash with the steps provided on latest Nightly (2015-10-26) using Prestigio PSP5508 (Android 4.4.2)
I can also confirm that it is working for me now.
Verified as fixed in Firefox 42 Beta 10; Device: Asus Transformer Pad (Android 4.2.1).
Verified as fixed on Firefox 43.0a2 (2015-10-27) Device: Prestigio PSP5508 (Android 4.4.2)
You need to log in before you can comment on or make changes to this bug.