content: Remove use of MessageLoopProxy and deprecated MessageLoop APIs

content: Remove use of MessageLoopProxy and deprecated MessageLoop APIs

This patch was mostly autogenerated with
https://codereview.chromium.org/1010073002/.

BUG=465354
TBR=nick@chromium.org

Committed: https://crrev.com/422456f9d53f0bf936a64f21a1463fd0abd3df84
Cr-Commit-Position: refs/heads/master@{#333081}

Committed: https://crrev.com/95082a6a47108c4e20a77cc224172a293065ab05
Cr-Commit-Position: refs/heads/master@{#333112}

[Sami, 2015-06-01 19:19:43]

skyostil@chromium.org changed reviewers:
+ sievers@chromium.org

[Sami, 2015-06-01 19:20:53]

I originally started doing this just for content/browser/, but turns out there
were only a few references left in the rest of content/ so I just folded
everything in here. PTAL (or tell me to split this up).

[no sievers, 2015-06-01 23:30:47]

lgtm

https://codereview.chromium.org/1159623009/diff/1/content/browser/browser_mai...
File content/browser/browser_main_loop.cc (right):

https://codereview.chromium.org/1159623009/diff/1/content/browser/browser_mai...
content/browser/browser_main_loop.cc:715:
base::MessageLoop::current()->task_runner()));
Should this also be base::ThreadTaskRunnerHandle::Get()?
Also you removed the message_loop.h include, so it might not compile.

https://codereview.chromium.org/1159623009/diff/1/content/browser/browser_thr...
File content/browser/browser_thread_impl.cc (right):

https://codereview.chromium.org/1159623009/diff/1/content/browser/browser_thr...
content/browser/browser_thread_impl.cc:41: class BrowserThreadTaskRunner :
public base::SingleThreadTaskRunner {
Not that your patch is affecting that in any way - but funky that browser
threads have two proxies/task runners (one created here and one created from the
MessageLoop itself).

[Sami, 2015-06-03 12:32:28]

skyostil@chromium.org changed reviewers:
+ nick@chromium.org

[Sami, 2015-06-03 14:55:21]

Thanks Daniel!

https://codereview.chromium.org/1159623009/diff/1/content/browser/browser_mai...
File content/browser/browser_main_loop.cc (right):

https://codereview.chromium.org/1159623009/diff/1/content/browser/browser_mai...
content/browser/browser_main_loop.cc:715:
base::MessageLoop::current()->task_runner()));
On 2015/06/01 23:30:47, sievers wrote:
> Should this also be base::ThreadTaskRunnerHandle::Get()?
> Also you removed the message_loop.h include, so it might not compile.

Thanks, fixed.

https://codereview.chromium.org/1159623009/diff/1/content/browser/browser_thr...
File content/browser/browser_thread_impl.cc (right):

https://codereview.chromium.org/1159623009/diff/1/content/browser/browser_thr...
content/browser/browser_thread_impl.cc:41: class BrowserThreadTaskRunner :
public base::SingleThreadTaskRunner {
On 2015/06/01 23:30:47, sievers wrote:
> Not that your patch is affecting that in any way - but funky that browser
> threads have two proxies/task runners (one created here and one created from
the
> MessageLoop itself).

I was wondering about that too. I thought maybe someone wanted more type safety,
but this class isn't used anywhere outside this file :P

[Sami, 2015-06-03 14:55:34]

The CQ bit was checked by skyostil@chromium.org

[Sami, 2015-06-03 14:55:35]

The patchset sent to the CQ was uploaded after l-g-t-m from sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/1159623009/#ps20001
(title: "Review comments.")

[commit-bot: I haz the power, 2015-06-03 14:57:03]

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

[commit-bot: I haz the power, 2015-06-03 15:33:34]

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

[commit-bot: I haz the power, 2015-06-03 15:33:38]

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

[Sami, 2015-06-03 17:55:50]

The CQ bit was checked by skyostil@chromium.org

[Sami, 2015-06-03 17:55:53]

The patchset sent to the CQ was uploaded after l-g-t-m from sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/1159623009/#ps40001
(title: "Build fix.")

[commit-bot: I haz the power, 2015-06-03 17:58:51]

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

[commit-bot: I haz the power, 2015-06-03 18:25:48]

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

[commit-bot: I haz the power, 2015-06-03 18:25:52]

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

[ncarter (slow), 2015-06-03 19:37:03]

[reviewers -= ncarter] sievers's review is good enough for me

[ncarter (slow), 2015-06-03 19:37:28]

nick@chromium.org changed reviewers:
- nick@chromium.org

[ncarter (slow), 2015-06-03 19:37:29]

[reviewers -= ncarter] sievers's review is good enough for me

[Sami, 2015-06-05 14:11:25]

The CQ bit was checked by skyostil@chromium.org

[Sami, 2015-06-05 14:11:26]

The patchset sent to the CQ was uploaded after l-g-t-m from sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/1159623009/#ps80001
(title: "Test build fix.")

[commit-bot: I haz the power, 2015-06-05 14:12:18]

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

[commit-bot: I haz the power, 2015-06-05 14:20:34]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-06-05 14:21:45]

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

[Sami, 2015-06-05 17:15:28]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2015-06-05 17:15:47]

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

[commit-bot: I haz the power, 2015-06-05 17:24:14]

Committed patchset #5 (id:80001)

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

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

[Sami, 2015-06-05 19:40:59]

The CQ bit was checked by skyostil@chromium.org

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

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

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

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-06-05 20:00:30]

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