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

Unified Diff: trunk/src/sync/internal_api/syncapi_server_connection_manager.cc

Issue 23658030: Revert 222154 "sync: Gracefully handle very early shutdown" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 3 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: trunk/src/sync/internal_api/syncapi_server_connection_manager.cc
===================================================================
--- trunk/src/sync/internal_api/syncapi_server_connection_manager.cc (revision 222204)
+++ trunk/src/sync/internal_api/syncapi_server_connection_manager.cc (working copy)
@@ -92,23 +92,17 @@
int port,
bool use_ssl,
bool use_oauth2_token,
- HttpPostProviderFactory* factory,
- CancelationSignal* cancelation_signal)
- : ServerConnectionManager(server,
- port,
- use_ssl,
- use_oauth2_token,
- cancelation_signal),
+ HttpPostProviderFactory* factory)
+ : ServerConnectionManager(server, port, use_ssl, use_oauth2_token),
post_provider_factory_(factory) {
DCHECK(post_provider_factory_.get());
}
SyncAPIServerConnectionManager::~SyncAPIServerConnectionManager() {}
-scoped_ptr<ServerConnectionManager::Connection>
+ServerConnectionManager::Connection*
SyncAPIServerConnectionManager::MakeConnection() {
- return scoped_ptr<Connection>(
- new SyncAPIBridgedConnection(this, post_provider_factory_.get()));
+ return new SyncAPIBridgedConnection(this, post_provider_factory_.get());
}
} // namespace syncer

Powered by Google App Engine
This is Rietveld 408576698