1370751 - Crash in mozilla::gl::AndroidNativeWindow::~AndroidNativeWindow

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: Toolbar, defect, P3)

Description

[Ting-Yu Chou [:ting] (away)]

This bug was filed from the Socorro interface and is 
report bp-e1d49ee3-f1ac-4a5a-82cd-ba1770170604.
=============================================================
Top #4 of Nightly 20170604100226 on Android, 5 reports from a single installation. But there're 25 reports from 13 installations in the past week.

Comment 1

[Ting-Yu Chou [:ting] (away)]

Any ideas?

Comment 2

[James Willcox (:snorp) ([e-mail]) (he/him)]

Crashing in plugin stuff, so not exactly a high priority, but I'll try to take a look. I don't see an immediate cause for the crash. We guard against trying to destroy a null ANativeWindow, so it must be garbage or in some other bad state.

Comment 3

[Kartikaya Gupta (email:[e-mail])]

Buildid         Count
20170530180455  1
20170531100238  18
20170602100159  2
20170604100226  5
20170606100206  1

So it's from something that landed near the end of May.

Comment 4

[Milan Sreckovic [:milan] (needinfo for best results)]

:snorp, I made it fix-optional for 55, please change it back if you think we should make sure we fix it in 55.

Comment 6

[Calixte Denizet (:calixte)]

The signature "mozilla::gl::AndroidNativeWindow::~AndroidNativeWindow" is ranked #2 and "@0x0 | mozilla::gl::AndroidNativeWindow::~AndroidNativeWindow" is ranked #3 in beta top-crashers.

Comment 7

[Sylvestre Ledru [:Sylvestre]]

Looks like it is skipping, can you help James?

Comment 8

[Calixte Denizet (:calixte)]

In summing the numbers for the 2 signatures, it's ranked #1 in FennecAndroid top-crashers.

Comment 9

[Milan Sreckovic [:milan] (needinfo for best results)]

Jamie, is this something you can help accelerate.

Comment 10

[Sylvestre Ledru [:Sylvestre]]

We are going to build a fennec 55.0.2 next Monday or Tuesday.
Would be great if we could have a fix for this crash.
thanks

Comment 11

[James Willcox (:snorp) ([e-mail]) (he/him)]

These are all from Flash users, which we've killed in 56. Fewer than 1% of them use Flash, so I don't think it's really worth fixing in 55. #1 crash is pretty interesting, regardless! It must be crashing a lot for those people. We could probably flip the pref to turn Flash off by default, but I am not sure that would really help, as they've already explicitly chosen to install Flash via pretty unorthodox means. They'll probably just turn it back on.

Comment 12

[Sylvestre Ledru [:Sylvestre]]

You mean that flash is not the future anymore ?  ;)
OK, I won't cry about that!

For my culture, you found that where?  "Flash™ Version" here ?
https://crash-stats.mozilla.com/signature/?signature=mozilla%3A%3Agl%3A%3AAndroidNativeWindow%3A%3A~AndroidNativeWindow&date=%3E%3D2017-08-04T19%3A07%3A00.000Z&date=%3C2017-08-11T19%3A07%3A00.000Z&_columns=date&_columns=product&_columns=version&_columns=build_id&_columns=platform&_columns=reason&_columns=address&_columns=install_time&_sort=-date&page=1#summary

Comment 13

[Kartikaya Gupta (email:[e-mail])]

This sounds like a wontfix based on the last few comments. Dropping needinfo as it is not needed any more.