Remove calls to deprecated MessageLoop methods in jingle.

Remove calls to deprecated MessageLoop methods in jingle.

This CL makes the following replacements in
jingle:

Before               After
----------------------------------------------------------
 x.PostTask()          x.task_runner()->PostTask()
   PostDelayedTask()                    PostDelayedTask()
   ReleaseSoon()                        ReleaseSoon()
   DeleteSoon()                         DeleteSoon()
x->PostTask()         y->task_runner()->PostTask()
   PostDelayedTask()                    PostDelayedTask()
   ReleaseSoon()                        ReleaseSoon()
   DeleteSoon()                         DeleteSoon()

 x.Run()              RunLoop().Run()
 x.RunUntilIdle()     RunLoop().RunUntilIdle()

x->Run()              RunLoop().Run()
x->RunUntilIdle()     RunLoop().RunUntilIdle()
    If |y| isn't MessageLoopForUI::current() or
    MessageLoopForIO::current()

 y.message_loop()->task_runner()
                      y.task_runner()
y->message_loop()->task_runner()
                      y->task_runner()
----------------------------------------------------------

|x| is a base::MessageLoop(ForUI|ForIO) or a pointer to
a base::MessageLoop(ForUI|ForIO). |y| is a base::Thread
or a pointer to a base::Thread.

This CL was generated using the MessageLoopDeprecatedMethods
clang-tidy fix available on the associated bug. Only files
that compile on Mac are affected. Follow-up CLs will make
these replacements for other platforms.

This CL doesn't change code behavior.

BUG=616447
R=sergeyu@chromium.org

