Fix DetachableResourceHandler when OnResponseCompleted is deferred.

Fix DetachableResourceHandler when OnResponseCompleted is deferred.

https://codereview.chromium.org/2526983002/ made it so that when the
next ResourceHandler defers OnResponseCompleted, and then the
ResourceHandler is detached, DetachableResourceHandler wouldn't tear
down the request.

This probably never happpens in practice, but it was supported before
that CL.

BUG=687240
Review-Url: https://codereview.chromium.org/2673453002
Cr-Commit-Position: refs/heads/master@{#447660}
Committed: https://chromium.googlesource.com/chromium/src/+/59420d8f9e1521355d98b03cbf2748f7ad0dbb07

[mmenke, 2017-02-01 20:24:19]

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

[mmenke, 2017-02-01 20:24:19]

csharrison:  I do have a test for this (Which is how I found it), but it's part
of the OnWillRead async CL, and was too lazy to pull it out.  I'll be sending
out that CL later today, most likely, though if you don't have time to review
that CL before I leave, that's fine.

Please do review this on, though.  If you feel strongly about tests, I can
unentangle them from the other CL (Which shouldn't be too hard, and isn't an
entirely unreasonable request, I'd just have to remove their use of the new
OnWillRead magic)

[Charlie Harrison, 2017-02-01 23:00:26]

LGTM this is fine to land imo

[mmenke, 2017-02-01 23:00:40]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2017-02-01 23:01:42]

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-02-02 00:41:10]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485990040811640, "parent_rev":
"6ba98183e9bfaf01aaa8c247bd3d1c13da47df88", "commit_rev":
"59420d8f9e1521355d98b03cbf2748f7ad0dbb07"}

[commit-bot: I haz the power, 2017-02-02 00:41:45]

Description was changed from

==========
Fix DetachableResourceHandler when OnResponseCompleted is deferred.

https://codereview.chromium.org/2526983002/ made it so that when the
next ResourceHandler defers OnResponseCompleted, and then the
ResourceHandler is detached, DetachableResourceHandler wouldn't tear
down the request.

This probably never happpens in practice, but it was supported before
that CL.

BUG=687240
==========

to

==========
Fix DetachableResourceHandler when OnResponseCompleted is deferred.

https://codereview.chromium.org/2526983002/ made it so that when the
next ResourceHandler defers OnResponseCompleted, and then the
ResourceHandler is detached, DetachableResourceHandler wouldn't tear
down the request.

This probably never happpens in practice, but it was supported before
that CL.

BUG=687240

Review-Url: https://codereview.chromium.org/2673453002
Cr-Commit-Position: refs/heads/master@{#447660}
Committed:
https://chromium.googlesource.com/chromium/src/+/59420d8f9e1521355d98b03cbf27...
==========

[commit-bot: I haz the power, 2017-02-02 00:41:50]

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