PasswordGenerationInteractiveTest.AutoSavingGeneratedPassword Flaking on Mac |
|||||
Issue descriptionExample flake: https://logs.chromium.org/v/?s=chromium%2Fbb%2Fchromium.mac%2FMac10.9_Tests%2F51019%2F%2B%2Frecipes%2Fsteps%2Finteractive_ui_tests%2F0%2Flogs%2FPasswordGenerationInteractiveTest.AutoSavingGeneratedPassword%2F0 Adding some password_manager OWNERS and disabling on Mac
,
Dec 4 2017
,
Dec 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c53af170b1fa766febd98ef74103b6a264b8e827 commit c53af170b1fa766febd98ef74103b6a264b8e827 Author: Christopher Lam <calamity@chromium.org> Date: Mon Dec 04 04:16:40 2017 Disable PasswordGenerationInteractiveTest.AutoSavingGeneratedPassword. Flake examples: https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.9%20Tests/builds/51019 https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.9%20Tests/builds/51016 https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/linux-chromeos-rel/builds/3170 TBR=mkwst@chromium.org Bug: 791389 Change-Id: I23a57c249151746ba07b70f36ee7aa4ee8da0aeb Reviewed-on: https://chromium-review.googlesource.com/804905 Commit-Queue: calamity <calamity@chromium.org> Reviewed-by: calamity <calamity@chromium.org> Cr-Commit-Position: refs/heads/master@{#521270} [modify] https://crrev.com/c53af170b1fa766febd98ef74103b6a264b8e827/chrome/browser/password_manager/password_generation_interactive_uitest.cc
,
Dec 4 2017
Changing component and Cc to generation sub-team.
,
Dec 4 2017
I re-enabled it 2 days (https://chromium-review.googlesource.com/c/chromium/src/+/797035). Fix doesn't work :( I am afraid the test should be disabled on all platforms.
,
Aug 13
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a2ce204cd470fc3258528404098577a0a42b3d70 commit a2ce204cd470fc3258528404098577a0a42b3d70 Author: Vasilii Sukhanov <vasilii@chromium.org> Date: Mon Aug 13 09:26:55 2018 Reenable PasswordGenerationInteractiveTest.AutoSavingGeneratedPassword. Bug: 791389 Change-Id: Iea3b6e0f2ec7899e099bd9d7e23017ac4c83e459 Reviewed-on: https://chromium-review.googlesource.com/1170908 Reviewed-by: Vadym Doroshenko <dvadym@chromium.org> Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#582537} [modify] https://crrev.com/a2ce204cd470fc3258528404098577a0a42b3d70/chrome/browser/password_manager/password_generation_interactive_uitest.cc
,
Aug 13
,
Aug 15
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/11cd89f67aa9bdfb2149f98732e7f72e4bb6746d commit 11cd89f67aa9bdfb2149f98732e7f72e4bb6746d Author: Morten Stenshorne <mstensho@chromium.org> Date: Wed Aug 15 11:35:04 2018 Revert "Reenable PasswordGenerationInteractiveTest.AutoSavingGeneratedPassword." This reverts commit a2ce204cd470fc3258528404098577a0a42b3d70. Reason for revert: Sorry, still flaky; see crbug.com/874380 Original change's description: > Reenable PasswordGenerationInteractiveTest.AutoSavingGeneratedPassword. > > Bug: 791389 > Change-Id: Iea3b6e0f2ec7899e099bd9d7e23017ac4c83e459 > Reviewed-on: https://chromium-review.googlesource.com/1170908 > Reviewed-by: Vadym Doroshenko <dvadym@chromium.org> > Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> > Cr-Commit-Position: refs/heads/master@{#582537} TBR=vasilii@chromium.org,dvadym@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 791389, 874380 Change-Id: I1551a9ae472ef3e6645622951bdfcd87f2fb74c7 Reviewed-on: https://chromium-review.googlesource.com/1175761 Reviewed-by: Morten Stenshorne <mstensho@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#583215} [modify] https://crrev.com/11cd89f67aa9bdfb2149f98732e7f72e4bb6746d/chrome/browser/password_manager/password_generation_interactive_uitest.cc
,
Aug 16
,
Aug 16
linux_chromium_asan_rel_ng and win7_chromium_rel_ng. The test timed out. Probably the keystrokes never reached the input field. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by calamity@chromium.org
, Dec 4 2017