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)

49 Branch
Unspecified
Android
defect

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 "
The line in the top frame of the crash stack was last touched in bug 1244875. Jim, thoughts?
Flags: needinfo?(nchen)
It's actually a Java exception. Would be great if we have STR.
Flags: needinfo?(nchen)
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
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)
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:
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.
Moving ni to esawin since he was the one who last touched the code in bug 1051556.
Flags: needinfo?(nchen) → needinfo?(esawin)
Component: Widget: Android → Keyboards and IME
Product: Core → Firefox for Android
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.
I've only unified the crash signatures in bug 1051556, Jim has the better context for IME.
Flags: needinfo?(esawin) → needinfo?(nchen)
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.
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
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.