Remove calls to deprecated MessageLoop methods in chrome.

Remove calls to deprecated MessageLoop methods in chrome.

This CL (hopefully) removes all calls to these methods in chrome:
- MessageLoop::PostTask
- MessageLoop::PostDelayedTask
- MessageLoop::DeleteSoon
- MessageLoop::ReleaseSoon
- MessageLoop::Run
- MessageLoop::RunUntilIdle

BUG=616447
R=sky@chromium.org

Committed: https://crrev.com/aa3834aaf2517ad14f4886ac34195770eb91de2f
Cr-Commit-Position: refs/heads/master@{#417639}

[fdoray, 2016-09-06 20:19:19]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-06 20:20:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-06 20:37:10]

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

[commit-bot: I haz the power, 2016-09-06 20:37:11]

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

[fdoray, 2016-09-07 13:54:38]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 13:54:54]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-07 14:38:43]

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

[commit-bot: I haz the power, 2016-09-07 14:38:44]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-09-07 15:53:59]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 15:54:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-07 16:06:16]

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

[commit-bot: I haz the power, 2016-09-07 16:06:19]

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

[fdoray, 2016-09-07 16:37:26]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 16:38:12]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-07 17:49:10]

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

[commit-bot: I haz the power, 2016-09-07 17:49:11]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-09-08 15:08:55]

PTAL

[sky, 2016-09-08 17:54:33]

LGTM

https://codereview.chromium.org/2315443004/diff/60001/chrome/browser/extensio...
File chrome/browser/extensions/activity_log/activity_log_browsertest.cc (right):

https://codereview.chromium.org/2315443004/diff/60001/chrome/browser/extensio...
chrome/browser/extensions/activity_log/activity_log_browsertest.cc:48: // This
is to exit RunLoop (run_loop.Run()) below
end sentence with '.'. That said, this comment isn't particularly helpful. What
is 'this' in this context? I suggest nuking the comment.

[fdoray, 2016-09-09 14:25:39]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-09 14:25:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[fdoray, 2016-09-09 14:33:51]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-09 14:34:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-09 15:20:14]

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

[commit-bot: I haz the power, 2016-09-09 15:20:15]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-09-09 18:10:42]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2016-09-09 18:10:43]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2315443004/#ps100001
(title: "remove unused include")

[fdoray, 2016-09-09 18:10:51]

https://codereview.chromium.org/2315443004/diff/60001/chrome/browser/extensio...
File chrome/browser/extensions/activity_log/activity_log_browsertest.cc (right):

https://codereview.chromium.org/2315443004/diff/60001/chrome/browser/extensio...
chrome/browser/extensions/activity_log/activity_log_browsertest.cc:48: // This
is to exit RunLoop (run_loop.Run()) below
On 2016/09/08 17:54:33, sky wrote:
> end sentence with '.'. That said, this comment isn't particularly helpful.
What
> is 'this' in this context? I suggest nuking the comment.

Actually, posting a QuitWhenIdleClosure() to the current MessageLoop has the
same effect as running the closure inline and is more complicated. I changed the
code to invoke |quit_when_idle_closure| directly.

[commit-bot: I haz the power, 2016-09-09 18:11:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-09 18:19:25]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-09-09 18:21:05]

Description was changed from

==========
Remove calls to deprecated MessageLoop methods in chrome.

This CL (hopefully) removes all calls to these methods in chrome:
- MessageLoop::PostTask
- MessageLoop::PostDelayedTask
- MessageLoop::DeleteSoon
- MessageLoop::ReleaseSoon
- MessageLoop::Run
- MessageLoop::RunUntilIdle

BUG=616447
R=sky@chromium.org
==========

to

==========
Remove calls to deprecated MessageLoop methods in chrome.

This CL (hopefully) removes all calls to these methods in chrome:
- MessageLoop::PostTask
- MessageLoop::PostDelayedTask
- MessageLoop::DeleteSoon
- MessageLoop::ReleaseSoon
- MessageLoop::Run
- MessageLoop::RunUntilIdle

BUG=616447
R=sky@chromium.org

Committed: https://crrev.com/aa3834aaf2517ad14f4886ac34195770eb91de2f
Cr-Commit-Position: refs/heads/master@{#417639}
==========

[commit-bot: I haz the power, 2016-09-09 18:21:07]

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