Revert https://codereview.chromium.org/2860033003

Revert https://codereview.chromium.org/2860033003

This refactored how the ProxyConfigTrackerImpl starts up, and broke
proxy configuration on ChromeOS in the process. The issue has been at
least partially fixed on trunk, but issue does not appear to be
completely fixed.

This also reverts https://codereview.chromium.org/2872903006, a small
CL that has no impact on behavior, which was landed on top of the
other CL.

Revert "Don't delay creation of system URLRequestContext until first use."

This reverts commit 02505dac8a35d644e38d27b01e48645ce39599bc.

Revert "Use URLRequestContextStorage for the SystemURLRequestContext."

This reverts commit f2e53b2bec633349e045fd4566d841289a295247.

TBR=mmenke@chromium.org
BUG=734565
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2972223002
Cr-Commit-Position: refs/branch-heads/3112@{#562}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed: https://chromium.googlesource.com/chromium/src/+/28255e31f57b4aa645e1a3d9acc02a133537d347

[mmenke, 2017-07-10 18:19:30]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2017-07-10 18:19:45]

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, 2017-07-10 18:19:46]

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

[commit-bot: I haz the power, 2017-07-10 18:19:47]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[mmenke, 2017-07-10 18:21:21]

Description was changed from

==========
Revert https://codereview.chromium.org/2860033003

This refactored how the ProxyConfigTrackerImpl starts up, and broke
proxy configuration on ChromeOS in the process. The issue has been at
least partially fixed on trunk, but issue does not appear to be
completely fixed.

This also reverts https://codereview.chromium.org/2872903006, a small
CL that has no impact on behavior, which was landed on top of the
other CL.

Revert "Don't delay creation of system URLRequestContext until first use."

This reverts commit 02505dac8a35d644e38d27b01e48645ce39599bc.

Revert "Use URLRequestContextStorage for the SystemURLRequestContext."

This reverts commit f2e53b2bec633349e045fd4566d841289a295247.

BUG=734565
==========

to

==========
Revert https://codereview.chromium.org/2860033003

This refactored how the ProxyConfigTrackerImpl starts up, and broke
proxy configuration on ChromeOS in the process. The issue has been at
least partially fixed on trunk, but issue does not appear to be
completely fixed.

This also reverts https://codereview.chromium.org/2872903006, a small
CL that has no impact on behavior, which was landed on top of the
other CL.

Revert "Don't delay creation of system URLRequestContext until first use."

This reverts commit 02505dac8a35d644e38d27b01e48645ce39599bc.

Revert "Use URLRequestContextStorage for the SystemURLRequestContext."

This reverts commit f2e53b2bec633349e045fd4566d841289a295247.

TBR=mmenke@chromium.org
BUG=734565
==========

[mmenke, 2017-07-10 18:21:25]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2017-07-10 18:21:46]

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, 2017-07-10 18:21:47]

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

[commit-bot: I haz the power, 2017-07-10 18:21:48]

CLs for remote refs other than refs/heads/master must contain NOTRY=true and
NOPRESUBMIT=true in order for the CQ to process them

[mmenke, 2017-07-10 18:37:27]

Description was changed from

==========
Revert https://codereview.chromium.org/2860033003

This refactored how the ProxyConfigTrackerImpl starts up, and broke
proxy configuration on ChromeOS in the process. The issue has been at
least partially fixed on trunk, but issue does not appear to be
completely fixed.

This also reverts https://codereview.chromium.org/2872903006, a small
CL that has no impact on behavior, which was landed on top of the
other CL.

Revert "Don't delay creation of system URLRequestContext until first use."

This reverts commit 02505dac8a35d644e38d27b01e48645ce39599bc.

Revert "Use URLRequestContextStorage for the SystemURLRequestContext."

This reverts commit f2e53b2bec633349e045fd4566d841289a295247.

TBR=mmenke@chromium.org
BUG=734565
==========

to

==========
Revert https://codereview.chromium.org/2860033003

This refactored how the ProxyConfigTrackerImpl starts up, and broke
proxy configuration on ChromeOS in the process. The issue has been at
least partially fixed on trunk, but issue does not appear to be
completely fixed.

This also reverts https://codereview.chromium.org/2872903006, a small
CL that has no impact on behavior, which was landed on top of the
other CL.

Revert "Don't delay creation of system URLRequestContext until first use."

This reverts commit 02505dac8a35d644e38d27b01e48645ce39599bc.

Revert "Use URLRequestContextStorage for the SystemURLRequestContext."

This reverts commit f2e53b2bec633349e045fd4566d841289a295247.

TBR=mmenke@chromium.org
BUG=734565
NOTRY=true
NOPRESUBMIT=true
==========

[mmenke, 2017-07-10 18:37:36]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2017-07-10 18:37:54]

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, 2017-07-10 18:38:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499711856236940, "parent_rev":
"a265a036fa4f54c791897c65597a9b8e6dacc999", "commit_rev":
"28255e31f57b4aa645e1a3d9acc02a133537d347"}

[commit-bot: I haz the power, 2017-07-10 18:38:37]

Description was changed from

==========
Revert https://codereview.chromium.org/2860033003

This refactored how the ProxyConfigTrackerImpl starts up, and broke
proxy configuration on ChromeOS in the process. The issue has been at
least partially fixed on trunk, but issue does not appear to be
completely fixed.

This also reverts https://codereview.chromium.org/2872903006, a small
CL that has no impact on behavior, which was landed on top of the
other CL.

Revert "Don't delay creation of system URLRequestContext until first use."

This reverts commit 02505dac8a35d644e38d27b01e48645ce39599bc.

Revert "Use URLRequestContextStorage for the SystemURLRequestContext."

This reverts commit f2e53b2bec633349e045fd4566d841289a295247.

TBR=mmenke@chromium.org
BUG=734565
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
Revert https://codereview.chromium.org/2860033003

This refactored how the ProxyConfigTrackerImpl starts up, and broke
proxy configuration on ChromeOS in the process. The issue has been at
least partially fixed on trunk, but issue does not appear to be
completely fixed.

This also reverts https://codereview.chromium.org/2872903006, a small
CL that has no impact on behavior, which was landed on top of the
other CL.

Revert "Don't delay creation of system URLRequestContext until first use."

This reverts commit 02505dac8a35d644e38d27b01e48645ce39599bc.

Revert "Use URLRequestContextStorage for the SystemURLRequestContext."

This reverts commit f2e53b2bec633349e045fd4566d841289a295247.

TBR=mmenke@chromium.org
BUG=734565
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2972223002
Cr-Commit-Position: refs/branch-heads/3112@{#562}
Cr-Branched-From:
b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed:
https://chromium.googlesource.com/chromium/src/+/28255e31f57b4aa645e1a3d9acc0...
==========

[commit-bot: I haz the power, 2017-07-10 18:38:38]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/28255e31f57b4aa645e1a3d9acc0...