Remove button from UnlockActivity

briar-android/src/main/java/org/briarproject/briar/android/login/UnlockActivity.java

@@ -10,7 +10,6 @@ import android.os.Bundle;
 import android.os.CancellationSignal;
 import android.support.annotation.Nullable;
 import android.support.annotation.RequiresApi;
-import android.widget.Button;
 import android.widget.Toast;
 
 import org.briarproject.bramble.api.nullsafety.MethodsNotNullByDefault;
@@ -25,8 +24,6 @@ import java.util.logging.Logger;
 import javax.inject.Inject;
 
 import static android.hardware.biometrics.BiometricPrompt.BIOMETRIC_ERROR_CANCELED;
-import static android.hardware.biometrics.BiometricPrompt.BIOMETRIC_ERROR_LOCKOUT;
-import static android.hardware.biometrics.BiometricPrompt.BIOMETRIC_ERROR_LOCKOUT_PERMANENT;
 import static android.hardware.biometrics.BiometricPrompt.BIOMETRIC_ERROR_USER_CANCELED;
 import static android.os.Build.VERSION.SDK_INT;
 import static android.view.View.INVISIBLE;
@@ -58,15 +55,10 @@ public class UnlockActivity extends BaseActivity {
 		overridePendingTransition(0, 0);
 		setContentView(R.layout.activity_unlock);
 
-		Button button = findViewById(R.id.unlock);
-		button.setOnClickListener(view -> requestUnlock());
-
 		if (!hasUsableFingerprint(this)) {
 			getWindow().setBackgroundDrawable(null);
-			button.setVisibility(INVISIBLE);
 			findViewById(R.id.image).setVisibility(INVISIBLE);
 		}
-
 		keyguardShown = state != null && state.getBoolean(KEYGUARD_SHOWN);
 	}
 
@@ -142,21 +134,19 @@ public class UnlockActivity extends BaseActivity {
 						errorCode == BIOMETRIC_ERROR_USER_CANCELED) {
 					finish();
 				}
-				// locked out due to 5 failed attempts, lasts for 30 seconds
-				else if (errorCode == BIOMETRIC_ERROR_LOCKOUT ||
-						errorCode == BIOMETRIC_ERROR_LOCKOUT_PERMANENT) {
+				// e.g. 5 failed attempts
+				else {
 					if (hasKeyguardLock(UnlockActivity.this)) {
 						requestKeyguardUnlock();
-					} else if (errString != null) {
+					} else {
 						// normally fingerprints require a screen lock, but
 						// who knows if that's true for all devices out there
-						Toast.makeText(UnlockActivity.this, errString,
-								Toast.LENGTH_LONG).show();
+						if (errString != null) {
+							Toast.makeText(UnlockActivity.this, errString,
+									Toast.LENGTH_LONG).show();
+						}
 						finish();
 					}
-				} else if (errString != null) {
-					Toast.makeText(UnlockActivity.this, errString,
-							Toast.LENGTH_LONG).show();
 				}
 			}
 

briar-android/src/main/res/layout/activity_unlock.xml

@@ -14,22 +14,11 @@
 		android:layout_height="150dp"
 		android:layout_margin="@dimen/margin_large"
 		android:src="@drawable/splash_screen"
-		app:layout_constraintBottom_toTopOf="@+id/unlock"
+		app:layout_constraintBottom_toBottomOf="parent"
 		app:layout_constraintEnd_toEndOf="parent"
 		app:layout_constraintStart_toStartOf="parent"
 		app:layout_constraintTop_toTopOf="parent"
 		app:layout_constraintVertical_bias="0.1"
 		app:tint="?attr/colorControlNormal"/>
 
-	<Button
-		android:id="@+id/unlock"
-		style="@style/BriarButton"
-		android:layout_width="match_parent"
-		android:layout_height="wrap_content"
-		android:layout_margin="@dimen/margin_large"
-		android:text="@string/lock_unlock"
-		app:layout_constraintBottom_toBottomOf="parent"
-		app:layout_constraintEnd_toEndOf="parent"
-		app:layout_constraintStart_toStartOf="parent"/>
-
 </android.support.constraint.ConstraintLayout>
\ No newline at end of file