DescriptionRevert of NetworkService: Destroy NetworkContexts on NetworkService teardown. (patchset #3 id:40001 of https://codereview.chromium.org/2962693002/ )
Reason for revert:
This conflicted with https://codereview.chromium.org/2951813002/, which landed at about the same time. Will fix and reland.
Original issue's description:
> NetworkService: Destroy NetworkContexts on NetworkService teardown.
>
> Also fix NetworkContexts potentially not being destroyed on connection
> error / consumer teardown, due to calling str::move twice on the same
> RequestPtr.
>
> BUG=736472
>
> Review-Url: https://codereview.chromium.org/2962693002
> Cr-Commit-Position: refs/heads/master@{#482745}
> Committed: https://chromium.googlesource.com/chromium/src/+/cfbf85b0c37adf3acccead282bf5c9b794ba4009
TBR=jam@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=736472
Review-Url: https://codereview.chromium.org/2960843005
Cr-Commit-Position: refs/heads/master@{#482763}
Committed: https://chromium.googlesource.com/chromium/src/+/1caf74b3fa2e801138207bae19471d408298268b
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|