ServiceWorker: remove EmbeddedWorkerMsg_StartWorker

content/renderer/service_worker/embedded_worker_dispatcher.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_RENDERER_SERVICE_WORKER_EMBEDDED_WORKER_DISPATCHER_H_
 #define CONTENT_RENDERER_SERVICE_WORKER_EMBEDDED_WORKER_DISPATCHER_H_
 
 #include <map>
 #include <memory>
 
@@ skipping 40 matching lines @@
     ~WorkerWrapper();
 
     blink::WebEmbeddedWorker* worker() { return worker_.get(); }
 
    private:
     ScopedChildProcessReference process_ref_;
     std::unique_ptr<blink::WebEmbeddedWorker> worker_;
     std::unique_ptr<EmbeddedWorkerDevToolsAgent> dev_tools_agent_;
   };
 
-  void OnStartWorker(const EmbeddedWorkerStartParams& params);
   void OnStopWorker(int embedded_worker_id);
   void OnResumeAfterDownload(int embedded_worker_id);
   void OnAddMessageToConsole(int embedded_worker_id,
                              ConsoleMessageLevel level,
                              const std::string& message);
 
   // These methods are used by EmbeddedWorkerInstanceClientImpl to keep
   // consistency between chromium IPC and mojo IPC.
   // TODO(shimazu): Remove them after all messages for EmbeddedWorker are
   // replaced by mojo IPC. (Tracking issue: https://crbug.com/629701)
   std::unique_ptr<WorkerWrapper> StartWorkerContext(

[falken, 2017/01/16 06:02:51]

Does comment 68-71 still apply to this method?

[shimazu, 2017/01/16 07:06:38]

No, it doesn't. 
StartWorkerContext can be alread moved to EWInstanceClient, but I'll do it when
removing EWDispatcher after mojofying two remaining messages.

For now let's move the comment after this method.

       const EmbeddedWorkerStartParams& params,
       std::unique_ptr<ServiceWorkerContextClient> context_client);
   void RegisterWorker(int embedded_worker_id,
                       std::unique_ptr<WorkerWrapper> wrapper);
   void UnregisterWorker(int embedded_worker_id);
   void RecordStopWorkerTimer(int embedded_worker_id);
 
   IDMap<std::unique_ptr<WorkerWrapper>> workers_;
   std::map<int /* embedded_worker_id */, base::TimeTicks> stop_worker_times_;
   base::WeakPtrFactory<EmbeddedWorkerDispatcher> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(EmbeddedWorkerDispatcher);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_SERVICE_WORKER_EMBEDDED_WORKER_DISPATCHER_H_