Remove use of deprecated MessageLoop methods in sync.

sync/engine/sync_scheduler_impl.cc

Index: sync/engine/sync_scheduler_impl.cc
diff --git a/sync/engine/sync_scheduler_impl.cc b/sync/engine/sync_scheduler_impl.cc
index e6b5a764875a38454370097925f1dcab1af9e312..668b503475c7aa4476d36e4ca32ed5554221c215 100644
--- a/sync/engine/sync_scheduler_impl.cc
+++ b/sync/engine/sync_scheduler_impl.cc
@@ -14,6 +14,8 @@
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
+#include "base/single_thread_task_runner.h"

[maxbogue, 2016/06/06 16:52:17]

Has it been considered to just include this header inside
thread_task_runner_handle.h instead of forward declaring it, so it doesn't need
to be included everywhere the other one does? Seems somewhat suboptimal, but
maybe I'm missing something.

+#include "base/threading/thread_task_runner_handle.h"
 #include "sync/engine/backoff_delay_provider.h"
 #include "sync/engine/syncer.h"
 #include "sync/protocol/proto_enum_conversions.h"
@@ -746,9 +748,9 @@ void SyncSchedulerImpl::TryCanaryJob() {
 void SyncSchedulerImpl::TrySyncSessionJob() {
   // Post call to TrySyncSessionJobImpl on current thread. Later request for
   // access token will be here.
-  base::MessageLoop::current()->PostTask(FROM_HERE, base::Bind(
-      &SyncSchedulerImpl::TrySyncSessionJobImpl,
-      weak_ptr_factory_.GetWeakPtr()));
+  base::ThreadTaskRunnerHandle::Get()->PostTask(
+      FROM_HERE, base::Bind(&SyncSchedulerImpl::TrySyncSessionJobImpl,
+                            weak_ptr_factory_.GetWeakPtr()));
 }
 
 void SyncSchedulerImpl::TrySyncSessionJobImpl() {