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

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

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=agl@chromium.org

Committed: https://crrev.com/392099634d1ca439a1365569f558a5661af17a0d
Cr-Commit-Position: refs/heads/master@{#402805}

[fdoray, 2016-06-22 18:54:48]

Can you review this CL? Thanks.

[fdoray, 2016-06-28 21:08:48]

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

[fdoray, 2016-06-28 21:08:49]

+gavinp@chromium.org: Can you review this CL? Thanks.

[agl, 2016-06-28 22:59:44]

LGTM.

sorry, missed this email.

https://codereview.chromium.org/2082773002/diff/1/net/quic/quic_connection_lo...
File net/quic/quic_connection_logger.h (right):

https://codereview.chromium.org/2082773002/diff/1/net/quic/quic_connection_lo...
net/quic/quic_connection_logger.h:14: #include "base/metrics/histogram_base.h"
Could HistogramBase just be forward declared since all the uses appear to be
pointers?

[fdoray, 2016-06-29 12:32:13]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2016-06-29 12:32:17]

The CQ bit was unchecked by fdoray@chromium.org

[fdoray, 2016-06-29 13:32:38]

https://codereview.chromium.org/2082773002/diff/1/net/quic/quic_connection_lo...
File net/quic/quic_connection_logger.h (right):

https://codereview.chromium.org/2082773002/diff/1/net/quic/quic_connection_lo...
net/quic/quic_connection_logger.h:14: #include "base/metrics/histogram_base.h"
On 2016/06/28 22:59:44, agl wrote:
> Could HistogramBase just be forward declared since all the uses appear to be
> pointers?

Done.

[fdoray, 2016-06-29 13:32:43]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2016-06-29 13:32:44]

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

[commit-bot: I haz the power, 2016-06-29 13:33: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-06-29 14:44:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-29 14:44:48]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-29 14:46:11]

Description was changed from

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

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=agl@chromium.org
==========

to

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

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=agl@chromium.org

Committed: https://crrev.com/392099634d1ca439a1365569f558a5661af17a0d
Cr-Commit-Position: refs/heads/master@{#402805}
==========

[commit-bot: I haz the power, 2016-06-29 14:46:12]

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