[chrome/renderer/media] Replace MessageLoopProxy usage with ThreadTaskRunnerHandle

Replace MessageLoopProxy usage with ThreadTaskRunnerHandle in chrome/renderer/media module.

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

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

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

[anujsharma, 2015-05-04 13:18:59]

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

[anujsharma, 2015-05-04 13:19:00]

PTAL

[miu, 2015-05-05 04:05:10]

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

[miu, 2015-05-05 04:05:11]

https://codereview.chromium.org/1123733002/diff/1/chrome/renderer/media/cast_...
File chrome/renderer/media/cast_ipc_dispatcher.h (right):

https://codereview.chromium.org/1123733002/diff/1/chrome/renderer/media/cast_...
chrome/renderer/media/cast_ipc_dispatcher.h:24: const
scoped_refptr<base::SingleThreadTaskRunner>& io_message_loop);
Should be renamed to io_task_runner or some such.  Also, needs a name change in
similar places in the LOC you've changed.

https://codereview.chromium.org/1123733002/diff/1/chrome/renderer/media/cast_...
File chrome/renderer/media/cast_receiver_session.h (right):

https://codereview.chromium.org/1123733002/diff/1/chrome/renderer/media/cast_...
chrome/renderer/media/cast_receiver_session.h:71: const
scoped_refptr<base::SingleThreadTaskRunner> io_message_loop_proxy_;
ditto: Should be renamed to: io_task_runner_ or some such.  And there are other
instances to search-and-replace...

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

Thanks miu for your comments!

I have made the changes.
Could you plesae take a look.

Thanks!

https://codereview.chromium.org/1123733002/diff/1/chrome/renderer/media/cast_...
File chrome/renderer/media/cast_ipc_dispatcher.h (right):

https://codereview.chromium.org/1123733002/diff/1/chrome/renderer/media/cast_...
chrome/renderer/media/cast_ipc_dispatcher.h:24: const
scoped_refptr<base::SingleThreadTaskRunner>& io_message_loop);
On 2015/05/05 04:05:11, miu wrote:
> Should be renamed to io_task_runner or some such.  Also, needs a name change
in
> similar places in the LOC you've changed.

Done.

https://codereview.chromium.org/1123733002/diff/1/chrome/renderer/media/cast_...
File chrome/renderer/media/cast_receiver_session.h (right):

https://codereview.chromium.org/1123733002/diff/1/chrome/renderer/media/cast_...
chrome/renderer/media/cast_receiver_session.h:71: const
scoped_refptr<base::SingleThreadTaskRunner> io_message_loop_proxy_;
On 2015/05/05 04:05:11, miu wrote:
> ditto: Should be renamed to: io_task_runner_ or some such.  And there are
other
> instances to search-and-replace...

Done.

[miu, 2015-05-05 05:17:28]

The same issue occurs in the other files.  |io_message_loop| and
|io_message_loop_proxy| need to be renamed in the other files as well.

[anujsharma, 2015-05-05 05:51:39]

On 2015/05/05 05:17:28, miu wrote:
> The same issue occurs in the other files.  |io_message_loop| and
> |io_message_loop_proxy| need to be renamed in the other files as well.

@miu 
I have changed s/io_message_loop/io_task_runner/ and
s/io_message_loop_proxy/io_task_runner/ 

Could you please have a look now.

[xhwang, 2015-05-05 16:01:58]

Thanks! looking good overall. Just a few nits.

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
File chrome/renderer/media/cast_ipc_dispatcher.h (right):

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/cast_ipc_dispatcher.h:64: // Message loop on which IPC
calls are driven.
Update this comment as well.

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
File chrome/renderer/media/cast_session.h (right):

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/cast_session.h:14: #include
"base/single_thread_task_runner.h"
Can you forward declare here and include in .cc?

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/cast_session.h:103: // Proxy to the IO message loop.
Update this comment.

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
File chrome/renderer/media/cast_session_delegate.h (right):

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/cast_session_delegate.h:15: #include
"base/single_thread_task_runner.h"
Can you forward declare here?

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
File chrome/renderer/media/chrome_webrtc_log_message_delegate.h (right):

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/chrome_webrtc_log_message_delegate.h:11: #include
"base/single_thread_task_runner.h"
forward declare?

[anujsharma, 2015-05-06 03:42:45]

Thanks xhwang for your comments!!

I have made the changes. Could you please take  a look.

Thanks!!

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
File chrome/renderer/media/cast_ipc_dispatcher.h (right):

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/cast_ipc_dispatcher.h:64: // Message loop on which IPC
calls are driven.
On 2015/05/05 16:01:57, xhwang wrote:
> Update this comment as well.

Done.

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
File chrome/renderer/media/cast_session.h (right):

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/cast_session.h:14: #include
"base/single_thread_task_runner.h"
On 2015/05/05 16:01:57, xhwang wrote:
> Can you forward declare here and include in .cc?

Done.

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/cast_session.h:103: // Proxy to the IO message loop.
On 2015/05/05 16:01:57, xhwang wrote:
> Update this comment.

Done.

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
File chrome/renderer/media/cast_session_delegate.h (right):

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/cast_session_delegate.h:15: #include
"base/single_thread_task_runner.h"
On 2015/05/05 16:01:57, xhwang wrote:
> Can you forward declare here?

Done.

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
File chrome/renderer/media/chrome_webrtc_log_message_delegate.h (right):

https://codereview.chromium.org/1123733002/diff/40001/chrome/renderer/media/c...
chrome/renderer/media/chrome_webrtc_log_message_delegate.h:11: #include
"base/single_thread_task_runner.h"
On 2015/05/05 16:01:58, xhwang wrote:
> forward declare?

Done.

[xhwang, 2015-05-06 04:23:18]

https://codereview.chromium.org/1123733002/diff/60001/chrome/renderer/media/c...
File chrome/renderer/media/chrome_webrtc_log_message_delegate.h (right):

https://codereview.chromium.org/1123733002/diff/60001/chrome/renderer/media/c...
chrome/renderer/media/chrome_webrtc_log_message_delegate.h:16: class
MessageLoopProxy;
SingleThreadTaskRunner?

[anujsharma, 2015-05-06 04:28:49]

Aah my bad :(

Sorry missed it. Thanks for pointing it out.

PTAL

https://codereview.chromium.org/1123733002/diff/60001/chrome/renderer/media/c...
File chrome/renderer/media/chrome_webrtc_log_message_delegate.h (right):

https://codereview.chromium.org/1123733002/diff/60001/chrome/renderer/media/c...
chrome/renderer/media/chrome_webrtc_log_message_delegate.h:16: class
MessageLoopProxy;
On 2015/05/06 04:23:18, xhwang wrote:
> SingleThreadTaskRunner?

Done.

[xhwang, 2015-05-06 05:55:37]

lgtm

Thanks for fixing this!

[anujsharma, 2015-05-06 05:58:59]

On 2015/05/06 05:55:37, xhwang wrote:
> lgtm
> 
> Thanks for fixing this!

Thanks xhwang for lgtm.

[anujsharma, 2015-05-06 05:59:07]

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

[commit-bot: I haz the power, 2015-05-06 05:59:25]

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

[commit-bot: I haz the power, 2015-05-06 06:50:32]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-05-06 06:51:19]

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