From a1c62688a8dc650f9a564e9cd86d9f26ec890d8d Mon Sep 17 00:00:00 2001 From: "rsimha@chromium.org" Date: Tue, 7 Sep 2010 22:43:05 +0000 Subject: Revert 58768 - Adding the sync-notification-host command line parameter to the integration tests. The sync integration tests currently access the network to communicate with the notification server. We need to move to a model where all server accesses are stubbed out by local servers. This is the first step towards implementing a local notification server. BUG=53933,53931 TEST=sync_integration_tests Review URL: http://codereview.chromium.org/3326007 TBR=rsimha@chromium.org Review URL: http://codereview.chromium.org/3318016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58769 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/test/live_sync/live_sync_test.cc | 5 ----- 1 file changed, 5 deletions(-) (limited to 'chrome/test/live_sync/live_sync_test.cc') diff --git a/chrome/test/live_sync/live_sync_test.cc b/chrome/test/live_sync/live_sync_test.cc index 5191855..5ff3e74 100644 --- a/chrome/test/live_sync/live_sync_test.cc +++ b/chrome/test/live_sync/live_sync_test.cc @@ -123,8 +123,6 @@ void LiveSyncTest::SetUp() { "transitional"); } - // TODO(akalin): Delete this block of code once a local python notification - // server is implemented. // The chrome sync builders are behind a firewall that blocks port 5222, the // default port for XMPP notifications. This causes the tests to spend up to a // minute waiting for a connection on port 5222 before they fail over to port @@ -251,9 +249,6 @@ void LiveSyncTest::SetUpLocalTestServer() { StringPrintf("http://%s:%d/chromiumsync", test_server_.host_port_pair().host().c_str(), test_server_.host_port_pair().port())); - - // TODO(akalin): Set the kSyncNotificationHost switch here once a local python - // notification server is implemented. } void LiveSyncTest::TearDownLocalTestServer() { -- cgit v1.1