Fix a DCHECK triggered by DetachableResourceLoader

Fix a DCHECK triggered by DetachableResourceHandler
under certain circumstances, when resuming a request.

BUG=411228
Committed: https://crrev.com/5c27061d1d3525e38d9328cd1b80d8ebcf8720a9
Cr-Commit-Position: refs/heads/master@{#293822}

[mmenke, 2014-09-05 20:52:52]

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

[mmenke, 2014-09-05 20:52:53]

The entirety of my coding for the week!  (No, not really, but not as far from it
as I'd like).

[davidben, 2014-09-05 20:59:27]

lgtm.

Nit: s/DetachableResourceLoader/DetachableResourceHandler/ :-)

I guess this assumes the downstream handler always calls LogBlocked. Which is
true because it's always AsyncResourceHandler... DetachableResourceHandler is
slightly screwy anyway and will probably look different if the RDH refactors we
talked about earlier ever happen.

[mmenke, 2014-09-05 21:02:56]

On 2014/09/05 20:59:27, David Benjamin wrote:
> lgtm.
> 
> Nit: s/DetachableResourceLoader/DetachableResourceHandler/ :-)
> 
> I guess this assumes the downstream handler always calls LogBlocked. Which is
> true because it's always AsyncResourceHandler... DetachableResourceHandler is
> slightly screwy anyway and will probably look different if the RDH refactors
we
> talked about earlier ever happen.

Description fixed.

LogUnblocked can safely be called without a preceding LogBlocked, and will end
up not doing anything.  Because it's weird, the BufferedResourceHandler, for
instance, doesn't log its nefarious activities.

[mmenke, 2014-09-08 18:32:53]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2014-09-08 18:48:44]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mmenke@chromium.org/545353002/20001

[commit-bot: I haz the power, 2014-09-09 00:47:48]

Committed patchset #2 (id:20001) as 6ca3c4d9ac6a8315b2bfb30c8b4a9916c87cf020

[commit-bot: I haz the power, 2014-09-10 03:49:17]

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