Commit c0ec7ec1 authored by rsimha@chromium.org's avatar rsimha@chromium.org

[sync] Re-enable SyncErrorTest.XmppAuthErrorTest

The sync integration test SyncErrorTest.XmppAuthErrorTest was flaky on Mac due to typed_url_data_type_controller.cc not being able to access the history service.

It appears that recent changes have fixed the flakiness, and the test passes with large --gtest_repeat numbers. Enabling this once the trybots are satisfied.

BUG=165328
TEST=SyncErrorTest.XmppAuthErrorTest

Review URL: https://codereview.chromium.org/12252029

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182370 0039d316-1c4b-4281-b951-d872f2087c98
parent e0cdea8c
......@@ -160,13 +160,7 @@ IN_PROC_BROWSER_TEST_F(SyncErrorTest, AuthErrorTest) {
// Trigger an XMPP auth error, and make sure sync treats it like any
// other auth error.
// This has been flaking a lot recently on Mac. http://crbug.com/165328
#if defined(OS_MACOSX)
#define MAYBE_XmppAuthErrorTest DISABLED_XmppAuthErrorTest
#else
#define MAYBE_XmppAuthErrorTest XmppAuthErrorTest
#endif
IN_PROC_BROWSER_TEST_F(SyncErrorTest, MAYBE_XmppAuthErrorTest) {
IN_PROC_BROWSER_TEST_F(SyncErrorTest, XmppAuthErrorTest) {
ASSERT_TRUE(SetupClients()) << "SetupClients() failed.";
TriggerXmppAuthError();
......
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