base: Remove MessageLoopProxy

base: Remove MessageLoopProxy

BUG=465354
Committed: https://crrev.com/b1f02993d46e10a58aeda7acc299350381046fa6
Cr-Commit-Position: refs/heads/master@{#335277}

[Sami, 2015-06-12 15:52:12]

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

[Sami, 2015-06-12 15:54:00]

This one isn't quite ready to land since some of the cleanup patches are still
in flight.

https://codereview.chromium.org/1180153002/diff/1/base/message_loop/message_l...
File base/message_loop/message_loop_task_runner_unittest.cc (right):

https://codereview.chromium.org/1180153002/diff/1/base/message_loop/message_l...
base/message_loop/message_loop_task_runner_unittest.cc:251: class
MessageLoopTaskRunnerThreadingTest : public testing::Test {
N.B. these were merged in from message_loop_proxy_impl_unittest.cc.

[danakj, 2015-06-12 17:49:19]

LGTM when it works. Nice job!

[Sami, 2015-06-17 18:06:50]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2015-06-17 18:08:47]

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

[commit-bot: I haz the power, 2015-06-17 18:21:24]

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

[commit-bot: I haz the power, 2015-06-17 18:21:24]

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

[Sami, 2015-06-18 00:36:26]

The CQ bit was checked by skyostil@chromium.org

[Sami, 2015-06-18 00:36:26]

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

[commit-bot: I haz the power, 2015-06-18 00:39:42]

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

[commit-bot: I haz the power, 2015-06-18 03:20:44]

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

[commit-bot: I haz the power, 2015-06-18 03:20:45]

Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[Sami, 2015-06-18 05:18:29]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2015-06-18 05:21:15]

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

[commit-bot: I haz the power, 2015-06-18 07:34:15]

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

[commit-bot: I haz the power, 2015-06-18 07:34:16]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)

[Sami, 2015-06-18 15:37:25]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2015-06-18 15:39:20]

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

[commit-bot: I haz the power, 2015-06-18 17:45:04]

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

[commit-bot: I haz the power, 2015-06-18 17:45:06]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)

[Sami, 2015-06-18 17:52:13]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2015-06-18 17:54:10]

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

[sdefresne, 2015-06-18 18:49:08]

On 2015/06/18 at 17:54:10, commit-bot wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1180153002/20001

Can we hold off landing this CL until iOS downstream has removed all its usage
of MessageLoopProxy?

[Sami, 2015-06-18 18:53:26]

The CQ bit was unchecked by skyostil@chromium.org

[Sami, 2015-06-18 18:53:49]

On 2015/06/18 18:49:08, sdefresne wrote:
> On 2015/06/18 at 17:54:10, commit-bot wrote:
> > CQ is trying da patch. Follow status at
> >  https://chromium-cq-status.appspot.com/patch-status/1180153002/20001
> 
> Can we hold off landing this CL until iOS downstream has removed all its usage
> of MessageLoopProxy?

Sure thing. Can you help me figure out a way to do that?

[sdefresne, 2015-06-18 19:14:04]

On 2015/06/18 at 18:53:49, skyostil wrote:
> On 2015/06/18 18:49:08, sdefresne wrote:
> > On 2015/06/18 at 17:54:10, commit-bot wrote:
> > > CQ is trying da patch. Follow status at
> > >  https://chromium-cq-status.appspot.com/patch-status/1180153002/20001
> > 
> > Can we hold off landing this CL until iOS downstream has removed all its
usage
> > of MessageLoopProxy?
> 
> Sure thing. Can you help me figure out a way to do that?

As discussed offline, I'll help refactor downstream code to remove the few usage
of MessageLoopProxy. Will ping this CL (and send it to CQ) when usage have been
removed downstream. Thank you.

[sdefresne, 2015-06-19 15:21:40]

On 2015/06/18 at 19:14:04, sdefresne wrote:
> On 2015/06/18 at 18:53:49, skyostil wrote:
> > On 2015/06/18 18:49:08, sdefresne wrote:
> > > On 2015/06/18 at 17:54:10, commit-bot wrote:
> > > > CQ is trying da patch. Follow status at
> > > >  https://chromium-cq-status.appspot.com/patch-status/1180153002/20001
> > > 
> > > Can we hold off landing this CL until iOS downstream has removed all its
usage
> > > of MessageLoopProxy?
> > 
> > Sure thing. Can you help me figure out a way to do that?
> 
> As discussed offline, I'll help refactor downstream code to remove the few
usage of MessageLoopProxy. Will ping this CL (and send it to CQ) when usage have
been removed downstream. Thank you.

We have a downstream CL to remove our usage of message_loop_proxy.
Sending this CL to the CQ.

[sdefresne, 2015-06-19 15:21:46]

The CQ bit was checked by sdefresne@chromium.org

[commit-bot: I haz the power, 2015-06-19 15:22:12]

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

[commit-bot: I haz the power, 2015-06-19 17:23:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-19 17:23:48]

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