Add user-journey completion check for importing passwords via desktop sync

[CDRussell]

Task/Issue URL: https://app.asana.com/0/1203822806345703/1207149249611733/f

Description

Steps to test this PR

Suggest logcat filter: message~:"user-journey" message~:"Pixel sent: .*autofill"

Testing the "immediate success" flow

• [ ] Fresh install
• [ ] Visit Passwords screen, then tap the Import Passwords button
• [ ] Verify you see Starting user-journey success clock for import passwords screen in the logs
• [ ] Verify you see m_autofill_logins_import_user_journey_started in the logs
• [ ] Tap Sync with Desktop button
• [ ] Enable sync, and sync with a desktop device
• [ ] Go back to the "Import Passwords" screen. verify logs contain m_autofill_logins_import_success

Testing the timed out failure flow

• [ ] Fresh install
• [ ] Visit Passwords screen, then tap the Import Passwords button
• [ ] Set the system date to >= 3 days into the future
• [ ] Return to the app. Verify the logs contain m_autofill_logins_import_failure

Testing the delayed success flow

• [ ] Fresh install
• [ ] Visit Passwords screen, then tap the Import Passwords button
• [ ] Hit back, and return to the browser activity
• [ ] Go to Settings -> Sync & Backup and setup syncing with a desktop device
• [ ] Background the app and restore it.
• [ ] Verify log contains m_autofill_logins_import_success

Restarting the clock

• [ ] Fresh install
• [ ] Visit Passwords screen, then tap the Import Passwords button
• [ ] Verify the logs contain Starting user-journey success clock for import passwords screen
• [ ] Hit back to return to passwords screen
• [ ] Tap Import Passwords button again
• [ ] Verify logs contain Restarting user-journey success clock for import passwords screen
• [ ] Verify logs contain m_autofill_logins_import_user_journey_restarted
• [ ] Background the app and restore it. Verify logs indicate 48h clock was restarted when you re-entered the import passwords screen, by looking at the time remaining

[CDRussell]

• #4698 👈
• #4692
• #4687
• develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CDRussell and the rest of your teammates on Graphite

[cmonfortep]

"Background the app and restore it. Verify logs indicate 48h clock was restarted when you re-entered the import passwords screen, by looking at the time remaining"

This is the only step that didn't work for me.

[CDRussell]

"Background the app and restore it. Verify logs indicate 48h clock was restarted when you re-entered the import passwords screen, by looking at the time remaining"

This is the only step that didn't work for me.

I'm guessing I know why, and the instructions could have been clearer. Simply returning to the screen won't reset the clock. You have to come in with the Activity being created (basically, through the Import Passwords button/menu again) but doing this won't print the time remaining.

So you have to use the import passwords button again and flip it background/foreground to see the timer back up at 48h. but anyway, was just an extra check and i've ensured it's all good.

[CDRussell]

Merge activity

• Jul 3, 4:57 AM EDT: @CDRussell started a stack merge that includes this pull request via Graphite.
• Jul 3, 5:14 AM EDT: Graphite couldn't merge this pull request because a downstack PR #4692 failed to merge.
• Jul 3, 6:29 AM EDT: @CDRussell started a stack merge that includes this pull request via Graphite.
• Jul 3, 6:30 AM EDT: Graphite rebased this pull request as part of a merge.
• Jul 3, 6:43 AM EDT: @CDRussell merged this pull request with Graphite.