Closed Bug 1398532 Opened 5 years ago Closed 3 years ago

Investigate why Activity Stream home panel (possibly in combination with Photon toolbar changes) makes testSessionOOMSave so crashy

Categories

(Firefox for Android Graveyard :: Activity Stream, defect, P5)

Unspecified
Android
defect

Tracking

(fennec+, firefox57 affected)

RESOLVED WONTFIX
Tracking Status
fennec + ---
firefox57 --- affected

People

(Reporter: JanH, Unassigned)

References

Details

(Whiteboard: [MobileAS])

+++ This bug was initially created as a clone of Bug #1396324 +++

Starting from the last week of August, there's been an increasing number of intermittent crashes in testSessionOOMSave (compare https://bugzilla.mozilla.org/buglist.cgi?chfield=%5BBug%20creation%5D&chfieldfrom=2017-08-01&chfieldto=Now&classification=Client%20Software&product=Firefox%20for%20Android&query_format=advanced&short_desc=%20testSessionOOMSave%20&short_desc_type=allwordssubstr&columnlist=product%2Ccomponent%2Cassigned_to%2Cbug_status%2Cresolution%2Cshort_desc%2Ctarget_milestone%2Copendate%2Cchangeddate&order=opendate%20asc&list_id=13769877).

I'm not 100 % confident my attempt of a regression analysis in https://bugzilla.mozilla.org/attachment.cgi?id=8905240 is absolutely accurate, but it seems like the spike of crashes that started in the last week of August coincided with some changes to the URL bar. Likewise, it seems like the situation on mozilla-central had quietened down somewhat a few days ago, but a try run with my WIP patches for a scrollable URL display in bug 1271998 again caused a marked increase in testSessionOOMSave crashes.

On the other hand the results from bug 1396324 seem to indicate so far that choosing a different default home panel to avoid is a successful strategy to avoid these crashes, so the Activity Stream home panel is somehow involved in these crashes as well.
Adding this to the MobileAS triage too.
Whiteboard: [MobileAS]
The test is fixed (bug 1396324) and we don't know yet if this affects real users – we'll focus on other things first.
tracking-fennec: ? → +
Rank: 2
Priority: -- → P2
[eng triage recommendation] P2: this is still important because it's an issue that might affect users. However, this will involve more investigation, so we won't be actively working on this so we can focus on other bugs that might need uplift.
[triage] Keep rank: it could uncover issues that affect users but we may find those when this hits Beta too. It'd be great to investigate but not important as issues we know affects users.
All open Activity Stream bugs are moving from the whiteboard tag, "[mobileAS]", to the Firefox for Android component, "Activity Stream", so that I can keep better track of these bugs as the new triage owner; I will send out an email shortly with additional details, caveats, etc.
Component: Testing → Activity Stream
Thankfully they're low frequency so far, but we're still getting occasional intermittent crashes in testSessionOOMSave.
[triage] Bulk edit from title: this is a non-critical issue. Please remove priority if you wish this to be re-triaged.
Priority: P2 → P3
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195

Needinfo :susheel if you think this bug should be re-triaged.
Priority: P3 → P5

The leave-open keyword is there and there is no activity for 6 months.
:st3fan, maybe it's time to close this bug?

Flags: needinfo?(sarentz)

The leave-open keyword is there and there is no activity for 6 months.
:st3fan, maybe it's time to close this bug?

Flags: needinfo?(sarentz)
Status: NEW → RESOLVED
Closed: 3 years ago
Flags: needinfo?(sarentz)
Resolution: --- → WONTFIX
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.