Closed Bug 1086994 Opened 9 years ago Closed 9 years ago

Enable crash handling in search activity


(Firefox for Android Graveyard :: General, defect)

Not set


(firefox35 wontfix, firefox36 fixed, fennec35+)

Firefox 36
Tracking Status
firefox35 --- wontfix
firefox36 --- fixed
fennec 35+ ---


(Reporter: jchen, Assigned: jchen)



(1 file)

We should turn on crash handling in search activity, which can run independently of Gecko.
SearchActivity already has a dependency on GeckoAppShell through Telemetry. Because GeckoAppShell automatically registers a crash handler when it's initialized, we can just rely on it to handle search activity crashes.

This patch simply adds a stub method in GeckoAppShell, so that we explicitly load the class on search activity creation (instead of implicitly loading it through Telemetry).
Attachment #8509045 - Flags: review?(margaret.leibovic)
Comment on attachment 8509045 [details] [diff] [review]
Enable crash handling in search activity (v1)

Review of attachment 8509045 [details] [diff] [review]:

Sounds reasonable to me.
Attachment #8509045 - Flags: review?(margaret.leibovic) → review+
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Let's uplift this to 35.
tracking-fennec: --- → 35+
Comment on attachment 8509045 [details] [diff] [review]
Enable crash handling in search activity (v1)

Approval Request Comment
[Feature/regressing bug #]: search activity
[User impact if declined]: we might miss crash reports from search activity
[Describe test coverage new/current, TBPL]: no automated tests, baked on m-c
[Risks and why]: low-risk, small change to make sure crash handled is initialized when search activity is created
[String/UUID change made/needed]: none
Attachment #8509045 - Flags: approval-mozilla-aurora?
Attachment #8509045 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(In reply to [Away 18-Nov to 23-Nov] Ryan VanderMeulen [:RyanVM UTC-5] from comment #8)
> Backed out for bustage. I'm guessing this needs bug 1043457 to be uplifted
> as well?
> aurora

Oh, I'm sorry, I should have looked at the dependencies more closely. If the crash handling changes aren't in 35, I don't think we need to uplift this change as well. I just assumed that this should be uplifted as part of the search activity feature, which is new in 35.
Setting Fx35 to wontfix per comment 9. Feel free to change that back if there's disagreement.
Attachment #8509045 - Flags: approval-mozilla-aurora+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.