New issue
Advanced search Search tips

Issue 789917 link

Starred by 1 user
Status: Available
Owner: ----
Cc:
dvadym@chromium.org
kolos@chromium.org
vasi...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[Password Manager] PasswordManagerBrowserTestBase.ChangePwdWhenTheFormContainNotUsernameTextfield should simulate key presses instead of Javascript modifying

Project Member Reported by kolos@chromium.org, Nov 30 2017 
Since GetPasswordForm consider user input for fields filtering and classification, this test should simulate key presses.

Otherwise, autofilled value in old password field will be consider user input, but Javascript modified values (new password and confirmation fields) will be ignored as they don't have user input.

Comment 1 by kolos@chromium.org, Nov 30 2017

Cc: vasi...@chromium.org
Vasilii: is it fine to use content::SimulateKeyPress in password_manager_browsertest.cc? Or shall we move the test to the interactive tests?

Comment 2 by vasi...@chromium.org, Nov 30 2017 

You can try. There is a chance that the test will be flaky due to real focus events. Then we could move it to interactive tests.
Project Member

Comment 3 by sheriffbot@chromium.org, Dec 3

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 4 by vasi...@chromium.org, Dec 4

Status: Available (was: Untriaged)
►

Sign in to add a comment