Ensure TabQueueDispatcher has crash reporting enabled

RESOLVED FIXED in Firefox 40

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mhaigh, Assigned: mhaigh)

Tracking

unspecified
Firefox 40
All
Android
Points:
---

Firefox Tracking Flags

(firefox40 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Ensure we call GeckoAppShell.ensureCrashHandling() in the TabQueueDispatcher onCreate method
(Assignee)

Updated

3 years ago
Summary: Ensure TabQueueDispatcher enabled Crash Reporting → Ensure TabQueueDispatcher has crash reporting enabled
(Assignee)

Comment 1

3 years ago
This needs to be enabled as it's now going to be a fairly big entry point in to the app, so if something goes wrong before BrowserApp is opened, we need to be able to capture that data
(Assignee)

Comment 2

3 years ago
Created attachment 8590336 [details] [diff] [review]
Ensure TabQueueDispatcher has crash reporting  enabled
Attachment #8590336 - Flags: review?(rnewman)
Comment on attachment 8590336 [details] [diff] [review]
Ensure TabQueueDispatcher has crash reporting  enabled

Review of attachment 8590336 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/base/tabqueue/TabQueueDispatcher.java
@@ +36,1 @@
>          Intent intent = getIntent();

Oh hey! Take a look at SafeIntent.
Attachment #8590336 - Flags: review?(rnewman) → review+
https://hg.mozilla.org/mozilla-central/rev/8595bcc0458a
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
You need to log in before you can comment on or make changes to this bug.