Real (but naive) UDP socket sending.

chrome/browser/extensions/socket_api.h

 // Copyright (c) 2011 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_SOCKET_API_H_

[Aaron Boodman, 2011/12/02 06:06:12]

I have been trying to figure out what to do with APIs like this one that are
owned by the extension team, and/or aren't tightly related to some other module
in Chrome.

Here's what I think:

chrome/browser/extensions/api/<foo>/*

If this sgty (and Mihai), please move this there and make yourself an OWNER.

[miket_OOO, 2011/12/02 21:06:36]

Done!

 #define CHROME_BROWSER_EXTENSIONS_SOCKET_API_H_
 #pragma once
 
 #include "chrome/browser/extensions/extension_function.h"
 
+#include <string>
+
 namespace extensions {
 
+// Many of these socket functions are synchronous in the sense that
+// they don't involve blocking operations, but we've made them all
+// AsyncExtensionFunctions because the underlying UDPClientSocket
+// library wants all operations to happen on the same thread as the
+// one that created the socket. Too bad.
+
 class SocketCreateFunction : public AsyncExtensionFunction {
  public:
-  SocketCreateFunction();
-  virtual ~SocketCreateFunction();
+  SocketCreateFunction() {}

[Mihai Parparita -not on Chrome, 2011/12/01 23:39:32]

The style guide strongly discourages inline constructors and destructors:
http://www.chromium.org/developers/coding-style#TOC-Inline-functions

[miket_OOO, 2011/12/02 21:06:36]

Fixed.

+  virtual ~SocketCreateFunction() {}
   virtual bool RunImpl() OVERRIDE;
 
  protected:
   void WorkOnIOThread();
   void RespondOnUIThread();
 
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.create")
 };
 
-class SocketConnectFunction : public AsyncExtensionFunction {
+class SocketDestroyFunction : public AsyncExtensionFunction {
  public:
-  SocketConnectFunction();
-  virtual ~SocketConnectFunction();
+  SocketDestroyFunction() {}
+  virtual ~SocketDestroyFunction() {}
   virtual bool RunImpl() OVERRIDE;
 
  protected:
   void WorkOnIOThread();
   void RespondOnUIThread();
 
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.connect")
+ private:
+  int socket_id_;
+
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.destroy")
 };
 
-class SocketDisconnectFunction : public AsyncExtensionFunction {
+class SocketConnectFunction : public AsyncExtensionFunction {
  public:
-  SocketDisconnectFunction();
-  virtual ~SocketDisconnectFunction();
+  SocketConnectFunction() {}
+  virtual ~SocketConnectFunction() {}
   virtual bool RunImpl() OVERRIDE;
 
  protected:
   void WorkOnIOThread();
   void RespondOnUIThread();
 
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.disconnect")
+ private:
+  int socket_id_;
+  std::string address_;
+  int port_;
+
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.connect")
 };
 
-class SocketSendFunction : public AsyncExtensionFunction {
+class SocketCloseFunction : public AsyncExtensionFunction {
  public:
-  SocketSendFunction();
-  virtual ~SocketSendFunction();
+  SocketCloseFunction() {}
+  virtual ~SocketCloseFunction() {}
   virtual bool RunImpl() OVERRIDE;
 
  protected:
   void WorkOnIOThread();
   void RespondOnUIThread();
 
-  DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.send")
+ private:
+  int socket_id_;
+
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.close")
+};
+
+class SocketWriteFunction : public AsyncExtensionFunction {
+ public:
+  SocketWriteFunction() {}
+  virtual ~SocketWriteFunction() {}
+  virtual bool RunImpl() OVERRIDE;
+
+ protected:
+  void WorkOnIOThread();
+  void RespondOnUIThread();
+
+  int socket_id_;
+  std::string message_;
+
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.write")
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_SOCKET_API_H_