tree 6a83baeb7749eda4e573a1238fddaeac902387a3
parent 2faf59d54deaa08001ac5e07f5e7dfa8bf29132e
author Kenichi Ishibashi <bashi@chromium.org> 1544767486 +0000
committer Commit Bot <commit-bot@chromium.org> 1544767486 +0000

service worker: Remove a DCHECK from ServiceWorkerSubresourceLoader

Before this CL we had a DCHECK in OnFallback() to make sure
that the loader isn't the only one who has a reference to
|fallback_factory_|. The intent was to make sure that the
fallback factory lives long enough to handle the response,
but it turned out we wouldn't need to ensure that because
we don't need the request in that case.

Background context:
I hit this DCHECK while implementing PoC CL of generating
code cache in idle tasks.

Bug: 915081
Change-Id: Ia636edf31c6c1cf9b95ae0a3db9f913d578bd733
Reviewed-on: https://chromium-review.googlesource.com/c/1377487
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#616596}