Add Crashlytics tracking

Author: niftynei

app/build.gradle

@@ -1,4 +1,19 @@
+buildscript {
+    repositories {
+        maven { url 'https://maven.fabric.io/public' }
+    }
+
+    dependencies {
+        classpath 'io.fabric.tools:gradle:1.+'
+    }
+}
 apply plugin: 'com.android.application'
+apply plugin: 'io.fabric'
+
+repositories {
+    maven { url 'https://maven.fabric.io/public' }
+}
+
 
 android {
     compileSdkVersion 24
@@ -27,12 +42,15 @@ android {
         debug {
             applicationIdSuffix '.dev'
             manifestPlaceholders = [ permissionPackage : "com.humbughq.mobile.dev"]
+            manifestPlaceholders += [ fabricKey : ""]
+            ext.enableCrashlytics = false
             buildConfigField "String", "GOOGLE_CLIENT_ID", "\"659545034376-78hg5hmrvqr50nn53t0nme97ndhqjmvs.apps.googleusercontent.com\""
         }
         release {
             minifyEnabled false
             proguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-rules.txt'
             manifestPlaceholders = [ permissionPackage : "com.humbughq.mobile"]
+            manifestPlaceholders += [ fabricKey : project.property("fabricKey")]
             buildConfigField "String", "GOOGLE_CLIENT_ID", "TODO: FILL THIS IN"
         }
     }
@@ -71,6 +89,9 @@ dependencies {
     compile 'com.squareup.picasso:picasso:2.5.2'
     testCompile 'junit:junit:4.12'
     compile 'com.squareup.okhttp3:okhttp:3.4.1'
+    compile('com.crashlytics.sdk.android:crashlytics:2.6.2@aar') {
+        transitive = true;
+    }
 }
 
 apply plugin: 'com.google.gms.google-services'

app/libs/crashlytics.jar

@@ -62,14 +62,14 @@
 
         <service android:name=".gcm.GcmIntentService" />
 
-        <meta-data
-            android:name="com.crashlytics.ApiKey"
-            android:value="7c523eb4efdbd264d6d4a7403ee7a683b733a9bd" />
         <meta-data
             android:name="com.google.android.gms.version"
             android:value="@integer/google_play_services_version" />
 
         <activity android:name=".activities.DevAuthActivity" />
+        <meta-data
+            android:name="io.fabric.ApiKey"
+            android:value="${fabricKey}" />
     </application>
 
 </manifest>

app/src/main/AndroidManifest.xml

@@ -1,5 +1,7 @@
 package com.zulip.android;
 
+import com.crashlytics.android.Crashlytics;
+import io.fabric.sdk.android.Fabric;
 import java.io.IOException;
 import java.sql.SQLException;
 import java.util.Map;
@@ -82,6 +84,7 @@ public static ZulipApp get() {
     @Override
     public void onCreate() {
         super.onCreate();
+        Fabric.with(this, new Crashlytics());
         ZulipApp.setInstance(this);
 
         // This used to be from HumbugActivity.getPreferences, so we keep that
@@ -94,9 +97,6 @@ public void onCreate() {
         lastEventId = settings.getInt("lastEventId", -1);
         pointer = settings.getInt("pointer", -1);
 
-        if (BuildHelper.shouldLogToCrashlytics()) {
-            // TODO(lfaraone): figure out what to do about crash reporting
-        }
 
         this.api_key = settings.getString(API_KEY, null);
 

app/src/main/java/com/zulip/android/ZulipApp.java

@@ -2,8 +2,11 @@
 
 import android.util.Log;
 
+import com.crashlytics.android.Crashlytics;
 import com.zulip.android.util.BuildHelper;
 
+import io.fabric.sdk.android.Fabric;
+
 public class ZLog {
 
     private ZLog() {
@@ -13,7 +16,7 @@ public static void logException(Throwable e) {
         if (!BuildHelper.shouldLogToCrashlytics()) {
             Log.e("Error", "oops", e);
         } else {
-            // TODO(lfaraone): figure out what to do about crash reporting
+            Crashlytics.logException(e);
         }
     }