Remove calls to MessageLoop::current() in jingle.

Remove calls to MessageLoop::current() in jingle.

Why?
The fact that there's a MessageLoop on the thread is an
unnecessary implementation detail. When browser threads
are migrated to base/task_scheduler, tasks will no longer
have access to a MessageLoop.

These changes were generated manually.

BUG=616447
R=sergeyu@chromium.org

Committed: https://crrev.com/038016b7d0e4e1356bcc5a01ba9a78ed3f8b9151
Cr-Commit-Position: refs/heads/master@{#401032}

[Sergey Ulanov, 2016-06-20 22:53:25]

https://codereview.chromium.org/2088433002/diff/1/jingle/glue/proxy_resolving...
File jingle/glue/proxy_resolving_client_socket.cc (right):

https://codereview.chromium.org/2088433002/diff/1/jingle/glue/proxy_resolving...
jingle/glue/proxy_resolving_client_socket.cc:153:
CHECK(base::ThreadTaskRunnerHandle::IsSet());
I don't think you need this CHECK(). There is DCHECK in
ThreadTaskRunnerHandle::Get().

https://codereview.chromium.org/2088433002/diff/1/jingle/notifier/base/xmpp_c...
File jingle/notifier/base/xmpp_connection.cc (right):

https://codereview.chromium.org/2088433002/diff/1/jingle/notifier/base/xmpp_c...
jingle/notifier/base/xmpp_connection.cc:89:
CHECK(base::ThreadTaskRunnerHandle::IsSet());
I don't think you need this CHECK(). There is DCHECK in
ThreadTaskRunnerHandle::Get().

[fdoray, 2016-06-21 13:01:51]

PTAnL

https://codereview.chromium.org/2088433002/diff/1/jingle/glue/proxy_resolving...
File jingle/glue/proxy_resolving_client_socket.cc (right):

https://codereview.chromium.org/2088433002/diff/1/jingle/glue/proxy_resolving...
jingle/glue/proxy_resolving_client_socket.cc:153:
CHECK(base::ThreadTaskRunnerHandle::IsSet());
On 2016/06/20 22:53:25, Sergey Ulanov wrote:
> I don't think you need this CHECK(). There is DCHECK in
> ThreadTaskRunnerHandle::Get().

Done. I didn't want to replace a CHECK with a DCHECK without owner approval.

https://codereview.chromium.org/2088433002/diff/1/jingle/notifier/base/xmpp_c...
File jingle/notifier/base/xmpp_connection.cc (right):

https://codereview.chromium.org/2088433002/diff/1/jingle/notifier/base/xmpp_c...
jingle/notifier/base/xmpp_connection.cc:89:
CHECK(base::ThreadTaskRunnerHandle::IsSet());
On 2016/06/20 22:53:25, Sergey Ulanov wrote:
> I don't think you need this CHECK(). There is DCHECK in
> ThreadTaskRunnerHandle::Get().

Done.

[Sergey Ulanov, 2016-06-21 16:49:15]

LGTM. Thanks for the cleanup!

[fdoray, 2016-06-21 16:54:51]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-06-21 16:55:21]

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

[commit-bot: I haz the power, 2016-06-21 17:41:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-21 17:49:58]

Description was changed from

==========
Remove calls to MessageLoop::current() in jingle.

Why?
The fact that there's a MessageLoop on the thread is an
unnecessary implementation detail. When browser threads
are migrated to base/task_scheduler, tasks will no longer
have access to a MessageLoop.

These changes were generated manually.

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

to

==========
Remove calls to MessageLoop::current() in jingle.

Why?
The fact that there's a MessageLoop on the thread is an
unnecessary implementation detail. When browser threads
are migrated to base/task_scheduler, tasks will no longer
have access to a MessageLoop.

These changes were generated manually.

BUG=616447
R=sergeyu@chromium.org

Committed: https://crrev.com/038016b7d0e4e1356bcc5a01ba9a78ed3f8b9151
Cr-Commit-Position: refs/heads/master@{#401032}
==========

[commit-bot: I haz the power, 2016-06-21 17:50:00]

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