Commit b2588dbf authored by Mikel Astiz's avatar Mikel Astiz Committed by Commit Bot

Reenable test in TwoClientPasswordsSyncTest

Recent patches have fixed the observed DCHECK issue, so the flakiness
is believed to be gone.

Bug: 873494
Change-Id: I9821f879b0be1c0ffb11004be2ae69182b6fd342
Reviewed-on: https://chromium-review.googlesource.com/c/1347357Reviewed-by: default avatarJan Krcal <jkrcal@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610346}
parent ce0c2d1f
...@@ -99,14 +99,8 @@ IN_PROC_BROWSER_TEST_P(TwoClientPasswordsSyncTest, E2E_ENABLED(Race)) { ...@@ -99,14 +99,8 @@ IN_PROC_BROWSER_TEST_P(TwoClientPasswordsSyncTest, E2E_ENABLED(Race)) {
ASSERT_TRUE(SamePasswordFormsChecker().Wait()); ASSERT_TRUE(SamePasswordFormsChecker().Wait());
} }
// Disabled due to flakiness on Chrome OS: https://crbug.com/873494.
#if defined(OS_CHROMEOS)
#define MAYBE_SetPassphraseAndAddPassword DISABLED_SetPassphraseAndAddPassword
#else
#define MAYBE_SetPassphraseAndAddPassword SetPassphraseAndAddPassword
#endif
IN_PROC_BROWSER_TEST_P(TwoClientPasswordsSyncTest, IN_PROC_BROWSER_TEST_P(TwoClientPasswordsSyncTest,
E2E_ENABLED(MAYBE_SetPassphraseAndAddPassword)) { E2E_ENABLED(SetPassphraseAndAddPassword)) {
ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
GetSyncService(0)->SetEncryptionPassphrase(kValidPassphrase); GetSyncService(0)->SetEncryptionPassphrase(kValidPassphrase);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment