Closed
Bug 1229690
Opened 9 years ago
Closed 7 years ago
Autophone Talos - Intermittent Crash [@0x0] | Crash [@ libflashplayer.so@0x53a038]
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: bc, Unassigned)
References
Details
(Keywords: crash, intermittent-failure, regression)
Crash Data
Attachments
(4 files)
Beginning with Sunday afternoon 2015-11-29 https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=c336a29d0318 there has been an intermittent crash with Autophone Talos tpn. There are two distinct signatures:
1. libflashplayer.so + 0x53a038
2. 0x0
Due to the intermittent nature of the crash, I don't think the first occurrence is the regressor.
Note that https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=cc0dde3ac2a8
Bug 1206872 - Enable C++ APZ in Fennec nightly builds. r=mfinkle
landed Saturday evening 2015-11-28.
Reporter | ||
Comment 1•9 years ago
|
||
Reporter | ||
Comment 2•9 years ago
|
||
Please note : e5e5e5e5e5e5e5e5, fe01fe01fe01fe01 and 3feccccccccccccd
Reporter | ||
Comment 3•9 years ago
|
||
Comment 4•9 years ago
|
||
I strongly suspect the APZ changes are the cause, Having almost 3 weeks of data with <5 total failures for all inbound pushes, then getting 20-30% failures starting the 29th, makes me weary of APZ. It could be environmental, removing the libflashplayer.so if we can would be nice :) I believe APZ makes the painting area larger, this might allow for us to see more of the page than we normally would in tpn which could cause us to load flash elements.
Reporter | ||
Comment 6•9 years ago
|
||
We have plans to only install the flash apk for the robocop test which actually tests flash.
Comment 7•9 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #5)
> I agree with comment 4. Is there a need to keep libflashplayer.so?
Yes, we added it because we were missing regressions that affected a large portion of our users
Untracking, as this will be worked around in autophone.
tracking-fennec: ? → ---
Comment 9•9 years ago
|
||
I am not clear here- are we going with removing flashplayer by default and only using it for the specific robocop tests that need it? I could have misread :blassey's post in comment 7.
Comment 11•9 years ago
|
||
So my investigation in bug 1229118 revealed that for tp4m at least we weren't using a displayport before the C++ APZ landed, and that might be the case for autophone as well. If so, then the APZ code will almost definitely increase the displayport and cause things to get painted that weren't getting painted before, and that might trigger flash crashes. I can help diagnose what happened using try builds, if you're able to run the try builds on autophone. However if it turns out to be a result of the larger displayport we should evaluate whether we want to keep the new behaviour (which is more representative of what a user would experience) or artificially truncate the displayport.
Comment 12•7 years ago
|
||
Flash is going away (bug 1381916), so there's no point in pursuing this further.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•4 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
•