Remove all OnBeforeNetworkAccessed code above NetworkTransaction.

Remove all OnBeforeNetworkAccessed code above NetworkTransaction.

It's still needed at the NetworkTransaction layer for devtools
simulating being offline, though can probably get rid of that, if
we don't mind the offline check jumping ahead of the new throttler.

Also remove some OnBeforeNetworkAccessed code left over in
ResourceLoader.

BUG=479994
Committed: https://crrev.com/02b1f1dc9bbc5fe46f1159e00e207b4aa1733ffd
Cr-Commit-Position: refs/heads/master@{#420098}

[mmenke, 2016-09-19 22:52:48]

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

[commit-bot: I haz the power, 2016-09-19 22:53: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-20 01:44:30]

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

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

Dry run: This issue passed the CQ dry run.

[mmenke, 2016-09-20 14:35:31]

Description was changed from

==========
Remove all OnBeforeNetworkAccessed code above NetworkTransaction.

It's still needed at the NetworkTransaction layer for devtools
simulating being offline, though can probably get rid of this, if
we don't mind the offline check jumping ahead of the new throttler.

Also remove some OnBeforeNetworkAccessed code left over in
ResourceLoader.

BUG=479994
==========

to

==========
Remove all OnBeforeNetworkAccessed code above NetworkTransaction.

It's still needed at the NetworkTransaction layer for devtools
simulating being offline, though can probably get rid of that, if
we don't mind the offline check jumping ahead of the new throttler.

Also remove some OnBeforeNetworkAccessed code left over in
ResourceLoader.

BUG=479994
==========

[mmenke, 2016-09-20 14:36:07]

mmenke@chromium.org changed reviewers:
+ rdsmith@chromium.org

[mmenke, 2016-09-20 14:36:08]

Noticed we can remove yet more of this stuff.

[Randy Smith (Not in Mondays), 2016-09-21 16:17:13]

LGTM. 

What used to call into this/result in a deferred network start?

[mmenke, 2016-09-21 16:34:57]

On 2016/09/21 16:17:13, Randy Smith - Not in Mondays wrote:
> LGTM. 
> 
> What used to call into this/result in a deferred network start?

It was intended for ResourceScheduler, but never hooked up.  We had one random
consumer using it for connection timeouts, but that use was removed (Since it
was the only consumer of this API)

[mmenke, 2016-09-21 16:35:03]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2016-09-21 16:35:39]

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-21 17:50:20]

Description was changed from

==========
Remove all OnBeforeNetworkAccessed code above NetworkTransaction.

It's still needed at the NetworkTransaction layer for devtools
simulating being offline, though can probably get rid of that, if
we don't mind the offline check jumping ahead of the new throttler.

Also remove some OnBeforeNetworkAccessed code left over in
ResourceLoader.

BUG=479994
==========

to

==========
Remove all OnBeforeNetworkAccessed code above NetworkTransaction.

It's still needed at the NetworkTransaction layer for devtools
simulating being offline, though can probably get rid of that, if
we don't mind the offline check jumping ahead of the new throttler.

Also remove some OnBeforeNetworkAccessed code left over in
ResourceLoader.

BUG=479994
==========

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

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-21 17:52:31]

Description was changed from

==========
Remove all OnBeforeNetworkAccessed code above NetworkTransaction.

It's still needed at the NetworkTransaction layer for devtools
simulating being offline, though can probably get rid of that, if
we don't mind the offline check jumping ahead of the new throttler.

Also remove some OnBeforeNetworkAccessed code left over in
ResourceLoader.

BUG=479994
==========

to

==========
Remove all OnBeforeNetworkAccessed code above NetworkTransaction.

It's still needed at the NetworkTransaction layer for devtools
simulating being offline, though can probably get rid of that, if
we don't mind the offline check jumping ahead of the new throttler.

Also remove some OnBeforeNetworkAccessed code left over in
ResourceLoader.

BUG=479994

Committed: https://crrev.com/02b1f1dc9bbc5fe46f1159e00e207b4aa1733ffd
Cr-Commit-Position: refs/heads/master@{#420098}
==========

[commit-bot: I haz the power, 2016-09-21 17:52:32]

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