Closed
Bug 1307457
Opened 8 years ago
Closed 7 years ago
Crash in mozilla::widget::GeckoEditable::OnTextChange
Categories
(Firefox for Android Graveyard :: Keyboards and IME, defect, P2)
Tracking
(firefox49 affected, firefox50- unaffected)
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
firefox49 | --- | affected |
firefox50 | - | unaffected |
People
(Reporter: marcia, Unassigned)
Details
(Keywords: crash)
Crash Data
[Tracking Requested - why for this release]: New crash in 49 that we should track before we move to 50. This bug was filed from the Socorro interface and is report bp-e0cf98eb-829a-430a-ac14-f64862161004. ============================================================= Seen while looking at Firefox 49 crash stats. #12 top crash in the last seven days: http://bit.ly/2cQdJE2 Several comments mention the following: "trying to post on FB and it crashed" "it crashes everytime I try to forward on facebook" "I try to send a vídeo to my friend in facebook and ir crash all the time "
Comment 1•8 years ago
|
||
The line in the top frame of the crash stack was last touched in bug 1244875. Jim, thoughts?
Flags: needinfo?(nchen)
Comment 2•8 years ago
|
||
It's actually a Java exception. Would be great if we have STR.
Flags: needinfo?(nchen)
Comment 3•8 years ago
|
||
According to comment 0 trying to do stuff on Facebook triggers it. Do we not get the Java stacktrace for exceptions like this? I thought we had code for it at [1] but I don't see the annotation in the crash reports. [1] http://searchfox.org/mozilla-central/rev/8071f94c4d4b66833ad08db39565669faad94dfd/widget/android/jni/Utils.cpp#196
Priority: -- → P2
Comment 4•8 years ago
|
||
The report at https://crash-stats.mozilla.com/report/index/4d2745f4-cea3-470d-82be-a7b082161004#tab-metadata has some logcat that looks relevant. Is this of any use? 10-04 17:25:24.510 24004 24031 D GeckoBrowser: Unexpected failure during caret attach: Native touchCaret requested while Gecko enabled. 10-04 17:25:24.510 24004 24031 W GeckoEventDispatcher: No listeners for Gesture:CloseZoomedView 10-04 17:25:24.740 24004 24004 D GeckoLayerClient: Window-size changed to (1080,936) 10-04 17:25:34.670 24004 24031 E GeckoEditable: newEnd does not match text: 21 vs 16 10-04 17:25:34.760 24004 24031 W google-breakpad: ExceptionHandler::GenerateDump cloned child 10-04 17:25:34.760 24004 24031 W google-breakpad: 19678�r� 10-04 17:25:34.760 24004 24031 W google-breakpad: 10-04 17:25:34.760 24004 24031 W google-breakpad: ExceptionHandler::SendContinueSignalToChild sent continue signal to child 10-04 17:25:34.760 1967 1967 W google-breakpad: ExceptionHandler::WaitForContinueSignal waiting for continue signal... 10-04 17:25:36.170 1969 1969 D AndroidRuntime: >>>>>> START com.android.internal.os.RuntimeInit uid 10173 <<<<<<
Flags: needinfo?(nchen)
Comment 5•8 years ago
|
||
Here's another one from https://crash-stats.mozilla.com/report/index/9872204d-4aad-4cca-8065-f90572161002#tab-metadata. Definitely seems related to the "Unexpected failure during caret attach" business. 10-03 03:37:58.524 2653 2653 D GeckoLayerClient: Window-size changed to (1532,2268) 10-03 03:38:00.634 2653 2653 D ViewRootImpl: ViewPostImeInputStage ACTION_DOWN 10-03 03:38:00.714 2653 2684 D GeckoBrowser: Unexpected failure during caret attach: Native touchCaret requested while Gecko enabled. 10-03 03:38:00.714 2653 2684 W GeckoEventDispatcher: No listeners for Gesture:CloseZoomedView 10-03 03:38:00.914 2653 2653 D GeckoLayerClient: Window-size changed to (1532,1088) 10-03 03:38:01.654 2653 2684 E GeckoEditable: newEnd does not match text: 1 vs 0 10-03 03:38:01.714 2653 2684 W google-breakpad: ExceptionHandler::GenerateDump cloned child 10-03 03:38:01.714 2653 2684 W google-breakpad: 8523 10-03 03:38:01.714 2653 2684 W google-breakpad: 10-03 03:38:01.714 2653 2684 W google-breakpad: ExceptionHandler::SendContinueSignalToChild sent continue signal to child 10-03 03:38:01.724 8523 8523 W google-breakpad: ExceptionHandler::WaitForContinueSignal waiting for continue signal... 10-03 03:38:03.544 8525 8525 D AndroidRuntime:
Comment 6•8 years ago
|
||
Err actually it's the "newEnd does not match text" error that seems to be causing it. The "Unexpected failure during caret attach" error is from code that should be ripped out now that we are using the accessible carets.
Comment 7•8 years ago
|
||
Moving ni to esawin since he was the one who last touched the code in bug 1051556.
Flags: needinfo?(nchen) → needinfo?(esawin)
Updated•8 years ago
|
Component: Widget: Android → Keyboards and IME
Product: Core → Firefox for Android
Reporter | ||
Comment 8•8 years ago
|
||
http://bit.ly/2dSDgiy is a similar stack, not sure if it warrants another bug or not. That signature has about 3K in crashes in the last week.
Comment 9•8 years ago
|
||
I've only unified the crash signatures in bug 1051556, Jim has the better context for IME.
Flags: needinfo?(esawin) → needinfo?(nchen)
Comment 10•8 years ago
|
||
This would be difficult to fix without concrete STR, but I'm hoping bug 1307816 will help.
Flags: needinfo?(nchen)
Untracked as I don't see any instances of this crash in Fx50, tagging as a wontfix.
status-firefox50:
--- → wontfix
Comment 12•7 years ago
|
||
similar crash. hope this might help. https://crash-stats.mozilla.com/report/index/8b49e133-75a5-4fe1-9e44-9b1872161015
Comment 13•7 years ago
|
||
I'm marking this bug as WORKSFORME as bug crashlog signature didn't appear from a long time (over half year) in Firefox (except some obsolete Fx <50, no crashes starting since Fx 50).
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Resolution: FIXED → WORKSFORME
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•