Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4252)

Unified Diff: chrome/browser/ui/webui/sync_setup_browsertest.js

Issue 2867643003: Disabling SyncSetupWebUITestAsync.RestoreSyncDataTypes on Mac: flaky (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/sync_setup_browsertest.js
diff --git a/chrome/browser/ui/webui/sync_setup_browsertest.js b/chrome/browser/ui/webui/sync_setup_browsertest.js
index b3654ce11f4569472307af3faed4417767c0c88d..578dfd0b417ad5fe55ab90f4f6ed0fc900501112 100644
--- a/chrome/browser/ui/webui/sync_setup_browsertest.js
+++ b/chrome/browser/ui/webui/sync_setup_browsertest.js
@@ -103,9 +103,9 @@ TEST_F('SyncSetupWebUITestAsync', 'DISABLED_VerifySignIn', function() {
this.startSyncing();
});
-// This test is flaky on Linux and Windows bots. See crbug.com/579666 and
-// crbug.com/638884
-GEN('#if defined(OS_LINUX) || defined(OS_WIN)');
+// This test is flaky on Linux, Windows and Mac bots. See crbug.com/579666,
+// crbug.com/638884 and crbug.com/718947.
+GEN('#if defined(OS_LINUX) || defined(OS_WIN) || defined(OS_MACOSX)');
GEN('#define MAYBE_RestoreSyncDataTypes DISABLED_RestoreSyncDataTypes');
GEN('#else');
GEN('#define MAYBE_RestoreSyncDataTypes RestoreSyncDataTypes');
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698