DevTools: Partially redesigned DevToolsAndroidBridge and AndroidDeviceManager

chrome/browser/devtools/device/android_device_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 CHROME_BROWSER_DEVTOOLS_DEVICE_ANDROID_DEVICE_MANAGER_H_
 #define CHROME_BROWSER_DEVTOOLS_DEVICE_ANDROID_DEVICE_MANAGER_H_
 
 #include <vector>
 
 #include "base/memory/ref_counted.h"
 #include "base/threading/non_thread_safe.h"
 #include "chrome/browser/profiles/profile.h"
 #include "content/public/browser/browser_thread.h"
 #include "ui/gfx/size.h"
 
 namespace net {
 class StreamSocket;
 }
 
 class AndroidDeviceManager
     : public base::RefCountedThreadSafe<AndroidDeviceManager>,
       public base::NonThreadSafe {
  public:
   typedef base::Callback<void(int, const std::string&)> CommandCallback;
   typedef base::Callback<void(int result, net::StreamSocket*)> SocketCallback;
+  typedef base::Callback<void(const std::vector<std::string>&)> SerialsCallback;
 
   struct BrowserInfo {
     BrowserInfo();
 
     enum Type {
       kTypeChrome,
       kTypeWebView,
       kTypeOther
     };
 
     std::string socket_name;
     std::string display_name;
     Type type;
   };
 
   struct DeviceInfo {
     DeviceInfo();
     ~DeviceInfo();
 
     std::string model;
+    bool connected;
     gfx::Size screen_size;
     std::vector<BrowserInfo> browser_info;
   };
 
   typedef base::Callback<void(const DeviceInfo&)> DeviceInfoCallback;
 
-  class Device : public base::RefCounted<Device>,
+  class DeviceProvider : public base::RefCountedThreadSafe<DeviceProvider> {
+   public:
+    typedef AndroidDeviceManager::SerialsCallback SerialsCallback;
+    typedef AndroidDeviceManager::DeviceInfoCallback DeviceInfoCallback;
+    typedef AndroidDeviceManager::SocketCallback SocketCallback;
+    typedef AndroidDeviceManager::CommandCallback CommandCallback;
+
+    virtual void QueryDevices(const SerialsCallback& callback) = 0;
+
+    virtual void QueryDeviceInfo(const std::string& serial,
+                                 const DeviceInfoCallback& callback) = 0;
+
+    virtual void OpenSocket(const std::string& serial,
+                            const std::string& socket_name,
+                            const SocketCallback& callback) = 0;
+
+    virtual void SendJsonRequest(const std::string& serial,
+                                 const std::string& socket_name,
+                                 const std::string& request,
+                                 const CommandCallback& callback);
+
+    virtual void HttpUpgrade(const std::string& serial,
+                             const std::string& socket_name,
+                             const std::string& url,
+                             const SocketCallback& callback);
+
+    virtual void ReleaseDevice(const std::string& serial);
+
+   protected:
+    friend class base::RefCountedThreadSafe<DeviceProvider>;
+    DeviceProvider();
+    virtual ~DeviceProvider();
+  };
+
+  typedef std::vector<scoped_refptr<DeviceProvider> > DeviceProviders;
+
+  class AndroidWebSocket : public base::RefCountedThreadSafe<AndroidWebSocket> {
+   public:
+    class Delegate {
+     public:
+      virtual void OnSocketOpened() = 0;
+      virtual void OnFrameRead(const std::string& message) = 0;
+      virtual void OnSocketClosed(bool closed_by_device) = 0;
+
+     protected:
+      virtual ~Delegate() {}
+    };
+
+    AndroidWebSocket() {}
+
+    virtual void Connect() = 0;
+    virtual void Disconnect() = 0;
+    virtual void SendFrame(const std::string& message) = 0;
+
+   protected:
+    virtual ~AndroidWebSocket() {}
+
+   private:
+    friend class base::RefCountedThreadSafe<AndroidWebSocket>;
+
+    DISALLOW_COPY_AND_ASSIGN(AndroidWebSocket);
+  };
+
+  class Device : public base::RefCountedThreadSafe<Device>,

[pfeldman, 2014/05/26 09:57:14]

Lets swap them back for better diff.

[vkuzkokov, 2014/05/26 10:54:03]

Done.

                  public base::NonThreadSafe {
-   protected:
-    friend class AndroidDeviceManager;
-
+   public:
+    typedef AndroidDeviceManager::DeviceInfoCallback DeviceInfoCallback;
     typedef AndroidDeviceManager::CommandCallback CommandCallback;
     typedef AndroidDeviceManager::SocketCallback SocketCallback;
-    typedef AndroidDeviceManager::DeviceInfoCallback DeviceInfoCallback;
 
-    Device(const std::string& serial, bool is_connected);
+    void QueryDeviceInfo(const DeviceInfoCallback& callback);
 
-    virtual void QueryDeviceInfo(const DeviceInfoCallback& callback) = 0;
+    void OpenSocket(const std::string& socket_name,
+                    const SocketCallback& callback);
 
-    virtual void OpenSocket(const std::string& socket_name,
-                            const SocketCallback& callback) = 0;
+    void SendJsonRequest(const std::string& socket_name,
+                         const std::string& request,
+                         const CommandCallback& callback);
 
-    virtual void HttpQuery(const std::string& socket_name,
-                           const std::string& request,
-                           const CommandCallback& callback);
+    void HttpUpgrade(const std::string& socket_name,
+                     const std::string& url,
+                     const SocketCallback& callback);
+
+    scoped_refptr<AndroidWebSocket> CreateWebSocket(
+        const std::string& socket_name,
+        const std::string& url,
+        AndroidWebSocket::Delegate* delegate);
 
     std::string serial() { return serial_; }
-    bool is_connected() { return is_connected_; }
 
-    friend class base::RefCounted<Device>;
+   private:
+    friend class AndroidDeviceManager;
+    Device(scoped_refptr<base::MessageLoopProxy> device_message_loop,
+           const scoped_refptr<DeviceProvider>& provider,
+           const std::string& serial);
+
+    friend class base::RefCountedThreadSafe<Device>;
     virtual ~Device();
 
-   private:
-    const std::string serial_;
-    const bool is_connected_;
+    scoped_refptr<base::MessageLoopProxy> device_message_loop_;
+    scoped_refptr<DeviceProvider> provider_;

[pfeldman, 2014/05/26 09:57:14]

On what thread will this be released?

[vkuzkokov, 2014/05/26 10:54:03]

UI. Same way as in DevToolsAndroidBridge. It does seem dodgy though.

+    std::string serial_;
+    base::WeakPtrFactory<Device> weak_factory_;
 
     DISALLOW_COPY_AND_ASSIGN(Device);
   };
 
   typedef std::vector<scoped_refptr<Device> > Devices;
+  typedef base::Callback<void(const Devices&)> DevicesCallback;
 
-  class DeviceProvider
-      : public base::RefCountedThreadSafe<
-            DeviceProvider,
-            content::BrowserThread::DeleteOnUIThread> {
-   protected:
-    friend class AndroidDeviceManager;
+  static scoped_refptr<AndroidDeviceManager> Create();
 
-    typedef base::Callback<void(const Devices&)> QueryDevicesCallback;
+  void QueryDevices(scoped_refptr<base::MessageLoopProxy> device_message_loop,
+                    const DeviceProviders& providers,
+                    const DevicesCallback& callback);
 
-    virtual void QueryDevices(const QueryDevicesCallback& callback) = 0;
-
-   protected:
-    friend struct
-        content::BrowserThread::DeleteOnThread<content::BrowserThread::UI>;
-    friend class base::DeleteHelper<DeviceProvider>;
-
-    DeviceProvider();
-    virtual ~DeviceProvider();
+  struct DeviceDescriptor {
+    scoped_refptr<DeviceProvider> provider;
+    std::string serial;
   };
 
- public:
-  static scoped_refptr<AndroidDeviceManager> Create();
-
-  typedef std::vector<scoped_refptr<DeviceProvider> > DeviceProviders;
-  typedef base::Callback<void (const std::vector<std::string>&)>
-      QueryDevicesCallback;
-
-  void QueryDevices(const DeviceProviders& providers,
-                    const QueryDevicesCallback& callback);
-
-  void Stop();
-
-  bool IsConnected(const std::string& serial);
-
-  void QueryDeviceInfo(const std::string& serial,
-                       const DeviceInfoCallback& callback);
-
-  void OpenSocket(const std::string& serial,
-                  const std::string& socket_name,
-                  const SocketCallback& callback);
-
-  void HttpQuery(const std::string& serial,
-                 const std::string& socket_name,
-                 const std::string& request,
-                 const CommandCallback& callback);
-
-  void HttpUpgrade(const std::string& serial,
-                   const std::string& socket_name,
-                   const std::string& url,
-                   const SocketCallback& callback);
+  typedef std::vector<DeviceDescriptor> DeviceDescriptors;
 
  private:
+  friend class base::RefCountedThreadSafe<AndroidDeviceManager>;
   AndroidDeviceManager();
-
-  friend class base::RefCountedThreadSafe<AndroidDeviceManager>;
-
   virtual ~AndroidDeviceManager();
 
-  void QueryNextProvider(
-      const QueryDevicesCallback& callback,
-      const DeviceProviders& providers,
-      const Devices& total_devices,
-      const Devices& new_devices);
+  void UpdateDevices(scoped_refptr<base::MessageLoopProxy> device_message_loop,
+                     const DevicesCallback& callback,
+                     const DeviceDescriptors& descriptors);
 
-  Device* FindDevice(const std::string& serial);
-
-  typedef std::map<std::string, scoped_refptr<Device> > DeviceMap;
-  DeviceMap devices_;
-
-  bool stopped_;
+  typedef std::map<std::string, base::WeakPtr<Device> > DeviceWeakMap;
+  DeviceWeakMap devices_;
 };
 
 #endif  // CHROME_BROWSER_DEVTOOLS_DEVICE_ANDROID_DEVICE_MANAGER_H_