Remove calls to deprecated MessageLoop methods in remoting.

Remove calls to deprecated MessageLoop methods in remoting.

This CL makes the following replacements in
remoting:

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=garykac@chromium.org

Committed: https://crrev.com/2ad58be9c95de52a6e6962c83df9099c8084ddae
Cr-Commit-Position: refs/heads/master@{#401392}

[fdoray, 2016-06-22 15:44:14]

Can you review this CL? Thanks.

[Sergey Ulanov, 2016-06-22 17:50:14]

sergeyu@chromium.org changed reviewers:
+ sergeyu@chromium.org

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

lgtm

https://codereview.chromium.org/2082363002/diff/1/remoting/protocol/pseudotcp...
File remoting/protocol/pseudotcp_adapter_unittest.cc (right):

https://codereview.chromium.org/2082363002/diff/1/remoting/protocol/pseudotcp...
remoting/protocol/pseudotcp_adapter_unittest.cc:159:
TCPChannelTester(base::MessageLoop* message_loop,
nit: this can be replaced with scoped_refptr<base::SingleThreadTaskRunner>. But
feel free to keep it as is - I'll fix it later myself.

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

https://codereview.chromium.org/2082363002/diff/1/remoting/protocol/pseudotcp...
File remoting/protocol/pseudotcp_adapter_unittest.cc (right):

https://codereview.chromium.org/2082363002/diff/1/remoting/protocol/pseudotcp...
remoting/protocol/pseudotcp_adapter_unittest.cc:159:
TCPChannelTester(base::MessageLoop* message_loop,
On 2016/06/22 17:50:15, Sergey Ulanov wrote:
> nit: this can be replaced with scoped_refptr<base::SingleThreadTaskRunner>.
But
> feel free to keep it as is - I'll fix it later myself.

Done. Might as well do it now.

[fdoray, 2016-06-22 18:12:54]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2016-06-22 18:12:54]

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/2082363002/#ps40001
(title: "add missing include")

[commit-bot: I haz the power, 2016-06-22 18:13:38]

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

[Sergey Ulanov, 2016-06-22 18:35:22]

https://codereview.chromium.org/2082363002/diff/1/remoting/protocol/pseudotcp...
File remoting/protocol/pseudotcp_adapter_unittest.cc (right):

https://codereview.chromium.org/2082363002/diff/1/remoting/protocol/pseudotcp...
remoting/protocol/pseudotcp_adapter_unittest.cc:159:
TCPChannelTester(base::MessageLoop* message_loop,
On 2016/06/22 18:12:48, fdoray wrote:
> On 2016/06/22 17:50:15, Sergey Ulanov wrote:
> > nit: this can be replaced with scoped_refptr<base::SingleThreadTaskRunner>.
> But
> > feel free to keep it as is - I'll fix it later myself.
> 
> Done. Might as well do it now.

Thanks!

[commit-bot: I haz the power, 2016-06-22 20:36:48]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-22 20:39:22]

Description was changed from

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

This CL makes the following replacements in
remoting:

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=garykac@chromium.org
==========

to

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

This CL makes the following replacements in
remoting:

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=garykac@chromium.org

Committed: https://crrev.com/2ad58be9c95de52a6e6962c83df9099c8084ddae
Cr-Commit-Position: refs/heads/master@{#401392}
==========

[commit-bot: I haz the power, 2016-06-22 20:39:24]

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