ServiceWorker: Stop don't send a message before connection established

content/renderer/service_worker/embedded_worker_instance_client_impl.cc

Index: content/renderer/service_worker/embedded_worker_instance_client_impl.cc
diff --git a/content/renderer/service_worker/embedded_worker_instance_client_impl.cc b/content/renderer/service_worker/embedded_worker_instance_client_impl.cc
index 38014c8cc62640fcf7258ef9d588d784c2c0b1c3..fa37494f5f946357b78d6605cfa12b163b535fe0 100644
--- a/content/renderer/service_worker/embedded_worker_instance_client_impl.cc
+++ b/content/renderer/service_worker/embedded_worker_instance_client_impl.cc
@@ -61,7 +61,6 @@ void EmbeddedWorkerInstanceClientImpl::StartWorker(
 void EmbeddedWorkerInstanceClientImpl::StopWorker(
     const StopWorkerCallback& callback) {
   DCHECK(ChildThreadImpl::current());
-  DCHECK(embedded_worker_id_);

[nhiroki, 2016/12/15 05:43:26]

Instead of removing this, can you move this after line 67 in order to confirm
"embedded_worker_id_.value()" is valid for RecordStopWorkerTimer?

[shimazu, 2016/12/15 08:52:28]

Done.

   // StopWorker is possible to be called twice or before StartWorker().
   if (stop_callback_ || !wrapper_)
     return;