[content/browser/media] Replace MessageLoopProxy usage with ThreadTaskRunnerHandle

Replace MessageLoopProxy usage with ThreadTaskRunnerHandle in content/browser/media module.

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

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

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

[anujsharma, 2015-04-30 12:07:42]

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

[anujsharma, 2015-04-30 12:07:42]

PTAL

[xhwang, 2015-04-30 15:54:45]

Thanks! Just a few comments.

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
File content/browser/media/capture/web_contents_audio_input_stream_unittest.cc
(right):

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
content/browser/media/capture/web_contents_audio_input_stream_unittest.cc:11:
#include "base/message_loop/message_loop.h"
Do we still need this?

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
File content/browser/media/capture/web_contents_tracker.h (right):

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
content/browser/media/capture/web_contents_tracker.h:102: // MessageLoop
corresponding to the thread that called Start().
Update the comment.

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
content/browser/media/capture/web_contents_tracker.h:103:
scoped_refptr<base::SingleThreadTaskRunner> message_loop_;
Also update the variable name please.

[anujsharma, 2015-05-04 08:07:56]

Thanks xhwang for your comments!!

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

Thanks!!

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
File content/browser/media/capture/web_contents_audio_input_stream_unittest.cc
(right):

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
content/browser/media/capture/web_contents_audio_input_stream_unittest.cc:11:
#include "base/message_loop/message_loop.h"
On 2015/04/30 15:54:45, xhwang wrote:
> Do we still need this?

Yes, it is required!!

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
File content/browser/media/capture/web_contents_tracker.h (right):

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
content/browser/media/capture/web_contents_tracker.h:102: // MessageLoop
corresponding to the thread that called Start().
On 2015/04/30 15:54:45, xhwang wrote:
> Update the comment.

Done.

https://codereview.chromium.org/1112333002/diff/20001/content/browser/media/c...
content/browser/media/capture/web_contents_tracker.h:103:
scoped_refptr<base::SingleThreadTaskRunner> message_loop_;
On 2015/04/30 15:54:45, xhwang wrote:
> Also update the variable name please.

Done.

[xhwang, 2015-05-04 16:13:03]

lgtm

[anujsharma, 2015-05-04 16:39:54]

On 2015/05/04 16:13:03, xhwang wrote:
> lgtm

Thanks xhwang for lgtm.

[anujsharma, 2015-05-04 16:40:21]

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

[commit-bot: I haz the power, 2015-05-04 16:40:42]

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

[commit-bot: I haz the power, 2015-05-04 17:07:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-05-04 17:07:32]

Try jobs failed on following builders:
  cast_shell_android on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_andr...)

[anujsharma, 2015-05-05 04:29:10]

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

[anujsharma, 2015-05-05 04:29:10]

The patchset sent to the CQ was uploaded after l-g-t-m from xhwang@chromium.org
Link to the patchset: https://codereview.chromium.org/1112333002/#ps80001
(title: "Fixed build break")

[commit-bot: I haz the power, 2015-05-05 04:29:54]

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

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

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-05-05 05:36:43]

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