For #3074 - Also report to Socorro

mozilla-mobile · Nov 9, 2018 · b87d356 · b87d356
1 parent 8154a5b
commit b87d356
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/app/src/main/java/org/mozilla/focus/telemetry/CrashReporterWrapper.kt b/app/src/main/java/org/mozilla/focus/telemetry/CrashReporterWrapper.kt
@@ -7,6 +7,7 @@ import android.content.Context
 import android.os.Build
 import mozilla.components.lib.crash.Crash
 import mozilla.components.lib.crash.CrashReporter
+import mozilla.components.lib.crash.service.MozillaSocorroService
 import mozilla.components.lib.crash.service.SentryService
 import org.mozilla.focus.BuildConfig
 import org.mozilla.focus.locale.LocaleManager
@@ -38,7 +39,9 @@ object CrashReporterWrapper {
                 SentryService(
                         context,
                         sentryDsn,
-                        tags = createTags(context))
+                        tags = createTags(context),
+                        sendEventForNativeCrashes = true),
+                MozillaSocorroService(context, "Firefox Focus")
         )).install(context)
 
         onIsEnabledChanged(context)
@@ -61,9 +64,4 @@ object CrashReporterWrapper {
             Locale.getDefault().toLanguageTag()
         }
     }
-
-    fun captureGeckoCrash() {
-//        crashReporter?.submitReport(Crash())
-//        .capture("GeckoSession crashes, opening new session")
-    }
 }
diff --git a/app/src/main/java/org/mozilla/focus/web/GeckoWebViewProvider.kt b/app/src/main/java/org/mozilla/focus/web/GeckoWebViewProvider.kt
@@ -375,7 +375,6 @@ class GeckoWebViewProvider : IWebViewProvider {
 
                 override fun onCrash(session: GeckoSession) {
                     Log.i(TAG, "Crashed, opening new session")
-                    CrashReporterWrapper.captureGeckoCrash()
                     geckoSession.close()
                     geckoSession = createGeckoSession()
                     applySettingsAndSetDelegates()