VoIP: PR review remarks

matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/call/CallSignalingHandler.kt

@@ -211,7 +211,7 @@ internal class CallSignalingHandler @Inject constructor(private val activeCallHa
             activeCallHandler.getCallWithId(it)
         }
         if (currentCall == null) {
-            Timber.v("Call for content: $this is null")
+            Timber.v("Call with id $callId is null")
         }
         return currentCall
     }

matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/call/MxCallFactory.kt

@@ -41,7 +41,7 @@ internal class MxCallFactory @Inject constructor(
 ) {
 
     fun createIncomingCall(roomId: String, opponentUserId: String, content: CallInviteContent): MxCall? {
-        if (content.callId == null) return null
+        content.callId ?: return null
         return MxCallImpl(
                 callId = content.callId,
                 isOutgoing = false,

vector/src/main/res/layout/item_verification_action.xml

@@ -60,7 +60,7 @@
         app:layout_constraintStart_toStartOf="@+id/itemVerificationActionTitle"
         app:layout_constraintTop_toBottomOf="@+id/itemVerificationActionTitle"
         tools:text="For maximum security, do this in person"
-        />
+        tools:visibility="visible" />
 
     <ImageView
         android:id="@+id/itemVerificationActionIcon"