Remove MessageLoop::current() from browser_context.cc.

Remove MessageLoop::current() from browser_context.cc.

Whenever possible, use ThreadTaskRunnerHandle::Get() instead of
MessageLoop::current(). ThreadTaskRunnerHandle::Get() works within
TaskScheduler while MessageLoop::current() doesn't.

Good reasons to use MessageLoop::current():
- Add destruction, nesting or task observers.
- Run nested loops.

Bad reasons to use MessageLoop::current():
- Post tasks. Use ThreadTaskRunnerHandle::Get() instead.
- Watch a file descriptor. Use FileDescriptorWatcher instead.
- Verify that it is possible to post tasks to the current thread.
  Use ThreadTaskRunnerHandle::IsSet() instead.
- Verify that code runs on a specific thread. Use
  SingleThreadTaskRunner::BelongsToCurrentThread() instead.

BUG=650723
Committed: https://crrev.com/4f155f05fffa8d3a7dc3cba15579415800d5668c
Cr-Commit-Position: refs/heads/master@{#424715}

[fdoray, 2016-10-11 16:34:40]

The CQ bit was checked by [email protected] to run a CQ dry run

[commit-bot: I haz the power, 2016-10-11 16:34:55]

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

[fdoray, 2016-10-11 17:10:18]

[email protected] changed reviewers:
+ [email protected]

[fdoray, 2016-10-11 17:10:18]

PTAL

[commit-bot: I haz the power, 2016-10-11 17:26:48]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2016-10-11 17:26:49]

Dry run: This issue passed the CQ dry run.

[clamy, 2016-10-12 11:22:24]

Thanks! Lgtm.

[fdoray, 2016-10-12 11:23:09]

The CQ bit was checked by [email protected]

[commit-bot: I haz the power, 2016-10-12 11:23:25]

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-10-12 11:29:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-12 11:32:23]

Description was changed from

==========
Remove MessageLoop::current() from browser_context.cc.

Whenever possible, use ThreadTaskRunnerHandle::Get() instead of
MessageLoop::current(). ThreadTaskRunnerHandle::Get() works within
TaskScheduler while MessageLoop::current() doesn't.

Good reasons to use MessageLoop::current():
- Add destruction, nesting or task observers.
- Run nested loops.

Bad reasons to use MessageLoop::current():
- Post tasks. Use ThreadTaskRunnerHandle::Get() instead.
- Watch a file descriptor. Use FileDescriptorWatcher instead.
- Verify that it is possible to post tasks to the current thread.
  Use ThreadTaskRunnerHandle::IsSet() instead.
- Verify that code runs on a specific thread. Use
  SingleThreadTaskRunner::BelongsToCurrentThread() instead.

BUG=650723
==========

to

==========
Remove MessageLoop::current() from browser_context.cc.

Whenever possible, use ThreadTaskRunnerHandle::Get() instead of
MessageLoop::current(). ThreadTaskRunnerHandle::Get() works within
TaskScheduler while MessageLoop::current() doesn't.

Good reasons to use MessageLoop::current():
- Add destruction, nesting or task observers.
- Run nested loops.

Bad reasons to use MessageLoop::current():
- Post tasks. Use ThreadTaskRunnerHandle::Get() instead.
- Watch a file descriptor. Use FileDescriptorWatcher instead.
- Verify that it is possible to post tasks to the current thread.
  Use ThreadTaskRunnerHandle::IsSet() instead.
- Verify that code runs on a specific thread. Use
  SingleThreadTaskRunner::BelongsToCurrentThread() instead.

BUG=650723

Committed: https://crrev.com/4f155f05fffa8d3a7dc3cba15579415800d5668c
Cr-Commit-Position: refs/heads/master@{#424715}
==========

[commit-bot: I haz the power, 2016-10-12 11:32:24]

Patchset 1 (id:??) landed as
https://crrev.com/4f155f05fffa8d3a7dc3cba15579415800d5668c
Cr-Commit-Position: refs/heads/master@{#424715}