 Chromium Code Reviews
 Chromium Code Reviews Issue 
            2029413004:
    Remove use of deprecated MessageLoop methods in mojo.  (Closed)
    
  
    Issue 
            2029413004:
    Remove use of deprecated MessageLoop methods in mojo.  (Closed) 
  | DescriptionRemove use of deprecated MessageLoop methods in mojo.
MessageLoop::PostTask/PostDelayedTask/DeleteSoon/ReleaseSoon
are deprecated. This CL makes the following replacements to
remove some uses of these methods:
"MessageLoop(ForUI|ForIO)::current()->PostTask" ->
  "ThreadTaskRunnerHandle::Get()->PostTask"
"MessageLoop(ForUI|ForIO)::current()->PostDelayedTask" ->
  "ThreadTaskRunnerHandle::Get()->PostDelayedTask"
"MessageLoop(ForUI|ForIO)::current()->DeleteSoon" ->
  "ThreadTaskRunnerHandle::Get()->DeleteSoon"
"MessageLoop(ForUI|ForIO)::current()->ReleaseSoon" ->
  "ThreadTaskRunnerHandle::Get()->ReleaseSoon"
In files where these replacements are made, it adds these includes:
  #include "base/location.h"
  #include "base/single_thread_task_runner.h"
  #include "base/threading/thread_task_runner_handle.h"
And removes this include if it is no longer required:
  #include "base/message_loop/message_loop.h"
Why ThreadTaskRunnerHandle::Get() instead of
MessageLoop::current()->task_runner()?
 - The two are equivalent on threads that run a MessageLoop.
 - MessageLoop::current() doesn't work in base/task_scheduler
   because the scheduler's thread don't run MessageLoops.
   This CL will therefore facilitate the migration of browser
   threads to base/task_scheduler.
Steps to generate this patch:
1. Run message_loop_cleanup.py (see code on the bug).
2. Run tools/sort-headers.py on modified files.
3. Run git cl format.
BUG=616447
R=ben@chromium.org
Committed: https://crrev.com/76147bea5dc613dbc4aa592cbe139d6c2dbd0df9
Cr-Commit-Position: refs/heads/master@{#398108}
   Patch Set 1 #
 Messages
    Total messages: 9 (4 generated)
     | ||||||||||||||||||||||||||||