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

Issue 8587038: Revert 110450 - Re-enable stress sync integration tests (Closed)

Created:
9 years, 1 month ago by Raghu Simha
Modified:
9 years, 1 month ago
Reviewers:
Raghu Simha
CC:
chromium-reviews, Raghu Simha, ncarter (slow), akalin, tim (not reviewing)
Visibility:
Public.

Description

Revert 110450 - Re-enable stress sync integration tests Reason for revert: Redness on the waterfall. The stress sync integration tests use 10 sync clients, and were originally flaky due to testserver bugginess. Several fixes have gone in since then, and it might be time to re-enable those tests. Note to future Chromium Sheriffs: If these stress tests do fail or time out in future, feel free to revert this patch. BUG=69604 TEST=sync_integration_tests --gtest_filter=ManyClient* Review URL: http://codereview.chromium.org/8568033 TBR=rsimha@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110527

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -3 lines) Patch
M chrome/browser/sync/test/integration/many_client_bookmarks_sync_test.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/sync/test/integration/many_client_passwords_sync_test.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/sync/test/integration/many_client_preferences_sync_test.cc View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
Raghu Simha
9 years, 1 month ago (2011-11-17 18:28:28 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698