ServiceWorker: DCHECK is changed to CHECK to track the failure

content/browser/service_worker/service_worker_provider_host.cc

Index: content/browser/service_worker/service_worker_provider_host.cc
diff --git a/content/browser/service_worker/service_worker_provider_host.cc b/content/browser/service_worker/service_worker_provider_host.cc
index c914a550c9181138c4a0f56ec68b8da62c36d646..0790b9639c4a158ef12627e3334273e2994b6a52 100644
--- a/content/browser/service_worker/service_worker_provider_host.cc
+++ b/content/browser/service_worker/service_worker_provider_host.cc
@@ -282,8 +282,8 @@ void ServiceWorkerProviderHost::DisassociateRegistration() {
 
 void ServiceWorkerProviderHost::AddMatchingRegistration(
     ServiceWorkerRegistration* registration) {
-  DCHECK(ServiceWorkerUtils::ScopeMatches(
-        registration->pattern(), document_url_));
+  CHECK(
+      ServiceWorkerUtils::ScopeMatches(registration->pattern(), document_url_));

[nhiroki, 2016/08/09 07:04:28]

Can you add a TODO comment like this?

// TODO(shimazu): Change CHECK to DCHECK after https://crbug.com/634222 is
fixed.

   if (!IsContextSecureForServiceWorker())
     return;
   size_t key = registration->pattern().spec().size();