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

Unified Diff: chrome/browser/sync/engine/syncapi.cc

Issue 549176: Merge 37154 - Fixed race condition with AddressWatchTask and ServerConnection... (Closed) Base URL: svn://svn.chromium.org/chrome/branches/306/src/
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
« no previous file with comments | « chrome/browser/sync/engine/net/syncapi_server_connection_manager.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/engine/syncapi.cc
===================================================================
--- chrome/browser/sync/engine/syncapi.cc (revision 37263)
+++ chrome/browser/sync/engine/syncapi.cc (working copy)
@@ -1086,7 +1086,8 @@
string client_id = user_settings_->GetClientId();
connection_manager_.reset(new SyncAPIServerConnectionManager(
- sync_server_and_path, port, use_ssl, user_agent, client_id));
+ sync_server_and_path, port, use_ssl, user_agent, client_id,
+ post_factory));
// TODO(timsteele): This is temporary windows crap needed to listen for
// network status changes. We should either pump this up to the embedder to
@@ -1117,10 +1118,6 @@
}
#endif
- // Hand over the bridged POST factory to be owned by the connection
- // dir_manager.
- connection_manager()->SetHttpPostProviderFactory(post_factory);
-
// Watch various objects for aggregated status.
allstatus()->WatchConnectionManager(connection_manager());
« no previous file with comments | « chrome/browser/sync/engine/net/syncapi_server_connection_manager.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698