Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modify autofill update credential pixel to include backfilled parameter #5392

Open
wants to merge 1 commit into
base: feature/craig/autofill_increase_ratio_complete_credential_saves
Choose a base branch
from

Conversation

CDRussell
Copy link
Member

@CDRussell CDRussell commented Dec 13, 2024

Task/Issue URL: https://app.asana.com/0/1202926619870900/1208976665268815/f

Description

Steps to test this PR

Logcat filter: message~:"Pixel sent: m.autofill.logins.update.password.inline" | message~:"partial save" | message~:"backfill"

Updating password (with backfilling)

  • Add a single login for autofill.me with username=test and set a password for it (deleting any others you have)
  • Visit https://autofill.me/form/login-simple and decline offer to autofill
  • Enter username test
  • Leave password empty, and hit the Login button (to simulate a partial save)
  • Clear the username
  • Enter a different password than what is stored already
  • Hit the Login button
  • Verify in log that for m_autofill_logins_update_password_inline_displayed, backfilled=true
  • Dismiss the dialog, and verify that for m_autofill_logins_update_password_inline_dismissed, backfilled=true
  • Repeat from Clear the username step, and this time accept to update the password. Verify that m_autofill_logins_update_password_inline_confirmed has backfilled=true

Updating password (without backfilling)

  • Add a single login for autofill.me with username=test and set a password for it (deleting any others you have)
  • Visit https://autofill.me/form/login-simple and decline offer to autofill
  • Enter username test
  • Enter a different password than what is stored already
  • Hit the Login button
  • Verify in log that for m_autofill_logins_update_password_inline_displayed, backfilled=false
  • Dismiss the dialog, and verify that for m_autofill_logins_update_password_inline_dismissed, backfilled=false
  • Repeat and this time accept to update the password. Verify that m_autofill_logins_update_password_inline_confirmed has backfilled=false

Copy link
Member Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from 969ed0e to b196a10 Compare December 17, 2024 17:40
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from 0223949 to d147d82 Compare December 17, 2024 17:41
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from b196a10 to 5580d9f Compare December 18, 2024 17:02
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch 3 times, most recently from 276377c to beb9d61 Compare December 18, 2024 17:06
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from 5580d9f to cfea135 Compare December 18, 2024 17:53
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from beb9d61 to 1113c2c Compare December 18, 2024 17:53
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from cfea135 to 469dfbf Compare December 18, 2024 18:06
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from 1113c2c to a6e3dd9 Compare December 18, 2024 18:06
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from 469dfbf to b8badc8 Compare December 19, 2024 10:14
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from a6e3dd9 to 3b38a15 Compare December 19, 2024 10:14
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from b8badc8 to a36b674 Compare December 19, 2024 10:17
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from 3b38a15 to ca2147f Compare December 19, 2024 10:17
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from a36b674 to 16e1970 Compare December 20, 2024 16:42
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from ca2147f to 0264894 Compare December 20, 2024 16:42
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from 16e1970 to 45867a4 Compare January 10, 2025 15:53
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from 0264894 to ec6f3af Compare January 10, 2025 15:53
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from 45867a4 to f3882dc Compare January 13, 2025 15:13
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from ec6f3af to 438ce4c Compare January 13, 2025 15:14
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from f3882dc to 3d3a41e Compare January 13, 2025 16:18
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch 2 times, most recently from f6535fe to 5206a78 Compare January 14, 2025 14:50
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from 3d3a41e to 2e53317 Compare January 15, 2025 11:06
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from 5206a78 to cf5326b Compare January 15, 2025 11:06
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves branch from 2e53317 to a24930b Compare January 15, 2025 11:12
@CDRussell CDRussell force-pushed the feature/craig/autofill_increase_ratio_complete_credential_saves_pixels branch from cf5326b to 44cbd70 Compare January 15, 2025 11:13
@CDRussell CDRussell marked this pull request as ready for review January 15, 2025 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants