Commit 0a31cbaa authored by Jan Krcal's avatar Jan Krcal Committed by Commit Bot

Reenable TwoClientAppsSyncTest.StartWithDifferentApps on Win

This CL re-enables a previously flaky test on Windows. The underlying
cause has been fixed so let's give it a try if it stops flaking.

Bug: 434438
Change-Id: I09a85811db998960711586007005853ca4bb5499
Reviewed-on: https://chromium-review.googlesource.com/c/1304475Reviewed-by: default avatarMarc Treib <treib@chromium.org>
Commit-Queue: Jan Krcal <jkrcal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#603537}
parent ad60fee0
......@@ -6,7 +6,6 @@
#include "base/macros.h"
#include "base/strings/utf_string_conversions.h"
#include "build/build_config.h"
#include "chrome/browser/chrome_notification_types.h"
#include "chrome/browser/extensions/bookmark_app_helper.h"
#include "chrome/browser/extensions/extension_service.h"
......@@ -95,13 +94,7 @@ IN_PROC_BROWSER_TEST_P(TwoClientAppsSyncTest, E2E_ENABLED(StartWithSameApps)) {
// Install some apps on both clients, some on only one client, some on only the
// other, and sync. Both clients should end up with all apps, and the app and
// page ordinals should be identical.
// Flakily crashes on Windows, see crbug.com/434438.
#if defined(OS_WIN)
#define MAYBE_StartWithDifferentApps DISABLED_StartWithDifferentApps
#else
#define MAYBE_StartWithDifferentApps StartWithDifferentApps
#endif
IN_PROC_BROWSER_TEST_P(TwoClientAppsSyncTest, MAYBE_StartWithDifferentApps) {
IN_PROC_BROWSER_TEST_P(TwoClientAppsSyncTest, StartWithDifferentApps) {
ASSERT_TRUE(SetupClients());
int i = 0;
......
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