Index: content/browser/service_worker/service_worker_process_manager.cc |
diff --git a/content/browser/service_worker/service_worker_process_manager.cc b/content/browser/service_worker/service_worker_process_manager.cc |
index cb401d2720a9d9438b3512c95f10975736fa9868..f3ff0482c8fea21f03f94af83e4555d9e8fd1687 100644 |
--- a/content/browser/service_worker/service_worker_process_manager.cc |
+++ b/content/browser/service_worker/service_worker_process_manager.cc |
@@ -193,9 +193,7 @@ |
return; |
} |
- // TODO(falken): Revert to DCHECK. Temporary check for debugging |
- // crbug.com/639193 |
- CHECK(!base::ContainsKey(instance_info_, embedded_worker_id)) |
+ DCHECK(!base::ContainsKey(instance_info_, embedded_worker_id)) |
<< embedded_worker_id << " already has a process allocated"; |
if (can_use_existing_process) { |
@@ -258,9 +256,7 @@ |
if (IsShutdown()) { |
// Shutdown already released all instances. |
- // TODO(falken): Revert to DCHECK. Temporary check for debugging |
- // crbug.com/639193 |
- CHECK(instance_info_.empty()); |
+ DCHECK(instance_info_.empty()); |
return; |
} |
@@ -275,16 +271,12 @@ |
RenderProcessHost* rph = NULL; |
if (info->second.site_instance.get()) { |
rph = info->second.site_instance->GetProcess(); |
- // TODO(falken): Revert to DCHECK. Temporary check for debugging |
- // crbug.com/639193 |
- CHECK_EQ(info->second.process_id, rph->GetID()) |
+ DCHECK_EQ(info->second.process_id, rph->GetID()) |
<< "A SiteInstance's process shouldn't get destroyed while we're " |
"holding a reference to it. Was the reference actually held?"; |
} else { |
- // TODO(falken): Revert to DCHECK. Temporary check for debugging |
- // crbug.com/639193 |
rph = RenderProcessHost::FromID(info->second.process_id); |
- CHECK(rph) |
+ DCHECK(rph) |
<< "Process " << info->second.process_id |
<< " was destroyed unexpectedly. Did we actually hold a reference?"; |
} |