Move sync_client.cc away from base::Bind

BUG=714018

Change-Id: I59944400286435c71ed605ba019cd7f2680e5532
Reviewed-on: https://chromium-review.googlesource.com/1174656Reviewed-by: vitaliii <vitaliii@chromium.org>
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#584052}

components/sync/engine/net/network_time_update_callback.h

@@ -19,7 +19,7 @@ namespace syncer {
 // const base::Time& network_time - the new network time.
 // const base::TimeDelta& resolution - how precise the reading is.
 // const base::TimeDelta& latency - the http request's latency.
-using NetworkTimeUpdateCallback = base::Callback<
+using NetworkTimeUpdateCallback = base::RepeatingCallback<
     void(const base::Time&, const base::TimeDelta&, const base::TimeDelta&)>;
 
 }  // namespace syncer

components/sync/tools/sync_client.cc

@@ -395,7 +395,7 @@ int SyncClientMain(int argc, char* argv[]) {
   // HttpPostProviderFactory is removed.
   CancelationSignal factory_cancelation_signal;
   std::unique_ptr<HttpPostProviderFactory> post_factory(new HttpBridgeFactory(
-      context_getter.get(), base::Bind(&StubNetworkTimeUpdateCallback),
+      context_getter.get(), base::BindRepeating(&StubNetworkTimeUpdateCallback),
       &factory_cancelation_signal));
   post_factory->Init(kUserAgent, BindToTrackerCallback());
   // Used only when committing bookmarks, so it's okay to leave this as null.
@@ -427,7 +427,7 @@ int SyncClientMain(int argc, char* argv[]) {
   args.encryptor = &null_encryptor;
   args.unrecoverable_error_handler = WeakHandle<UnrecoverableErrorHandler>();
   args.report_unrecoverable_error_function =
-      base::Bind(LogUnrecoverableErrorContext);
+      base::BindRepeating(LogUnrecoverableErrorContext);
   args.cancelation_signal = &scm_cancelation_signal;
   sync_manager->Init(&args);
   // TODO(akalin): Avoid passing in model parameters multiple times by