Merge pull request #44 from react-native-webrtc/add_debug

Add some android debug

Author: manuquentin

README.md

@@ -426,6 +426,14 @@ class RNCallKeepExample extends React.Component {
 
 - On iOS, you should call `setup` each time you want to use callKeep.
 
+## Debug
+
+### Android
+
+```
+adb logcat *:S RNCallKeepModule:V
+```
+
 ## Contributing
 
 Any pull request, issue report and suggestion are highly welcome!

android/src/main/java/io/wazo/callkeep/RNCallKeepModule.java

@@ -27,6 +27,7 @@
 import android.content.pm.ApplicationInfo;
 import android.content.pm.PackageManager;
 import android.net.Uri;
+import android.util.Log;
 import android.os.Build;
 import android.os.Bundle;
 import android.support.annotation.Nullable;
@@ -68,6 +69,7 @@ public class RNCallKeepModule extends ReactContextBaseJavaModule {
     private static final String REACT_NATIVE_MODULE_NAME = "RNCallKeep";
     private static final String[] permissions = { Manifest.permission.READ_PHONE_STATE, Manifest.permission.CALL_PHONE };
 
+    private static final String TAG = "RNCallKeepModule";
     private static TelecomManager telecomManager;
     private static Promise hasPhoneAccountPromise;
     private ReactApplicationContext reactContext;
@@ -110,6 +112,8 @@ public void displayIncomingCall(String number, String callerName) {
             return;
         }
 
+        Log.d(TAG, "displayIncomingCall number: " + number + ", callerName: " + callerName);
+
         Bundle extras = new Bundle();
         Uri uri = Uri.fromParts(PhoneAccount.SCHEME_TEL, number, null);
 
@@ -121,10 +125,12 @@ public void displayIncomingCall(String number, String callerName) {
 
     @ReactMethod
     public void startCall(String number, String callerName) {
-        if (!isConnectionServiceAvailable() || !hasPhoneAccount() || !hasPermissions()) {
+        if (!isConnectionServiceAvailable() || !hasPhoneAccount() || !hasPermissions() || number == null) {
             return;
         }
 
+        Log.d(TAG, "startCall number: " + number + ", callerName: " + callerName);
+
         Bundle extras = new Bundle();
         Uri uri = Uri.fromParts(PhoneAccount.SCHEME_TEL, number, null);
 
@@ -139,6 +145,7 @@ public void startCall(String number, String callerName) {
 
     @ReactMethod
     public void endCall() {
+        Log.d(TAG, "endCall called");
         if (!isConnectionServiceAvailable() || !hasPhoneAccount()) {
             return;
         }
@@ -151,6 +158,8 @@ public void endCall() {
         conn.setDisconnected(new DisconnectCause(DisconnectCause.LOCAL));
         conn.destroy();
         VoiceConnectionService.deinitConnection();
+
+        Log.d(TAG, "endCall executed");
     }
 
     @ReactMethod

android/src/main/java/io/wazo/callkeep/VoiceConnectionService.java

@@ -49,6 +49,7 @@
 public class VoiceConnectionService extends ConnectionService {
     private static Connection connection;
     private static Boolean isAvailable = false;
+    private static final String TAG = "RNCallKeepModule";
 
     public static Connection getConnection() {
         return connection;
@@ -60,6 +61,7 @@ public static void setAvailable(Boolean value) {
 
 
     public static void deinitConnection() {
+        Log.d(TAG, "deinitConnection");
         connection = null;
     }
 
@@ -109,6 +111,7 @@ public void onCallAudioStateChanged(CallAudioState state) {
             @Override
             public void onAnswer() {
                 super.onAnswer();
+                Log.d(TAG, "onAnswer called");
                 if (connection == null) {
                     return;
                 }
@@ -118,6 +121,7 @@ public void onAnswer() {
 
                 sendCallRequestToActivity(ACTION_ANSWER_CALL, null);
                 sendCallRequestToActivity(ACTION_AUDIO_SESSION, null);
+                Log.d(TAG, "onAnswer executed");
             }
 
             @Override
@@ -128,20 +132,24 @@ public void onPlayDtmfTone(char dtmf) {
             @Override
             public void onDisconnect() {
                 super.onDisconnect();
+                Log.d(TAG, "onDisconnect called");
                 if (connection == null) {
                     return;
                 }
 
+
                 connection.setDisconnected(new DisconnectCause(DisconnectCause.LOCAL));
                 connection.destroy();
                 connection = null;
 
                 sendCallRequestToActivity(ACTION_END_CALL, null);
+                Log.d(TAG, "onDisconnect executed");
             }
 
             @Override
             public void onAbort() {
                 super.onAbort();
+                Log.d(TAG, "onAbort called");
                 if (connection == null) {
                     return;
                 }
@@ -150,6 +158,7 @@ public void onAbort() {
                 connection.destroy();
 
                 sendCallRequestToActivity(ACTION_END_CALL, null);
+                Log.d(TAG, "onAbort executed");
             }
 
             @Override
@@ -171,6 +180,7 @@ public void onUnhold() {
             @Override
             public void onReject() {
                 super.onReject();
+                Log.d(TAG, "onReject called");
                 if (connection == null) {
                     return;
                 }
@@ -179,6 +189,7 @@ public void onReject() {
                 connection.destroy();
 
                 sendCallRequestToActivity(ACTION_END_CALL, null);
+                Log.d(TAG, "onReject executed");
             }
         };