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

Unified Diff: chrome/browser/sync/engine/net/syncapi_server_connection_manager.cc

Issue 552157: Fixed race condition with AddressWatchTask and ServerConnectionManager. (Closed)
Patch Set: Created 10 years, 11 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
Index: chrome/browser/sync/engine/net/syncapi_server_connection_manager.cc
diff --git a/chrome/browser/sync/engine/net/syncapi_server_connection_manager.cc b/chrome/browser/sync/engine/net/syncapi_server_connection_manager.cc
index 19981de2b410abfff6d68d82153b7f0532870e81..84c7237f14c3b336de734e63f9195e0f5def978c 100644
--- a/chrome/browser/sync/engine/net/syncapi_server_connection_manager.cc
+++ b/chrome/browser/sync/engine/net/syncapi_server_connection_manager.cc
@@ -63,15 +63,6 @@ bool SyncAPIBridgedPost::Init(const char* path, const string& auth_token,
return true;
}
-SyncAPIServerConnectionManager::~SyncAPIServerConnectionManager() {
- delete post_provider_factory_;
-}
-
-void SyncAPIServerConnectionManager::SetHttpPostProviderFactory(
- HttpPostProviderFactory* factory) {
- if (post_provider_factory_)
- delete post_provider_factory_;
- post_provider_factory_ = factory;
-}
+SyncAPIServerConnectionManager::~SyncAPIServerConnectionManager() {}
tim (not reviewing) 2010/01/26 20:08:45 may as well throw this in .h
} // namespace sync_api
« no previous file with comments | « chrome/browser/sync/engine/net/syncapi_server_connection_manager.h ('k') | chrome/browser/sync/engine/syncapi.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698