search

mozilla-mobile / firefox-ios

Firefox for iOS
Mozilla Public License 2.0
12.1k stars 2.88k forks source link

Remove Deferred usage from Authenticator #18911

Open data-sync-user opened 4 months ago

data-sync-user commented 4 months ago

Prerequisite

This task can be tackled once the needed RustLogins functions have been added on the RustLogins class. See Deferred tasks here.

Context

RustLogins is used as part of the profile object in multiple places throughout the codebase. As a first step to remove Deferred, we can remove it from the high level production code, leaving us with an interface that doesn’t depend on this legacy object.

For this current task, this means adding a using the new completion handlers method added on the RustLogins object through the LoginsProtocol.

Acceptance criteria

Use the new methods on the LoginsProtocol so the Authenticator class is not using Deferred anymore.

Contributor note

This a contributor task, if you plan on working on this ticket please write so as a comment under this task so others can know this is being worked on.

┆Issue is synchronized with this Jira Task

data-sync-user commented 2 months ago

➤ Andrei Bodea commented:

Verified as fixed on v127 (41813) with iPhone 13 Pro (15.7.1).