Refactor APIResourceController to ProfileKeyedService.

chrome/browser/extensions/api/socket/socket_api.h

 // Copyright (c) 2012 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_EXTENSIONS_API_SOCKET_SOCKET_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_SOCKET_SOCKET_API_H_
 
 #include "base/memory/ref_counted.h"
 #include "chrome/browser/extensions/api/api_function.h"
+#include "chrome/browser/extensions/api/api_resource_manager.h"
 #include "chrome/common/extensions/api/experimental_socket.h"
 #include "net/base/address_list.h"
 #include "net/base/host_resolver.h"
 
 #include <string>
 
+class IOThread;
+
 namespace net {
 class IOBuffer;
 }
 
-class IOThread;
-
 namespace extensions {
 
-class APIResourceController;
-class APIResourceEventNotifier;
+class ApiResourceEventNotifier;
+class Socket;
 
-class SocketExtensionFunction : public AsyncAPIFunction {
+class SocketExtensionFunction : public AsyncApiFunction {

[asargent_no_longer_on_chrome, 2012/07/16 18:08:25]

would it make sense to name this "SocketAsyncApiFunction" for consistency with
the SerialAsyncApiFunction superclass you added for the serial API?

[miket_OOO, 2012/07/16 19:46:02]

Done, and confirmed that usb devices are following the same scheme.

+ public:
+  SocketExtensionFunction();
+
  protected:
-  virtual ~SocketExtensionFunction() {}
+  virtual ~SocketExtensionFunction();
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
+  virtual bool PrePrepare() OVERRIDE;
   virtual void Work() OVERRIDE;
   virtual bool Respond() OVERRIDE;
+
+  ApiResourceManager<Socket>* manager_;
 };
 
 class SocketExtensionWithDnsLookupFunction : public SocketExtensionFunction {
  protected:
   SocketExtensionWithDnsLookupFunction();
   virtual ~SocketExtensionWithDnsLookupFunction();
 
   void StartDnsLookup(const std::string& hostname);
   virtual void AfterDnsLookup(int lookup_result) = 0;
 
@@ skipping 13 matching lines @@
 
 class SocketCreateFunction : public SocketExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.create")
 
   SocketCreateFunction();
 
  protected:
   virtual ~SocketCreateFunction();
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void Work() OVERRIDE;
 
  private:
   enum SocketType {
     kSocketTypeInvalid = -1,
     kSocketTypeTCP,
     kSocketTypeUDP
   };
 
   scoped_ptr<api::experimental_socket::Create::Params> params_;
   SocketType socket_type_;
   int src_id_;
-  APIResourceEventNotifier* event_notifier_;
+  ApiResourceEventNotifier* event_notifier_;
 };
 
 class SocketDestroyFunction : public SocketExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.destroy")
 
  protected:
   virtual ~SocketDestroyFunction() {}
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void Work() OVERRIDE;
 
  private:
   int socket_id_;
 };
 
 class SocketConnectFunction : public SocketExtensionWithDnsLookupFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.connect")
 
   SocketConnectFunction();
 
  protected:
   virtual ~SocketConnectFunction();
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void AsyncWorkStart() OVERRIDE;
 
   // SocketExtensionWithDnsLookupFunction:
   virtual void AfterDnsLookup(int lookup_result) OVERRIDE;
 
  private:
   void StartConnect();
   void OnConnect(int result);
 
   int socket_id_;
   std::string hostname_;
   int port_;
 };
 
 class SocketDisconnectFunction : public SocketExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.disconnect")
 
  protected:
   virtual ~SocketDisconnectFunction() {}
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void Work() OVERRIDE;
 
  private:
   int socket_id_;
 };
 
 class SocketBindFunction : public SocketExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.bind")
 
  protected:
   virtual ~SocketBindFunction() {}
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void Work() OVERRIDE;
 
  private:
   int socket_id_;
   std::string address_;
   int port_;
 };
 
 class SocketReadFunction : public SocketExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.read")
 
   SocketReadFunction();
 
  protected:
   virtual ~SocketReadFunction();
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void AsyncWorkStart() OVERRIDE;
   void OnCompleted(int result, scoped_refptr<net::IOBuffer> io_buffer);
 
  private:
   scoped_ptr<api::experimental_socket::Read::Params> params_;
 };
 
 class SocketWriteFunction : public SocketExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.write")
 
   SocketWriteFunction();
 
  protected:
   virtual ~SocketWriteFunction();
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void AsyncWorkStart() OVERRIDE;
   void OnCompleted(int result);
 
  private:
   int socket_id_;
   scoped_refptr<net::IOBuffer> io_buffer_;
   size_t io_buffer_size_;
 };
 
 class SocketRecvFromFunction : public SocketExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.recvFrom")
 
   SocketRecvFromFunction();
 
  protected:
   virtual ~SocketRecvFromFunction();
 
-  // AsyncAPIFunction
+  // AsyncApiFunction
   virtual bool Prepare() OVERRIDE;
   virtual void AsyncWorkStart() OVERRIDE;
   void OnCompleted(int result,
                    scoped_refptr<net::IOBuffer> io_buffer,
                    const std::string& address,
                    int port);
 
  private:
   scoped_ptr<api::experimental_socket::RecvFrom::Params> params_;
 };
 
 class SocketSendToFunction : public SocketExtensionWithDnsLookupFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.sendTo")
 
   SocketSendToFunction();
 
  protected:
   virtual ~SocketSendToFunction();
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void AsyncWorkStart() OVERRIDE;
   void OnCompleted(int result);
 
   // SocketExtensionWithDnsLookupFunction:
   virtual void AfterDnsLookup(int lookup_result) OVERRIDE;
 
  private:
   void StartSendTo();
 
   int socket_id_;
   scoped_refptr<net::IOBuffer> io_buffer_;
   size_t io_buffer_size_;
   std::string hostname_;
   int port_;
 };
 
 class SocketSetKeepAliveFunction : public SocketExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.setKeepAlive")
 
   SocketSetKeepAliveFunction();
 
  protected:
   virtual ~SocketSetKeepAliveFunction();
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void Work() OVERRIDE;
 
  private:
   scoped_ptr<api::experimental_socket::SetKeepAlive::Params> params_;
 };
 
 class SocketSetNoDelayFunction : public SocketExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.setNoDelay")
 
   SocketSetNoDelayFunction();
 
  protected:
   virtual ~SocketSetNoDelayFunction();
 
-  // AsyncAPIFunction:
+  // AsyncApiFunction:
   virtual bool Prepare() OVERRIDE;
   virtual void Work() OVERRIDE;
 
  private:
   scoped_ptr<api::experimental_socket::SetNoDelay::Params> params_;
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_SOCKET_SOCKET_API_H_