New issue
Advanced search Search tips

Issue 628377 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug

Blocking:
issue 355145



Sign in to add a comment

CredentialManagerBrowserTest.UpdateViaAPIAndAutofill is flaky on Linux Tests (dbg)

Project Member Reported by wjmaclean@chromium.org, Jul 14 2016

Issue description

Test output

[ RUN      ] CredentialManagerBrowserTest.UpdateViaAPIAndAutofill
Xlib:  extension "RANDR" missing on display ":9".
Xlib:  extension "RANDR" missing on display ":9".
[31507:31507:0714/023235:WARNING:audio_manager.cc(317)] Multiple instances of AudioManager detected
[31507:31507:0714/023235:WARNING:audio_manager.cc(278)] Multiple instances of AudioManager detected
[31507:31507:0714/023236:WARNING:password_store_factory.cc(250)] Using basic (unencrypted) store for password storage. See https://chromium.googlesource.com/chromium/src/+/master/docs/linux_password_storage.md for more information about password storage options.
[31507:31652:0714/023237:WARNING:simple_synchronous_entry.cc(1054)] Could not open platform files for entry.
[31507:31645:0714/023237:WARNING:embedded_test_server.cc(193)] Request not handled. Returning 404: /favicon.ico
../../chrome/browser/password_manager/credential_manager_browsertest.cc:194: Failure
Value of: stored[signin_form.signon_realm][0]
  Actual: PasswordForm({
   "origin": "http://127.0.0.1:46088/",
   "password_value": "12345",
   "preferred": true,
   "signon_realm": "http://127.0.0.1:46088/",
   "username_value": "user"
})
Expected: signin_form
Which is: PasswordForm({
   "origin": "http://127.0.0.1:46088/",
   "password_value": "12345",
   "preferred": true,
   "signon_realm": "http://127.0.0.1:46088/",
   "skip_next_zero_click": false,
   "times_used": 1,
   "username_value": "user"
})
[31507:31507:0714/023238:WARNING:url_request_context_getter.cc(43)] URLRequestContextGetter leaking due to no owning thread.
[  FAILED  ] CredentialManagerBrowserTest.UpdateViaAPIAndAutofill, where TypeParam =  and GetParam() =  (4505 ms)

Status: Started (was: Untriaged)
This looks like a race condition with updating the password store.

However, the second run looks like another bug

[ RUN      ] CredentialManagerBrowserTest.UpdateViaAPIAndAutofill
Xlib:  extension "RANDR" missing on display ":9".
Xlib:  extension "RANDR" missing on display ":9".
[21676:21676:0714/024245:WARNING:audio_manager.cc(317)] Multiple instances of AudioManager detected
[21676:21676:0714/024245:WARNING:audio_manager.cc(278)] Multiple instances of AudioManager detected
[21676:21676:0714/024245:WARNING:password_store_factory.cc(250)] Using basic (unencrypted) store for password storage. See https://chromium.googlesource.com/chromium/src/+/master/docs/linux_password_storage.md for more information about password storage options.
[21676:21729:0714/024246:WARNING:simple_synchronous_entry.cc(1054)] Could not open platform files for entry.
[21676:21725:0714/024247:WARNING:embedded_test_server.cc(193)] Request not handled. Returning 404: /favicon.ico
../../chrome/browser/password_manager/credential_manager_browsertest.cc:207: Failure
Value of: stored[signin_form.signon_realm][0]
  Actual: PasswordForm({
   "action": "http://127.0.0.1:60931/password/done.html",
   "origin": "http://127.0.0.1:60931/",
   "password_elem": "password_field",
   "password_value": "12345",
   "preferred": true,
   "signon_realm": "http://127.0.0.1:60931/",
   "skip_next_zero_click": false,
   "times_used": 2,
   "username_elem": "username_field",
   "username_value": "user"
})
Expected: signin_form
Which is: PasswordForm({
   "origin": "http://127.0.0.1:60931/",
   "password_value": "12345",
   "preferred": true,
   "signon_realm": "http://127.0.0.1:60931/",
   "skip_next_zero_click": false,
   "times_used": 1,
   "username_value": "user"
})
[21676:21676:0714/024247:WARNING:url_request_context_getter.cc(43)] URLRequestContextGetter leaking due to no owning thread.
[  FAILED  ] CredentialManagerBrowserTest.UpdateViaAPIAndAutofill, where TypeParam =  and GetParam() =  (3515 ms)

Comment 3 by vabr@chromium.org, Jul 21 2016

Blocking: 355145
Components: UI>Browser>Passwords
Labels: Hotlist-TechnicalDebt OS-Linux
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cd8c0b41b62492a20846f4068cab2b5309df1834

commit cd8c0b41b62492a20846f4068cab2b5309df1834
Author: vasilii <vasilii@chromium.org>
Date: Thu Jul 21 14:33:37 2016

Fix CredentialManagerBrowserTest.UpdateViaAPIAndAutofill flakiness.

BUG= 628377 , 629459 

Review-Url: https://codereview.chromium.org/2169883002
Cr-Commit-Position: refs/heads/master@{#406845}

[modify] https://crrev.com/cd8c0b41b62492a20846f4068cab2b5309df1834/chrome/browser/password_manager/credential_manager_browsertest.cc

Status: Fixed (was: Started)

Sign in to add a comment