Open Bug 1814143 Opened 2 years ago Updated 2 years ago

[Spike] Investigate the need for using the KeyguardManager separately

Categories

(Firefox for Android :: Autofill, task, P3)

All
Android
task

Tracking

()

People

(Reporter: boek, Unassigned)

Details

From github: https://github.com/mozilla-mobile/fenix/issues/19709.

Description

In the BiometricPromptPreferenceFragment and in some places in BaseBrowserFragment, we try to fallback to the KeyguardManager for PIN/password support.

This isn't something that should be needed since we allow PIN/password as a fallback when we make a request through the Biometric support library. The biometric library also abstracts out the correct calls for backward compatibility for devices that do not have support for full support.

Something we could adjust is the code checking for biometric support for DEVICE_CREDENTIAL.

Deliverables

  • Can we remove the complexities of the KeyguardManager from our code base and use the AndroidX library alone? The less code, the better. :)
  • If we do need it, can we hide those within the BiometricPromptFeature so that it can be used in multiple places since we have two separate impls of a similar code flow?

┆Issue is synchronized with this Jira Task

Change performed by the Move to Bugzilla add-on.

Type: defect → task
Severity: -- → N/A
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.