[remoting] Replace MessageLoopProxy usage with ThreadTaskRunnerHandle

Replace MessageLoopProxy usage with ThreadTaskRunnerHandle in remoting module.

MessageLoopProxy is deprecated.
This basically does a search and replace:

MessageLoopProxy::current() -> ThreadTaskRunnerHandle::Get().

BUG=391045
Committed: https://crrev.com/bb802938545822678c2ef22719f80f143f3c093e
Cr-Commit-Position: refs/heads/master@{#328413}

[anujsharma, 2015-04-28 13:46:05]

anujk.sharma@samsung.com changed reviewers:
+ sergeyu@chromium.org

[anujsharma, 2015-04-28 13:46:06]

PTAL

[Sergey Ulanov, 2015-04-28 17:03:23]

Thank you for the cleanup!

https://codereview.chromium.org/1113493005/diff/1/remoting/host/heartbeat_sen...
File remoting/host/heartbeat_sender.h (right):

https://codereview.chromium.org/1113493005/diff/1/remoting/host/heartbeat_sen...
remoting/host/heartbeat_sender.h:22: class SingleThreadTaskRunner;
SingleThreadTaskRunner is not used in this file, so you can remove this and also
don't need to include single_thread_task_runner.h . You certainly don't need
both forward-declare it here and include single_thread_task_runner.h at the same
time

https://codereview.chromium.org/1113493005/diff/1/remoting/host/policy_watche...
File remoting/host/policy_watcher_unittest.cc (right):

https://codereview.chromium.org/1113493005/diff/1/remoting/host/policy_watche...
remoting/host/policy_watcher_unittest.cc:71: policy_loader_ = new
policy::FakeAsyncPolicyLoader(message_loop_proxy_);
I think you can replace message_loop_proxy_ here with
base::ThreadTaskRunnerHandle::Get() and then remove message_loop_proxy_. It's
not used anywhere else.

https://codereview.chromium.org/1113493005/diff/1/remoting/protocol/clipboard...
File remoting/protocol/clipboard_thread_proxy.h (right):

https://codereview.chromium.org/1113493005/diff/1/remoting/protocol/clipboard...
remoting/protocol/clipboard_thread_proxy.h:17: class SingleThreadTaskRunner;
Don't need this - it's not used in this file.

[anujsharma, 2015-05-04 12:38:21]

Thanks Sergey for your comments!!

Sorry for the delay in response.

I have made the changes as per your suggestions.
Could you please have a look now.

Thanks!!

https://codereview.chromium.org/1113493005/diff/1/remoting/host/heartbeat_sen...
File remoting/host/heartbeat_sender.h (right):

https://codereview.chromium.org/1113493005/diff/1/remoting/host/heartbeat_sen...
remoting/host/heartbeat_sender.h:22: class SingleThreadTaskRunner;
On 2015/04/28 17:03:22, Sergey Ulanov wrote:
> SingleThreadTaskRunner is not used in this file, so you can remove this and
also
> don't need to include single_thread_task_runner.h . You certainly don't need
> both forward-declare it here and include single_thread_task_runner.h at the
same
> time

Done.

https://codereview.chromium.org/1113493005/diff/1/remoting/host/policy_watche...
File remoting/host/policy_watcher_unittest.cc (right):

https://codereview.chromium.org/1113493005/diff/1/remoting/host/policy_watche...
remoting/host/policy_watcher_unittest.cc:71: policy_loader_ = new
policy::FakeAsyncPolicyLoader(message_loop_proxy_);
On 2015/04/28 17:03:22, Sergey Ulanov wrote:
> I think you can replace message_loop_proxy_ here with
> base::ThreadTaskRunnerHandle::Get() and then remove message_loop_proxy_. It's
> not used anywhere else.

Done.

https://codereview.chromium.org/1113493005/diff/1/remoting/protocol/clipboard...
File remoting/protocol/clipboard_thread_proxy.h (right):

https://codereview.chromium.org/1113493005/diff/1/remoting/protocol/clipboard...
remoting/protocol/clipboard_thread_proxy.h:17: class SingleThreadTaskRunner;
On 2015/04/28 17:03:23, Sergey Ulanov wrote:
> Don't need this - it's not used in this file.

Done.

[Sergey Ulanov, 2015-05-04 20:49:58]

couple more nits.

https://codereview.chromium.org/1113493005/diff/120001/remoting/client/client...
File remoting/client/client_status_logger_unittest.cc (right):

https://codereview.chromium.org/1113493005/diff/120001/remoting/client/client...
remoting/client/client_status_logger_unittest.cc:81:
scoped_refptr<base::SingleThreadTaskRunner> message_loop_proxy_;
This isn't used anywhere and can be removed.

https://codereview.chromium.org/1113493005/diff/120001/remoting/host/host_sta...
File remoting/host/host_status_logger_unittest.cc (right):

https://codereview.chromium.org/1113493005/diff/120001/remoting/host/host_sta...
remoting/host/host_status_logger_unittest.cc:142:
scoped_refptr<base::SingleThreadTaskRunner> message_loop_proxy_;
This is not used anywhere and can be removed.

[anujsharma, 2015-05-05 10:59:05]

Thanks sergey for your comments!!

Fixed the comments!!

Could you please take a look.

Thanks!!

https://codereview.chromium.org/1113493005/diff/120001/remoting/client/client...
File remoting/client/client_status_logger_unittest.cc (right):

https://codereview.chromium.org/1113493005/diff/120001/remoting/client/client...
remoting/client/client_status_logger_unittest.cc:81:
scoped_refptr<base::SingleThreadTaskRunner> message_loop_proxy_;
On 2015/05/04 20:49:58, Sergey Ulanov wrote:
> This isn't used anywhere and can be removed.

Removed!!

https://codereview.chromium.org/1113493005/diff/120001/remoting/host/host_sta...
File remoting/host/host_status_logger_unittest.cc (right):

https://codereview.chromium.org/1113493005/diff/120001/remoting/host/host_sta...
remoting/host/host_status_logger_unittest.cc:142:
scoped_refptr<base::SingleThreadTaskRunner> message_loop_proxy_;
On 2015/05/04 20:49:58, Sergey Ulanov wrote:
> This is not used anywhere and can be removed.

Done.

[Sergey Ulanov, 2015-05-05 18:50:31]

LGTM. Thank you for the cleanup!

[anujsharma, 2015-05-05 19:05:46]

The CQ bit was checked by anujk.sharma@samsung.com

[anujsharma, 2015-05-05 19:06:42]

On 2015/05/05 18:50:31, Sergey Ulanov wrote:
> LGTM. Thank you for the cleanup!

Thanks sergey for lgtm.

[commit-bot: I haz the power, 2015-05-05 19:08:22]

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

[commit-bot: I haz the power, 2015-05-05 21:57:41]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2015-05-05 21:58:34]

Patchset 8 (id:??) landed as
https://crrev.com/bb802938545822678c2ef22719f80f143f3c093e
Cr-Commit-Position: refs/heads/master@{#328413}