Send WorkerHostMsg_WorkerConnected in EmbeddedSharedWorkerStub::OnConnect.

content/renderer/shared_worker/embedded_shared_worker_stub.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/shared_worker/embedded_shared_worker_stub.h"
 
 #include "base/message_loop/message_loop_proxy.h"
 #include "content/child/scoped_child_process_reference.h"
 #include "content/child/shared_worker_devtools_agent.h"
 #include "content/child/webmessageportchannel_impl.h"
@@ skipping 124 matching lines @@
 }
 
 void EmbeddedSharedWorkerStub::OnConnect(int sent_message_port_id,
                                          int routing_id) {
   WebMessagePortChannelImpl* channel =
       new WebMessagePortChannelImpl(routing_id,
                                     sent_message_port_id,
                                     base::MessageLoopProxy::current().get());
   if (runing_) {
     impl_->connect(channel);
+    Send(new WorkerHostMsg_WorkerConnected(channel->message_port_id(),
+                                           route_id_));

[kinuko, 2014/03/10 13:26:02]

Should we add ConnectToChannel() method which always does impl_->connect() and
Send(Connected) together?  (I'm fine with the current code too though)

[horo, 2014/03/11 01:14:12]

Done.

   } else {
     // If two documents try to load a SharedWorker at the same time, the
     // WorkerMsg_Connect for one of the documents can come in before the
     // worker is started. Just queue up the connect and deliver it once the
     // worker starts.
     pending_channels_.push_back(channel);
   }
 }
 
 void EmbeddedSharedWorkerStub::OnTerminateWorkerContext() {
   runing_ = false;
   impl_->terminateWorkerContext();
 }
 
 }  // namespace content