Commit c60fd59c authored by David Roger's avatar David Roger Committed by Commit Bot

Reenable DiceSigninUiUtilTest

The flake should have been fixed. This CL re-enables the tests.

Bug: 1014790
Change-Id: I9873a6d7a9f650f31b6992a7a10dd796ebbbb174
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948433Reviewed-by: default avatarBoris Sazonov <bsazonov@chromium.org>
Commit-Queue: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#720932}
parent f16a97ac
......@@ -356,14 +356,7 @@ TEST_F(DiceSigninUiUtilTest, EnableSyncWithAccountThatNeedsReauth) {
}
}
// TODO(https://crbug.com/1014790): This test is timing out on Mac 10.12
#if defined(OS_MACOSX)
#define MAYBE_EnableSyncForNewAccountWithNoTab \
DISABLED_EnableSyncForNewAccountWithNoTab
#else
#define MAYBE_EnableSyncForNewAccountWithNoTab EnableSyncForNewAccountWithNoTab
#endif
TEST_F(DiceSigninUiUtilTest, MAYBE_EnableSyncForNewAccountWithNoTab) {
TEST_F(DiceSigninUiUtilTest, EnableSyncForNewAccountWithNoTab) {
base::HistogramTester histogram_tester;
base::UserActionTester user_action_tester;
......@@ -392,9 +385,7 @@ TEST_F(DiceSigninUiUtilTest, MAYBE_EnableSyncForNewAccountWithNoTab) {
active_contents->GetVisibleURL());
}
// TODO(https://crbug.com/1014790): This is flaky on several bots.
TEST_F(DiceSigninUiUtilTest,
DISABLED_EnableSyncForNewAccountWithNoTabWithExisting) {
TEST_F(DiceSigninUiUtilTest, EnableSyncForNewAccountWithNoTabWithExisting) {
base::HistogramTester histogram_tester;
base::UserActionTester user_action_tester;
......
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