DevTools: remove the WorkerDevToolsManager base - implementations are diverging.

content/browser/devtools/service_worker_devtools_manager.h

 // 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.
 
 #ifndef CONTENT_BROWSER_DEVTOOLS_SERVICE_WORKER_DEVTOOLS_MANAGER_H_
 #define CONTENT_BROWSER_DEVTOOLS_SERVICE_WORKER_DEVTOOLS_MANAGER_H_
 
 #include <map>
 
 #include "base/basictypes.h"
 #include "base/memory/singleton.h"
 #include "base/memory/weak_ptr.h"
-#include "content/browser/devtools/worker_devtools_manager.h"
+#include "base/observer_list.h"
 #include "content/browser/shared_worker/shared_worker_instance.h"

[dgozman, 2015/03/07 20:46:40]

nit: unnecessary include

[pfeldman, 2015/03/08 12:17:27]

Done.

[pfeldman, 2015/03/08 12:17:27]

Done.

+#include "content/public/browser/devtools_agent_host.h"
 
 namespace content {
 
+class DevToolsAgentHostImpl;
 class ServiceWorkerDevToolsAgentHost;
 class ServiceWorkerContextCore;
 
 // Manages WorkerDevToolsAgentHost's for Service Workers.
 // This class lives on UI thread.
-class CONTENT_EXPORT ServiceWorkerDevToolsManager
-    : public WorkerDevToolsManager {
+class CONTENT_EXPORT ServiceWorkerDevToolsManager {
  public:
+  typedef std::pair<int, int> WorkerId;
+
+  class Observer {
+   public:
+    virtual void WorkerCreated(DevToolsAgentHost* host) {}
+    virtual void WorkerDestroyed(DevToolsAgentHost* host) {}
+
+   protected:
+    virtual ~Observer() {}
+  };
+
   class ServiceWorkerIdentifier {
    public:
     ServiceWorkerIdentifier(
         const ServiceWorkerContextCore* context,
         base::WeakPtr<ServiceWorkerContextCore> context_weak,
         int64 version_id,
         const GURL& url);
     ServiceWorkerIdentifier(const ServiceWorkerIdentifier& other);
     ~ServiceWorkerIdentifier();
 
     bool Matches(const ServiceWorkerIdentifier& other) const;
 
     const ServiceWorkerContextCore* context() const { return context_; }
     base::WeakPtr<ServiceWorkerContextCore> context_weak() const {
       return context_weak_;
     }
     int64 version_id() const { return version_id_; }
     GURL url() const { return url_; }
 
    private:
     const ServiceWorkerContextCore* const context_;
     const base::WeakPtr<ServiceWorkerContextCore> context_weak_;
     const int64 version_id_;
     const GURL url_;
   };
 
   // Returns the ServiceWorkerDevToolsManager singleton.
   static ServiceWorkerDevToolsManager* GetInstance();
 
+  DevToolsAgentHostImpl* GetDevToolsAgentHostForWorker(int worker_process_id,

[dgozman, 2015/03/07 20:46:40]

return ServiceWorkerDevToolsAgentHost*?

[pfeldman, 2015/03/08 12:17:27]

Clients don't need it to be that specific, you only force them to add more
includes.

+                                                       int worker_route_id);
+  void AddAllAgentHosts(DevToolsAgentHost::List* result);
+
   // Returns true when the worker must be paused on start because a DevTool
   // window for the same former ServiceWorkerIdentifier is still opened or
   // debug-on-start is enabled in chrome://serviceworker-internals.
   bool WorkerCreated(int worker_process_id,
                      int worker_route_id,
                      const ServiceWorkerIdentifier& service_worker_id);
+  void WorkerReadyForInspection(int worker_process_id, int worker_route_id);

[dgozman, 2015/03/07 20:46:40]

Should all this methods be public? Perhaps, "friend class
ServiceWorkerDevToolsAgentHost" is enough to make some of them private?

[pfeldman, 2015/03/08 12:17:27]

It is the other way around, these are the instrumentation calls that get
delegated to the agent hosts.

   void WorkerStopIgnored(int worker_process_id, int worker_route_id);
+  void WorkerDestroyed(int worker_process_id, int worker_route_id);
+  void RemoveInspectedWorkerData(WorkerId id);
+
+  void AddObserver(Observer* observer);
+  void RemoveObserver(Observer* observer);
 
   void set_debug_service_worker_on_start(bool debug_on_start) {
     debug_service_worker_on_start_ = debug_on_start;
   }
   bool debug_service_worker_on_start() const {
     return debug_service_worker_on_start_;
   }
 
  private:
   friend struct DefaultSingletonTraits<ServiceWorkerDevToolsManager>;
   friend class ServiceWorkerDevToolsAgentHost;
 
+  using AgentHostMap = std::map<WorkerId, ServiceWorkerDevToolsAgentHost*>;
+
   ServiceWorkerDevToolsManager();
-  ~ServiceWorkerDevToolsManager() override;
+  ~ServiceWorkerDevToolsManager();
 
   AgentHostMap::iterator FindExistingWorkerAgentHost(
       const ServiceWorkerIdentifier& service_worker_id);
 
+  // Resets to its initial state as if newly created.
+  void ResetForTesting();
+
+  ObserverList<Observer> observer_list_;
+  AgentHostMap workers_;
   bool debug_service_worker_on_start_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerDevToolsManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_DEVTOOLS_SERVICE_WORKER_DEVTOOLS_MANAGER_H_