ServiceWorker: Mojofication of EWInstance::StopWorker

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 49 matching lines @@
     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(
       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);
+  // Return false if no worker found
+  bool SetStopWorkerTimer(int embedded_worker_id);

[horo, 2016/09/13 08:25:33]

It doesn't set a timer.

RecordStopWorkerTime is better.

[shimazu, 2016/09/20 04:56:20]

Done.

 
   IDMap<WorkerWrapper, IDMapOwnPointer> 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_