Remove MessageLoop::current() from chrome_metrics_service_client.cc

Remove MessageLoop::current() from chrome_metrics_service_client.cc

Whenever possible, use Thread/SequencedTaskRunnerHandle::Get() instead
of MessageLoop::current(). Thread/SequencedTaskRunnerHandle::Get() work
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 Thread/SequencedTaskRunnerHandle::Get() instead.
- Watch a file descriptor. Use FileDescriptorWatcher instead.
- Verify that it is possible to post tasks to the current thread.
  Use Thread/SequencedTaskRunnerHandle::IsSet() instead.
- Verify that code runs on a specific thread. Use
  SingleThreadTaskRunner::BelongsToCurrentThread() instead.

BUG=650723
Committed: https://crrev.com/0c755de5d020e417e439174144040713001f5c6b
Cr-Commit-Position: refs/heads/master@{#426200}

[fdoray, 2016-10-12 15:43:19]

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

[commit-bot: I haz the power, 2016-10-12 15:43:30]

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-10-12 16:30:43]

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

[commit-bot: I haz the power, 2016-10-12 16:30:44]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-10-12 17:03:02]

fdoray@chromium.org changed reviewers:
+ rkaplow@chromium.org

[fdoray, 2016-10-12 17:03:02]

PTAL

[rkaplow, 2016-10-13 22:15:04]

lgtm

[fdoray, 2016-10-14 10:24:11]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-10-14 10:24:27]

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-14 10:34:15]

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

[commit-bot: I haz the power, 2016-10-14 10:34:17]

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

[fdoray, 2016-10-14 12:03:33]

fdoray@chromium.org changed reviewers:
+ jochen@chromium.org

[fdoray, 2016-10-14 12:03:34]

+jochen@: PTAL

[jochen (gone - plz use gerrit), 2016-10-19 14:37:12]

lgtm

[fdoray, 2016-10-19 14:38:00]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-10-19 14:38:20]

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-19 15:29:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-21 13:08:41]

Description was changed from

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

Whenever possible, use Thread/SequencedTaskRunnerHandle::Get() instead
of MessageLoop::current(). Thread/SequencedTaskRunnerHandle::Get() work
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 Thread/SequencedTaskRunnerHandle::Get() instead.
- Watch a file descriptor. Use FileDescriptorWatcher instead.
- Verify that it is possible to post tasks to the current thread.
  Use Thread/SequencedTaskRunnerHandle::IsSet() instead.
- Verify that code runs on a specific thread. Use
  SingleThreadTaskRunner::BelongsToCurrentThread() instead.

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

to

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

Whenever possible, use Thread/SequencedTaskRunnerHandle::Get() instead
of MessageLoop::current(). Thread/SequencedTaskRunnerHandle::Get() work
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 Thread/SequencedTaskRunnerHandle::Get() instead.
- Watch a file descriptor. Use FileDescriptorWatcher instead.
- Verify that it is possible to post tasks to the current thread.
  Use Thread/SequencedTaskRunnerHandle::IsSet() instead.
- Verify that code runs on a specific thread. Use
  SingleThreadTaskRunner::BelongsToCurrentThread() instead.

BUG=650723

Committed: https://crrev.com/0c755de5d020e417e439174144040713001f5c6b
Cr-Commit-Position: refs/heads/master@{#426200}
==========

[commit-bot: I haz the power, 2016-10-21 13:08:42]

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