Committed: https://crrev.com/83f4a1ae815486cc99b983f4b38191da96b71c8e
Cr-Commit-Position: refs/heads/master@{#401400}

[fdoray, 2016-06-22 15:41:13]

Can you review this CL? Thanks.

[Sergey Ulanov, 2016-06-22 17:42:29]

https://codereview.chromium.org/2082353002/diff/1/jingle/glue/chrome_async_so...
File jingle/glue/chrome_async_socket.cc (right):

https://codereview.chromium.org/2082353002/diff/1/jingle/glue/chrome_async_so...
jingle/glue/chrome_async_socket.cc:123: message_loop->task_runner()->PostTask(
Shouldn't this be ThreadTaskRunnerHandle::get()->PostTask?

[fdoray, 2016-06-22 17:48:06]

PTAnL

https://codereview.chromium.org/2082353002/diff/1/jingle/glue/chrome_async_so...
File jingle/glue/chrome_async_socket.cc (right):

https://codereview.chromium.org/2082353002/diff/1/jingle/glue/chrome_async_so...
jingle/glue/chrome_async_socket.cc:123: message_loop->task_runner()->PostTask(
On 2016/06/22 17:42:29, Sergey Ulanov wrote:
> Shouldn't this be ThreadTaskRunnerHandle::get()->PostTask?

Done. My script wasn't smart enough for this :)

[Sergey Ulanov, 2016-06-22 17:52:15]

LGTM when my comment is addressed

https://codereview.chromium.org/2082353002/diff/20001/jingle/glue/task_pump.cc
File jingle/glue/task_pump.cc (right):

https://codereview.chromium.org/2082353002/diff/20001/jingle/glue/task_pump.c...
jingle/glue/task_pump.cc:30: current_message_loop->task_runner()->PostTask(
ThreadTaskRunnerHandle::Get() here too.

[fdoray, 2016-06-22 18:15:47]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2016-06-22 18:15:48]

The patchset sent to the CQ was uploaded after l-g-t-m from sergeyu@chromium.org
Link to the patchset: https://codereview.chromium.org/2082353002/#ps40001
(title: "use ThreadTaskRunnerHandle")

[fdoray, 2016-06-22 18:15:57]

https://codereview.chromium.org/2082353002/diff/20001/jingle/glue/task_pump.cc
File jingle/glue/task_pump.cc (right):

https://codereview.chromium.org/2082353002/diff/20001/jingle/glue/task_pump.c...
jingle/glue/task_pump.cc:30: current_message_loop->task_runner()->PostTask(
On 2016/06/22 17:52:15, Sergey Ulanov wrote:
> ThreadTaskRunnerHandle::Get() here too.

Done.

[commit-bot: I haz the power, 2016-06-22 18:16:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2082353002/40001

[commit-bot: I haz the power, 2016-06-22 21:06:02]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-22 21:11:43]

Description was changed from

==========
Remove calls to deprecated MessageLoop methods in jingle.

This CL makes the following replacements in
jingle:

Before               After
----------------------------------------------------------
 x.PostTask()          x.task_runner()->PostTask()
   PostDelayedTask()                    PostDelayedTask()
   ReleaseSoon()                        ReleaseSoon()
   DeleteSoon()                         DeleteSoon()
x->PostTask()         y->task_runner()->PostTask()
   PostDelayedTask()                    PostDelayedTask()
   ReleaseSoon()                        ReleaseSoon()
   DeleteSoon()                         DeleteSoon()

 x.Run()              RunLoop().Run()
 x.RunUntilIdle()     RunLoop().RunUntilIdle()

x->Run()              RunLoop().Run()
x->RunUntilIdle()     RunLoop().RunUntilIdle()
    If |y| isn't MessageLoopForUI::current() or
    MessageLoopForIO::current()

 y.message_loop()->task_runner()
                      y.task_runner()
y->message_loop()->task_runner()
                      y->task_runner()
----------------------------------------------------------

|x| is a base::MessageLoop(ForUI|ForIO) or a pointer to
a base::MessageLoop(ForUI|ForIO). |y| is a base::Thread
or a pointer to a base::Thread.

This CL was generated using the MessageLoopDeprecatedMethods
clang-tidy fix available on the associated bug. Only files
that compile on Mac are affected. Follow-up CLs will make
these replacements for other platforms.

This CL doesn't change code behavior.

BUG=616447
R=sergeyu@chromium.org
==========

to

==========
Remove calls to deprecated MessageLoop methods in jingle.

This CL makes the following replacements in
jingle:

Before               After
----------------------------------------------------------
 x.PostTask()          x.task_runner()->PostTask()
   PostDelayedTask()                    PostDelayedTask()
   ReleaseSoon()                        ReleaseSoon()
   DeleteSoon()                         DeleteSoon()
x->PostTask()         y->task_runner()->PostTask()
   PostDelayedTask()                    PostDelayedTask()
   ReleaseSoon()                        ReleaseSoon()
   DeleteSoon()                         DeleteSoon()

 x.Run()              RunLoop().Run()
 x.RunUntilIdle()     RunLoop().RunUntilIdle()

x->Run()              RunLoop().Run()
x->RunUntilIdle()     RunLoop().RunUntilIdle()
    If |y| isn't MessageLoopForUI::current() or
    MessageLoopForIO::current()

 y.message_loop()->task_runner()
                      y.task_runner()
y->message_loop()->task_runner()
                      y->task_runner()
----------------------------------------------------------

|x| is a base::MessageLoop(ForUI|ForIO) or a pointer to
a base::MessageLoop(ForUI|ForIO). |y| is a base::Thread
or a pointer to a base::Thread.

This CL was generated using the MessageLoopDeprecatedMethods
clang-tidy fix available on the associated bug. Only files
that compile on Mac are affected. Follow-up CLs will make
these replacements for other platforms.

This CL doesn't change code behavior.

BUG=616447
R=sergeyu@chromium.org

Committed: https://crrev.com/83f4a1ae815486cc99b983f4b38191da96b71c8e
Cr-Commit-Position: refs/heads/master@{#401400}
==========

[commit-bot: I haz the power, 2016-06-22 21:11:45]

Patchset 3 (id:??) landed as
https://crrev.com/83f4a1ae815486cc99b983f4b38191da96b71c8e
Cr-Commit-Position: refs/heads/master@{#401400}